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

Don't autoselect any button in "keep files" dialog #5931

Closed
PVince81 opened this issue Jul 31, 2017 · 3 comments
Closed

Don't autoselect any button in "keep files" dialog #5931

PVince81 opened this issue Jul 31, 2017 · 3 comments

Comments

@PVince81
Copy link
Contributor

Steps:

  1. Setup sync client
  2. Let it upload some files
  3. Close sync client
  4. Reset ownCloud instances with new account with same credentials
  5. Open sync client
  6. Now quickly type some text somewhere and make sure to use a lot of spaces

Expected result

Whenever the "keep files" popup appears, it must stay and I need to explicitly choose to "keep files" or "delete". Don't auto-focus on any of the buttons.

Actual result

While typing long sentences, the dialog appears just at the moment I hit the spacebar. And at this time, one of the buttons (not sure which) was focused so hitting the spacebar selects it.

Versions

OC master
owncloud-client-2.3.2-1.5.x86_64

Minor issue since this doesn't happen often.

@PVince81
Copy link
Contributor Author

PVince81 commented Aug 2, 2017

I had a quick look at tried to set setDefaultButton(0) but it doesn't disable the default button. I suppose there might be a more hacky way to do it but it somehow feels wrong.

This reminds me a bit of some dialogs (was it Firefox?) where the OK button is disabled until the window is actually focussed. This lead me to think that it is probably wrong to have this one window automatically focused. It seems the flag WindowStaysOnTopHint is set which, from my understanding, would make it stay on top of any other windows on the desktop.

A possible alternative for this would be to just remove this flag and let the window manager prompt the user as it would do for any other popup dialog. And/or use the system's notification system to display a notification with buttons (this is possible on Plasma 5 at least), which would integrate better.

Thoughts ?

@guruz
Copy link
Contributor

guruz commented Aug 3, 2017

Ouch. i think this was reported before and is indeed annoying.

@TheOneRing
Copy link
Contributor

Duplicate of #5931

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants