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

Difficulties closing palette when in JavaScript view on small screen #2000

Closed
DavidWhaleMEF opened this issue Apr 29, 2017 · 8 comments
Closed

Comments

@DavidWhaleMEF
Copy link

See screenshot below. When running on a small screen in javascript mode, I can't seen to find a sensible way to close the palette when it opens with very long names, as the scroll bars don't scroll outside of the popup region.

screen shot 2017-04-29 at 11 53 01

Tried on both live and beta today.

A small workaround seems to be to press the arrow that expands and collapses the advanced palette, but I found this by chance, and I'm not sure it's obvious to all users.

Just sending this in as feedback, as I suspect this will affect the mobile experience if anyone goes into javascript view to patch a couple of lines of code when on mobile.

BETA
pxt.microbit.org version: 0.12.5
Microsoft MakeCode version: 0.12.63
C++ runtime version: v2.0.0-rc9

LIVE
pxt.microbit.org version: 0.9.77
Microsoft MakeCode version: 0.12.25
C++ runtime version: v2.0.0-rc8

@tballmsft
Copy link
Contributor

tballmsft commented Apr 29, 2017

Good catch. Blockly recognized the ESC key to close the flyout. We should at least have a keystroke.

But this doesn't solve the problem on mobile. We could also close the flyout if you click on it (but not on a block). But that isn't the greatest solution either.

Perhaps we need a X (close) button on the flyout in mobile?

@pelikhan
Copy link
Member

pelikhan commented Apr 29, 2017 via email

@DavidWhaleMEF
Copy link
Author

Doesn't work for me? I have to click outside of the grey area, and if the grey area takes up the whole screen, there is no way to click outside the grey area.

@pelikhan
Copy link
Member

pelikhan commented May 2, 2017 via email

@DavidWhaleMEF
Copy link
Author

Ah I see, great, thanks, I misinterpreted.

So you mean 'we should fix this problem by changing the functionality, so that when you click on the grey area it closes the fly out'.

Got it. Thanks :)

@samelhusseini
Copy link
Contributor

Matching blockly's toolbox behaviour, we don't close the flyout when you tap the background of the flyout. For big flyouts on small screens, you can still close the flyout by tapping the category again. (Just like in Blockly).

@samelhusseini
Copy link
Contributor

This has been fixed in master. We recognise the escape key and limit the width of a block in the flyout.

@lock
Copy link

lock bot commented Sep 26, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Sep 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants