sentry-wizard icon indicating copy to clipboard operation
sentry-wizard copied to clipboard

Wizard project selection "glitches" when moving the arrow up

Open lucas-zimerman opened this issue 2 years ago • 8 comments

Environment

-Latest version of wizard.

  • Windows 11
  • VS Code - Powershell terminal.

Steps to Reproduce

on the option of selecting the project, scroll down. then scroll up and it'll start to mark multiple projects as selected.

I also noticed that at first it didn't happen when I was using on my main display (2k resolution), but it started to break once I tried it on a 1080p display, and continued broken even after moving to the 2k monitor

Not broken video: https://github.com/getsentry/sentry-wizard/assets/8229322/a5d17894-ca6d-4c43-a5b5-b37fac4f4598 Broken video: https://github.com/getsentry/sentry-wizard/assets/8229322/2ed12990-616d-4803-8e9f-3faba42c8cd0

Expected Result

to only a single project to be marked as the selected one.

Actual Result

it seems to only happen when you move the arrow up. image

BONUS POINTS: Allow to search the project by typing the name of it, it's quite hard to find the correct project since the project names aren't sortedby their names but by the org's name

lucas-zimerman avatar Oct 26 '23 18:10 lucas-zimerman

@lucas-zimerman I think this happens when the height of the terminal window is smaller than the "page height" of the project list. Does this bug disappear if you increase the terminal window height?

Currently, we configured clack to show 12 projects per page. Maybe we should reduce this to something between 5-8?

Lms24 avatar Oct 31 '23 09:10 Lms24

It seems to glitch when the terminal window size is around 1219x399 pixels image Resizing to fullscreen seems to fix it (note the top is just garbage of unchanged text on the buffer so its ok) image

My suspicious is that the Wizard requires 16 vertical character lines available, anything lower than that seems to glitch the scroll and the list image image

By default on my VSCode I have 13 visible vertical character lines so it'll glitch on the small monitor, where on the larger monitor i have 16 vertical lines.

lucas-zimerman avatar Nov 01 '23 03:11 lucas-zimerman

I wonder if we can detect the number of lines and adjust the sliding window/page size for the list accordingly

Lms24 avatar Nov 03 '23 08:11 Lms24

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you label it Status: Backlog or Status: In Progress, I will leave it alone ... forever!


"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

github-actions[bot] avatar Nov 25 '23 00:11 github-actions[bot]

image It seems to be also happening on MacOS

lucas-zimerman avatar Dec 06 '23 13:12 lucas-zimerman

bad stalebot 😠

hmm yeah I still don't know how to fix this other than detecting the available lines or making the window smaller

Lms24 avatar Dec 06 '23 13:12 Lms24

backlogging for now but don't have time to look into it in greater detail

Lms24 avatar Dec 06 '23 13:12 Lms24

this is an issue with the library itself, this could be fixed once https://github.com/natemoo-re/clack/pull/174 is merged.

lucas-zimerman avatar Dec 21 '23 18:12 lucas-zimerman