Code review comment for lp:~trb143/openlp/bug-850254

Revision history for this message
Raoul Snyman (raoul-snyman) wrote :

I agree with Jonathan, I think we should probably move away from a "toggle" mentality, and move to a "hide|theme|desktop|show" idea - then we can keep the API the same, just adding new items, keeping the API backwards compatible.

Then we can show 4 buttons in the toolbar, and using the poll event, indicate which button is active.

Further thoughts/ideas/comments/etc ?

« Back to merge proposal