Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes 2 bugs with fullscreen: #644

Closed
wants to merge 1 commit into from
Closed

Fixes 2 bugs with fullscreen: #644

wants to merge 1 commit into from

Conversation

ivantodorovich
Copy link

  1. Properly toggle active class on $('.btn-os.fullscreen') when exiting
    fullscreen from the maximize button. Before this commit the button class
    won't be toggled and the button would show incorrect icons.
  2. Fixes a bug where it got weird and unintuitive if the user wanted to
    incorrectly unmaximize the window while being in fullscreen -- with the
    window already unmaximized (but in fullscreen). Now, if the user
    clicks the maximize button while being in fullscreen, it not only exits
    fullscreen but also unmaximizes it.

PS: Mi ingles apesta.. espero se entienda!

1. Properly toggle active class on $('.btn-os.fullscreen') when exiting
fullscreen from the maximize button. Before this commit the button class
won't be toggled and the button would show incorrect icons.

2. Fixes a bug where it got weird and unintuitive if the user wanted to
incorrectly unmaximize the window while being in fullscreen -- while the
window was already unmaximized (but in fullscreen). Now, if the user
clicks the maximize button while being in fullscreen, it not only exits
fullscreen but also unmaximizes it.
HammamSamara pushed a commit to HammamSamara/popcorn-app that referenced this pull request Apr 5, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant