[UI Bug] Plannotator vertical spacing changes when expanding a file diff
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 68/100
- Issue type
- Bug
- Clarity
- Mostly clear
- Activity status
- Active
- Tech stack
- typescript
- Domain
- frontend
Research direction
Start by locating the code review file list and the file diff expansion behavior in the TypeScript UI. Reproduce the issue by expanding a file and observing whether the list shifts under the cursor and changes the focused file. Done when expanding a diff preserves consistent spacing and keeps the cursor over the selected file.
Written by the indexing model from the issue text.
Description
What happened?
Hi hi!
During a review in Plannotator, clicking on a file to expand its diff changes the vertical spacing between items in the file list. This causes the previously listed file to move under the cursor and changes the focused file. See the cursor pointer in the screenshots, it's at the same position in both captures
Expected Behavior:
Expanding a file should preserve consistent vertical spacing between list items, keeping the cursor over the selected file.
Thanks, Leo
Plannotator version
plannotator 0.28.49
OS
Linux
Agent
OpenCode
Where did it happen?
Code review
Browser
Brave 1.98.63 - Chromium: 156.0.8078.80
- Dominant language
- TypeScript
- Stars
- 8.7k
- Forks
- 649
- Avg merge
- 8h 40m
- Merged PRs (30d)
- 786
Getting set up
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 backnotprop/plannotator
-
Difficulty 2/5 1-3 hours Newbie friendliness 66/100
backnotprop/plannotator#2643 ·
Maintainers usually reply within 1 day
-
annotate shows the launch directory's repo, not the file'sPossibly taken A pull request linked to this issue is open or already merged. Open
Difficulty 2/5 1-3 hours Newbie friendliness 85/100
backnotprop/plannotator#2290 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
backnotprop/plannotator#2243 · 2 comments · 2 reactions ·
Maintainers usually reply within 1 day
-
Annotate documentation omits per-file version history and privacy controlsMay be free again A pull request for this issue was closed without being merged. Open
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
backnotprop/plannotator#1086 · 2 comments ·
Maintainers usually reply within 1 day
-
Markdown: code in callouts, link labels and double-backtick spans renders with its backticks showingPossibly taken @uvw claimed this 1 day ago. Open
Difficulty 3/5 1-2 days Newbie friendliness 35/100
backnotprop/plannotator#2718 ·
Maintainers usually reply within 1 day
All issues in backnotprop/plannotator
Similar issues
-
ArcGIS Maps SDK for JavaScript c-carousel calcite-design-tokens enhancement impact - p3 - not time sensitive needs triage
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Esri/calcite-design-system#15565 ·
Maintainers usually reply within 1 day
-
customer-reported question
Difficulty 2/5 1-3 hours Newbie friendliness 66/100
Azure/azure-rest-api-specs#48017 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
microsoft/TypeScript#65370 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 83/100
ts-graphviz/ts-graphviz#1904 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
Maintainers usually reply within 1 day