-
Notifications
You must be signed in to change notification settings - Fork 35
Bring back fullscreen hint #436
base: xenial
Are you sure you want to change the base?
Conversation
ac57330
to
ada363e
Compare
Hmm... While the hint is correct sometimes, I'm having a lot of trouble triggering the "swipe up" action. Is it the bottom edge swipe? Can we make that clearer? |
Yes, it's a bottom swipe which is exactly the same as before. |
Hmm... maybe something like We could also bring this text up from the bottom of the screen rather than the top. Other options include maintaining the bottom edge interaction method, where a hint appears for a short time followed by a chevron... but we'd need to be careful to avoid the chevron getting in the way of page content or conflicting with the real tab pip. |
Oh, one more thing (sorry)... On my tablet, which doesn't have a physical keyboard, the hint unhelpfully says "Press ESC to exit full screen" |
I moved it to the bottom and changed the text. I've also fixed issue with the text when in wide layout. |
yeah, for some reason if you try to roll your own bottom edge - not the one with cheveron/hint, it doesn't trigger as well, I think there are extra things they are doing for swipe usability too |
qtwebbrowser does a scroll tracker and when you drag on the screen it brings a hidden bar in sight |
Morph also has one -- well, webbrowser-app had one, and it got disabled when it became Morph. #333 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
290aee3
to
601e037
Compare
Yeah sorry, I combined all my open PRs into one, didn't know fixing conflicts does this 😅 |
Added back the fullscreen hint that shows up when the webview goes fullscreen i.e. fullscreen video