GithubHelp home page GithubHelp logo

Comments (6)

alyssaxuu avatar alyssaxuu commented on May 27, 2024 1

Should work fine now, I've tested it while recording and clicking on the page itself and have been able to close with the Esc key.

from screenity.

ceskyDJ avatar ceskyDJ commented on May 27, 2024

The same problem is with drawing submenu.

from screenity.

ceskyDJ avatar ceskyDJ commented on May 27, 2024

This still doesn't work (I have updated 3.1.2). After the bubble disappears, ESC key press doesn't close drawing neither blur submenu.

I mean this by "bubble":
image

Just to be sure... Steps for reproducing:

  1. Open blur or drawing submenu of toolbar by clicking on button in toolbar.
  2. Use selected tool from the active submenu (e.g. pencil) on the screen.
  3. Press ESC key.
  4. Nothing happens. Should deactivate the selected tool and close the submenu.

It this how it is supposed to work or did I just completely misunderstand that?

from screenity.

ceskyDJ avatar ceskyDJ commented on May 27, 2024

I've tested it while recording and clicking on the page itself and have been able to close with the Esc key.

Hmm, so it should work as I think. Where the hell is the problem after?

ESC press doesn't work just after the bubble disappears (I just wait for the disappearing without any other interaction), too. Maybe it'll help you a bit.

from screenity.

alyssaxuu avatar alyssaxuu commented on May 27, 2024

Ah I understand now, you mean after the toast disappears - I didn't actually implement it to close with Esc outside the timeframe when the tooltips are visible. I think my reasoning was that a page may use Esc for other purposes, so it might cause some conflict in some cases (the same reason why I avoid using shortcuts besides the ones defined in the manifest file). I've had issues anyway with keys in general as some websites use focus trapping which is hard to override.

I could consider adding them in, I'd just have to see how it feels in context. I could see it potentially being problematic in pages that use popups / modals that can be dismissed with Esc, but I would have to see. I don't think it would be challenging to implement in any case.

from screenity.

ceskyDJ avatar ceskyDJ commented on May 27, 2024

I see it as my problem with English communication, so I'm sorry for unclear specification of the issue. I use English passively most of the time (watching some English content, read websites, papers, translating SW to Czech, etc.), but my active knowledge of English is very poor. So, sometimes (like this time, unfortunately) I just don't give clear enough information...

I was thinking about it a little more and I realized that you're just right. Especially for closing modals or special high keyboard-oriented websites in general it could make serious problems. So, it's better to let it without the way for closing it via keyboard.

Maybe there is a even better solution for closing it. But, as you noted, we need some time of using it to figure out, which could it be.

from screenity.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.