Skip to content

application.py: allow users to escape the fullscreen mode using F7/F11

Alexander Rüedlinger requested to merge lexruee:fix-fullscreen-shortcut into master

First of all, lollypop is awesome 😄. Keep up the good work!

Currently (c7f820f9), lollypop does not allow users to escape the fullscreen mode using the F7/F11 shortcuts. In most applications, such as web browsers, it is common that the same shortcut can be used to escape the fullscreen mode.

In short, this pull-request introduces small changes so that the fullscreen mode can be escaped using F7/F11.

Cheers, Alex

Edit: I think the class Application could be refactored by eliminating the private instance variable __is_fs by using the public method is_fullscreen(). Moreover, the method is_fullscreen() could be adjusted (see below) to use the private instance variable __fs to check whether lollypop is in fullscreen mode.

    def is_fullscreen(self):
        """
            Return True if application is fullscreen
        """
        return self.__fs is not None

Merge request reports