← Back to team overview

openlp-core team mailing list archive

Re: [Merge] lp:~smpettit/openlp/preview-edit-custom into lp:openlp

 

"Fixing" this is like going to church, there are different routes I can take....

1.  Remove the ' ' from the plugin name and replace it with '_'
2.  Add code as suggested above to replace ' ' with '.'
3.  Add code to slidecontroller.py (where the signal is being emitted) to replace ' ' with '.'

I'm in favor of 1 and make this a "standard" coding practice "When using multiple word item names, the words are to be separated by a '_", not a ' '...  ie.  'Custom_Slides' not 'Custom Slides'.  "

Just let me know which route to take
-- 
https://code.launchpad.net/~smpettit/openlp/preview-edit-custom/+merge/64781
Your team OpenLP Core is subscribed to branch lp:openlp.


References