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

Disable up/down keyboard navigation when adding subscribers via stream settings. #29690

Closed
laurynmm opened this issue Apr 11, 2024 · 7 comments · Fixed by #29839
Closed

Disable up/down keyboard navigation when adding subscribers via stream settings. #29690

laurynmm opened this issue Apr 11, 2024 · 7 comments · Fixed by #29839

Comments

@laurynmm
Copy link
Collaborator

Currently, when adding subscribers to a stream (via the stream settings overlay), using the up/down arrow key moves the focus from the input to the streams list (on the left). Steps to reproduce this are:

  • Go to stream settings
  • Go to the subscribers tab
  • Click/select the add subscribers box
  • Either leave the box blank or input a user in the box
  • Use the up or down arrow

Instead, when the Add subscribers input is selected in the streams setting overlay, using the up/down arrow keys shouldn't do anything. See relevant CZO thread.

Screenshot of subscribers tab of stream settings overlay

Screenshot from 2024-04-11 13-11-17

@zulipbot
Copy link
Member

Hello @zulip/server-streams members, this issue was labeled with the "area: stream settings" label, so you may want to check it out!

@alya alya added the bug label Apr 11, 2024
@Kirsty21
Copy link
Collaborator

@zulipbot claim

@zulipbot
Copy link
Member

@Kirsty21 We noticed that you have not made any updates to this issue or linked PRs for 10 days. Please comment here if you are still actively working on it. Otherwise, we'd appreciate a quick @zulipbot abandon comment so that someone else can claim this issue and continue from where you left off.

If we don't hear back, you will be automatically unassigned in 4 days. Thanks!

@Kirsty21
Copy link
Collaborator

@zulipbot abandon

@PieterCK
Copy link
Collaborator

@zulipbot claim

PieterCK added a commit to PieterCK/zulip that referenced this issue Apr 25, 2024
Add a rule to the switch_rows(event) function to avoid
switching stream row when pressing up/down key by checking
the current URL hash and the add_subscribers_pill focus state.
Add a utility function to hash_parser to help implement new
switch_rows behavior mentioned above.

Fixes zulip#29690
PieterCK added a commit to PieterCK/zulip that referenced this issue Apr 25, 2024
Add a rule to the switch_rows(event) function to avoid
switching stream row when pressing up/down key by checking
the current URL hash and the add_subscribers_pill focus state.
Add a utility function to hash_parser to help implement new
switch_rows behavior mentioned above.

Fixes zulip#29690
@PieterCK
Copy link
Collaborator

PieterCK commented Apr 25, 2024

Heya, I've submitted a small PR to fix this issue. It's still missing some tests and might be a bit rough since this is the first time getting my hand dirty on some frontend stuff, but the desired behavior has been achieved. Let me know what you think whenever you have the time to check it. 👍 @laurynmm

#29839

@PieterCK
Copy link
Collaborator

Deleted my earlier comment since it is irrelevant now

PieterCK added a commit to PieterCK/zulip that referenced this issue Apr 26, 2024
Add a rule to the switch_rows(event) function to avoid
switching stream row when pressing up/down key by checking
the current URL hash and the add_subscribers_pill focus state.
Add a utility function to hash_parser to help implement new
switch_rows behavior mentioned above.

Fixes zulip#29690
PieterCK added a commit to PieterCK/zulip that referenced this issue Apr 26, 2024
Add a rule to the switch_rows(event) function to avoid
switching stream row when pressing up/down key by checking
the current URL hash and the add_subscribers_pill focus state.
Add a utility function to hash_parser to help implement new
switch_rows behavior mentioned above.

Fixes zulip#29690
PieterCK added a commit to PieterCK/zulip that referenced this issue Apr 26, 2024
Add a rule to the switch_rows(event) function to avoid
switching stream row when pressing up/down key by checking
the current URL hash and the add_subscribers_pill focus state.
Add a utility function to hash_parser to help implement new
switch_rows behavior mentioned above.

Fixes zulip#29690
PieterCK added a commit to PieterCK/zulip that referenced this issue Apr 29, 2024
Add a rule to the switch_rows(event) function to avoid
switching stream row when pressing up/down key by checking
the current URL hash and the add_subscribers_pill focus state.
Add a utility function to hash_parser to help implement new
switch_rows behavior mentioned above.

Fixes zulip#29690
PieterCK added a commit to PieterCK/zulip that referenced this issue Apr 29, 2024
Add a rule to the switch_rows(event) function to avoid
switching stream row when pressing up/down key by checking
the current URL hash and the add_subscribers_pill focus state.
Add a utility function to hash_parser to help implement new
switch_rows behavior mentioned above.

Fixes zulip#29690.
timabbott pushed a commit to PieterCK/zulip that referenced this issue Apr 30, 2024
Add a rule to the switch_rows(event) function to avoid
switching stream row when pressing up/down key by checking
the current URL hash and the add_subscribers_pill focus state.
Add a utility function to hash_parser to help implement new
switch_rows behavior mentioned above.

Fixes zulip#29690.
timabbott pushed a commit that referenced this issue Apr 30, 2024
Add a rule to the switch_rows(event) function to avoid
switching stream row when pressing up/down key by checking
the current URL hash and the add_subscribers_pill focus state.
Add a utility function to hash_parser to help implement new
switch_rows behavior mentioned above.

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

Successfully merging a pull request may close this issue.

5 participants