Detail sidebar collapse drops keyboard focus
Maintainers usually reply within 1 day
@GYJ99 is already working on this.
Since Nov 9, 2026.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 78/100
- Issue type
- Bug
- Clarity
- Clearly specified
- Activity status
- Active
- Tech stack
- next.js, react, typescript
- Domain
- accessibility, frontend, testing
Research direction
Read the two top bar components and their Browser Mode specs: web/app/components/app-sidebar/app-detail-top.tsx, dataset-detail-top.tsx, and the corresponding tests under __tests__. Run the keyboard journeys to confirm focus is lost after collapse. Done when both tests pass and focus stays on or moves to the replacement expand button.
Written by the indexing model from the issue text.
Description
Affected code
web/app/components/app-sidebar/app-detail-top.tsx:30-44,103-111web/app/components/app-sidebar/dataset-detail-top.tsx:28-42,104-112web/app/components/app-sidebar/__tests__/app-detail-top.browser.spec.tsx:57-69web/app/components/app-sidebar/__tests__/dataset-detail-top.browser.spec.tsx:53-65
Reproduction
- Open an app or dataset detail page.
- Focus the expanded sidebar collapse button with the keyboard.
- Press Enter to collapse the sidebar.
- Try to press Enter again to expand it.
Expected
Focus remains on, or is transferred to, the replacement expand button so the keyboard journey can continue immediately.
Actual
The focused collapse button is conditionally removed, focus falls back to document.body, and Enter cannot reopen the sidebar until the user navigates back to the new button.
How the tests exposed it
The Browser Mode cleanup added matching it.fails keyboard journeys in the app and dataset top-bar specs. Both activate the real controlled owner and assert focus moves to the replacement expand control; both currently fail because focus is lost.
Cleanup PR: https://github.com/langgenius/dify/pull/45932
- Dominant language
- Python
- Stars
- 160k
- Forks
- 25.4k
- Avg merge
- 1d 1h
- Merged PRs (30d)
- 608
Getting set up
Starts the project's dev container in your browser, under your own GitHub account.
- No Dockerfile or Docker Compose file
- Has a pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from langgenius/dify
-
Custom API tool sends unset optional header and cookie parameters as empty values or "None"Possibly taken @TemitopeKadri claimed this today. Open
Difficulty 2/5 1-3 hours Newbie friendliness 77/100
langgenius/dify#46010 · 1 reaction ·
Maintainers usually reply within 1 day
-
[Bug] CotAgentOutputParser overwrites action_name with non-input keys and collapses multi-block code fencesPossibly taken A pull request linked to this issue is open or already merged. Open
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
langgenius/dify#45324 · 1 comment · 1 reaction ·
Maintainers usually reply within 1 day
-
[Chore/Refactor] bring jieba keyword table handler tests under strict pyreflyMay be free again @Luckyou-d claimed this 7 days ago, and no pull request is open. Open
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
langgenius/dify#45319 · 1 comment ·
Maintainers usually reply within 1 day
-
[Bug] Agent resource access page loses its sidebar on refreshPossibly taken @XiaoBa-Yu claimed this 8 days ago. Open1.18.0
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
langgenius/dify#45291 · 1 reaction ·
Maintainers usually reply within 1 day
-
Parent-child full-doc mode raises IndexError when extraction yields zero documentsPossibly taken @simpleqt claimed this 9 days ago. Open1.18.0
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
langgenius/dify#45272 · 1 comment · 1 reaction ·
Maintainers usually reply within 1 day
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
Maintainers usually reply within 1 day
-
With emit_ontology_keys, one ontology key change is reported as two errors (keys and keys_digest)Possibly taken @rvandewater claimed this today. Openbug priority:low
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
Difficulty 1/5 1-3 hours Newbie friendliness 84/100
iausathub/satchecker#286 ·
Maintainers usually reply within 1 day
-
bug test
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
OpenAssetIO/OpenAssetIO#1527 ·
-
automated endpoint-dead needs-curation
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
almanac-data/climate-almanac#128 ·
Maintainers usually reply within 1 day