fix: Specify min width for selection cell by enjeck · Pull Request #2181 · nextcloud/tables

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@enjeck

Copy link Copy Markdown

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
Signed-off-by: Enjeck C. <patrathewhiz@gmail.com>

@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 Copy Markdown

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

Reviewers

@blizzz blizzz Awaiting requested review from blizzz blizzz is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@enjeck enjeck

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@enjeck @juliusknorr