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

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

Can you also change the method names to make them a little more descriptive please? Like...

  def displayStatus(self):
      """
      Obtain the display status of system.
      """

  def changeDisplayStatus(self, action):
      """
      Toggle the display of the system including the status button.
      """

review: Needs Fixing

« Back to merge proposal