Skip to content

Fixing hide-select-buttons#3962

Closed
dessalines wants to merge 2 commits into
hide-select-buttonsfrom
hide-select-buttons-dess
Closed

Fixing hide-select-buttons#3962
dessalines wants to merge 2 commits into
hide-select-buttonsfrom
hide-select-buttons-dess

Conversation

@dessalines
Copy link
Copy Markdown
Member

@dessalines dessalines commented Mar 12, 2026

  • Adds a chevron up, styles those buttons correctly
  • hides it if not logged in (because otherwise those buttons do nothing)
  • Fixes the diff

@Nutomic
Copy link
Copy Markdown
Member

Nutomic commented Mar 12, 2026

Right now this doesnt work well because the buttons move around when you click expand, quite disorienting. With my PR you can expand and collapse the extra buttons repeatedly without moving the mouse at all.

@dessalines
Copy link
Copy Markdown
Member Author

dessalines commented Mar 12, 2026

You could put them after the expand button, but the main thing is to not create a brand new flowrow like with the other PR. I prefer this because it keeps the checkboxes near each other tho, and the dropdowns all after.

@Nutomic
Copy link
Copy Markdown
Member

Nutomic commented Apr 13, 2026

Merged in #3986

@Nutomic Nutomic closed this Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants