`e` key archives the wrong issue/PR when opened using a mouse · octobox/octobox#3325

(0 评论) (1 反应) (0 负责人)Ruby (3,737 star) (311 fork)batch import

bughelp wanted

描述

Are you experiencing this issue in octobox.io or your own instance?

octobox.io
My instance

🐞 Problem I use both the keyboard shortcuts and a mouse. When I click on an issue/PR in a list of those (for example, Inbox), and then hit e to archive, it often archives a different issue/PR than was currently open.

🎯 Goal I would like to be able to use both the keyboard and a mouse, and have it behave in a sensible way.

💡 Possible solutions I suspect there is a location in the inbox for up/down navigation. Once an issue/PR is open, it shouldn't use this location but instead apply keyboard commands (like e) to the currently open issue/PR.

贡献者指南

技术栈: javascriptreact
领域: frontend
议题类型: bug
难度: 3
预计时间: half day
活动状态: fresh
清晰度: clear
前置要求: basic knowledge of Reactunderstanding of keyboard events
新手友好度: 70
研究方向: Investigate the keyboard shortcut handling code in the frontend, likely in a React component managing the inbox list. Look for the `e` key binding and how the currently selected item is tracked. The issue suggests that clicking an item does not update the internal 'cursor' position for keyboard navigation, so the archive action applies to the last keyboard focused item instead of the clicked one. Check the state management for selected item and ensure that mouse clicks update that state.