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

New annotations don't appear to have keyboard focus (until you start typing) #2126

Closed
seanh opened this Issue Apr 2, 2015 · 11 comments

Comments

Projects
None yet
6 participants
@seanh
Contributor

seanh commented Apr 2, 2015

When the user creates a new annotation the sidebar automatically scrolls to show it, but the keyboard focus is not always automatically in the new annotation's text area. Sometimes it is, sometimes it isn't.

@JakeHartnell

This comment has been minimized.

Show comment
Hide comment
@JakeHartnell

JakeHartnell Apr 7, 2015

Contributor

I can't reproduce this. Any steps to preform or a site where this happens?

Contributor

JakeHartnell commented Apr 7, 2015

I can't reproduce this. Any steps to preform or a site where this happens?

@seanh

This comment has been minimized.

Show comment
Hide comment
@seanh

seanh Apr 8, 2015

Contributor

Hmm, I really have seen this lots of times, but now I can't figure out how to reproduce it on master. Closing for now

Contributor

seanh commented Apr 8, 2015

Hmm, I really have seen this lots of times, but now I can't figure out how to reproduce it on master. Closing for now

@seanh seanh closed this Apr 8, 2015

@edrex

This comment has been minimized.

Show comment
Hide comment
@edrex

edrex Sep 21, 2015

Input focus isn't set to the new annotation text field for me (ever).

Chrome 45.0.2454.93 OS X, using the chrome browser app.

This costs a scan/seek/click cycle, which the user really feels IMO, after clicking the extension button, waiting for the overlay to load, finding and clicking the "New Note" button.

Auto-scrolling to the new annotation works.

Should I report this as a new issue or would reopening this make sense?

edrex commented Sep 21, 2015

Input focus isn't set to the new annotation text field for me (ever).

Chrome 45.0.2454.93 OS X, using the chrome browser app.

This costs a scan/seek/click cycle, which the user really feels IMO, after clicking the extension button, waiting for the overlay to load, finding and clicking the "New Note" button.

Auto-scrolling to the new annotation works.

Should I report this as a new issue or would reopening this make sense?

@dwhly

This comment has been minimized.

Show comment
Hide comment
@dwhly

dwhly Sep 21, 2015

Member

While the bootstrappy blue border isn't active on the text edit box, if you start typing you should see that the actual focus is on the element. We should fix the first, but just checking that you're seeing that the focus is indeed set correctly.

Member

dwhly commented Sep 21, 2015

While the bootstrappy blue border isn't active on the text edit box, if you start typing you should see that the actual focus is on the element. We should fix the first, but just checking that you're seeing that the focus is indeed set correctly.

@edrex

This comment has been minimized.

Show comment
Hide comment
@edrex

edrex Sep 22, 2015

Ah, when I start typing the focus is switched to the new annotation input box, but I think the focus should be set when the note is created so focus cues work.

edrex commented Sep 22, 2015

Ah, when I start typing the focus is switched to the new annotation input box, but I think the focus should be set when the note is created so focus cues work.

@seanh

This comment has been minimized.

Show comment
Hide comment
@seanh

seanh Sep 22, 2015

Contributor

Yep, I can reproduce it as @dwhly and @edrex say on master: text area does not appear to be focused but if you start typing it is. Re-opening

Contributor

seanh commented Sep 22, 2015

Yep, I can reproduce it as @dwhly and @edrex say on master: text area does not appear to be focused but if you start typing it is. Re-opening

@seanh seanh reopened this Sep 22, 2015

@seanh seanh changed the title from New annotations don't always have keyboard focus to New annotations don't appear to have keyboard focus (until you start typing) Sep 22, 2015

@tilgovi

This comment has been minimized.

Show comment
Hide comment
@tilgovi

tilgovi Sep 22, 2015

Contributor

@seanh not sure that's fixable, since it's probably UA / OS dependent. I don't know of any way to change the focus to the iframe, just set the focused within the frame. If the OS doesn't show a background window/frame as having focus, it may not be possible.

Contributor

tilgovi commented Sep 22, 2015

@seanh not sure that's fixable, since it's probably UA / OS dependent. I don't know of any way to change the focus to the iframe, just set the focused within the frame. If the OS doesn't show a background window/frame as having focus, it may not be possible.

@edrex

This comment has been minimized.

Show comment
Hide comment
@edrex

edrex Sep 22, 2015

Ah, I suspected there might be some iframe-related reason it worked this way. that's a bummer. It would be best if the focus could be set immediately. What is it that allows changing the focused frame on keystroke but not on create? Otherwise, maybe some class manipulation + style rule to visually highlight it (although that wouldn't help screen readers etc)?

edrex commented Sep 22, 2015

Ah, I suspected there might be some iframe-related reason it worked this way. that's a bummer. It would be best if the focus could be set immediately. What is it that allows changing the focused frame on keystroke but not on create? Otherwise, maybe some class manipulation + style rule to visually highlight it (although that wouldn't help screen readers etc)?

@edrex

This comment has been minimized.

Show comment
Hide comment
@edrex

edrex Sep 22, 2015

Tried to create a test case, but I'm not sure if I'm replicating the conditions for the extension correctly. Autofocus in the newly created frame works there.

http://codepen.io/edrex/pen/dYpyov

edrex commented Sep 22, 2015

Tried to create a test case, but I'm not sure if I'm replicating the conditions for the extension correctly. Autofocus in the newly created frame works there.

http://codepen.io/edrex/pen/dYpyov

@tilgovi

This comment has been minimized.

Show comment
Hide comment
@tilgovi

tilgovi Sep 22, 2015

Contributor

If I understand correctly, the focus is there, and at least the browser knows, and maybe directs the key events properly, but the OS might not and therefore the text area widget might not show the blinking beam cursor or whatever until you type or otherwise force the focus over there. I'm just speculating. If it has to do with the cursor style in a "background" window, that's not something I think we have any styling control over.

Contributor

tilgovi commented Sep 22, 2015

If I understand correctly, the focus is there, and at least the browser knows, and maybe directs the key events properly, but the OS might not and therefore the text area widget might not show the blinking beam cursor or whatever until you type or otherwise force the focus over there. I'm just speculating. If it has to do with the cursor style in a "background" window, that's not something I think we have any styling control over.

@nickstenning nickstenning added P3: Next sprint and removed Bug labels Feb 10, 2016

@nickstenning

This comment has been minimized.

Show comment
Hide comment
@nickstenning

nickstenning Apr 20, 2016

Contributor

Fixed by #3041.

Contributor

nickstenning commented Apr 20, 2016

Fixed by #3041.

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