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

UX: Fix tab usage #280

Closed
wants to merge 3 commits into from
Closed

UX: Fix tab usage #280

wants to merge 3 commits into from

Conversation

marcelltoth
Copy link
Contributor

Closes #277
Closes #278

Having selected a suggestion with the arrow keys
image
And pressing the TAB key leads to this state:
image
The same thing happens when clicking on a suggested item instead of pressing TAB.

Typing a custom (unknown) header and pressing TAB leads to the same result:
image
image

Copy link

@XVincentX XVincentX left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll review the code and try to debug — blocking this at the moment because it's completely broken in Safari.

Screen-Recording-2020-03-03-at-1

Bug 1

The Tab does not work in Safari, it stays stuck on the header

image

If I push the tab button, the selection stays on account-id and it won't move

Bug 2

If I click the "Key" to add a new header, it won't work. The selection stays on account-id and it won't move from there. The only way to make it work is to delete all the headers

Bug 3

The tab seems to be working if I cancel all the headers, but it only goes through if I select a know header; if I write a custom one and I Tab, the selection won't move from there

@XVincentX
Copy link

@marcelltoth Updated with broken cases. It seems to be working fine in Chrome, so I guess this might be a problem for you to debug, isn't it?

@XVincentX
Copy link

Apparently by removing the autoFocus property the behaviour are the same.

@marcelltoth
Copy link
Contributor Author

marcelltoth commented Mar 4, 2020

@marcelltoth Updated with broken cases. It seems to be working fine in Chrome, so I guess this might be a problem for you to debug, isn't it?

Well, it's going to be quite problematic for me to debug. There has been no Safari release for anything but macOS for a while, and I don't have such a thing.

Neither of these is reproducible in Chrome, right?

@ssspear maybe do you happen to have a MacBook?

@ssspear
Copy link

ssspear commented Mar 4, 2020

maybe do you happen to have a MacBook?

I do have a MacBook and can take a look. Crazy that there isn't a Safari download for Windows

@ssspear
Copy link

ssspear commented Mar 4, 2020

Yikes my Safari is straight up crashing every time I load this. Before it crashes, I do experience the same issue though. @XVincentX Should I work on debugging this since @marcelltoth won't be able to?

@XVincentX
Copy link

@ssspear @marcelltoth No, this is low priority. Skip this.

@marcelltoth
Copy link
Contributor Author

Closing because RM will be going through a remake.

@philsturgeon philsturgeon deleted the fix/request-maker-tab-index branch January 8, 2021 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

UX: Tab does not insert header name suggestion Request Maker Tab index is broken in headers
4 participants