> I do not like the name hideCursor when it changes in state. > Change the message to "Hide Mouse Cursor when over display window"
Where is this? The advanced settings tab, the code comments, etc. (I've never heard "in state" before.) I made the information in the advanced settings more descriptive, so I might have fixed this. > When do we need to call hideCursor on all the set methods. Could it not > be incorporated in the buildHtml method which is called once per > serviceItem being loaded to build the HTML. Yes, I guess it could be. I just moved it there. -- https://code.launchpad.net/~floft/openlp/hidecursor/+merge/46429 Your team OpenLP Core is subscribed to branch lp:openlp. _______________________________________________ Mailing list: https://launchpad.net/~openlp-core Post to : openlp-core@lists.launchpad.net Unsubscribe : https://launchpad.net/~openlp-core More help : https://help.launchpad.net/ListHelp
