Skip to content

Conversation

@enjeck
Copy link
Contributor

@enjeck enjeck commented Dec 3, 2025

Simple fix to avoid excessive jumping when inline editing.

Before

Bildschirmaufzeichnung.vom.2025-11-13.15-57-10.webm

After

Screen.Recording.2025-12-03.at.05.38.31.mov

@enjeck enjeck requested a review from blizzz as a code owner December 3, 2025 05:49
@enjeck enjeck self-assigned this Dec 3, 2025
@enjeck enjeck force-pushed the fix-selection-width branch from da4ffa1 to 8d92219 Compare December 3, 2025 05:51
@enjeck enjeck marked this pull request as draft December 3, 2025 06:21
@juliusknorr
Copy link
Member

A few remarks:

  • We have two "jumps" in the layout there, where i think one could be solved a bit more elegant
  • The first happens when the plain rendering switches to the multiselect
    • Could we also give the non-edit rendering some padding so that ideally the text is staying in position
    • Enough padding to reserve space for the multiselect could already help for the first problem jumping
  • The second happens when the multiselect is selected, i have not found the css rule that influences that on a quick check, but as soon as the focus is there the select has a .vs--open class which then leads to the label not taking up available space anymore

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.

3 participants