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 regression in overlay of full-screen mode and disables full-screen in some widgets #2877

Merged
merged 2 commits into from
Mar 11, 2021

Conversation

ksraj123
Copy link
Member

This PR does the following

  • Fixing issue with overlay, steps to reproduce - maximise a widget window by dragging it to the top then minimise it using the minimise icon, the overlay sticks around and could not be removed unless the widgetWindow is closed. Ideal behaviour should be that the overlay should appear only on dragging the window to the top to indicate the window could be maximised and should not stick around in after the window is minimised.
    Kapture 2021-03-11 at 07 03 05

  • Maximise functionality in all forms (double click, drag to top and maximise button) disabled for the help, tempo, pitch slider, pitch staircase and timbre widgets.

Please share feedback and suggestions. Thanks
@meganindya

@ksraj123 ksraj123 changed the title Fullscreen improvements Fixes regression in overlay of full-screen mode and disables full-screen in some widgets Mar 11, 2021
Copy link
Member

@meganindya meganindya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@meganindya meganindya merged commit b5aca5a into sugarlabs:master Mar 11, 2021
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.

2 participants