fix: Specify min width for selection cell by enjeck · Pull Request #2181 · nextcloud/tables
Navigation Menu
{{ message }}
- Notifications You must be signed in to change notification settings
- Fork 38
Draft
fix: Specify min width for selection cell#2181
fix: Specify min width for selection cell#2181
Conversation
Copy link Copy Markdown
Contributor
enjeck
commented
Dec 3, 2025
enjeck
commented
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
requested a review
from blizzz
as a code owner
enjeck
self-assigned this
Signed-off-by: Enjeck C. <patrathewhiz@gmail.com>
enjeck
force-pushed
the
fix-selection-width
branch
from
da4ffa1 to
8d92219
Compare
enjeck
marked this pull request as draft
Copy link Copy Markdown
Member
juliusknorr
commented
Dec 3, 2025
juliusknorr commented
Dec 3, 2025A 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--openclass which then leads to the label not taking up available space anymore
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment