Skip to content

fix(relation): Fix ellipsis for long selected relation labels - #2878

Open
Koc wants to merge 1 commit into
mainfrom
bugfix/fix-styling-relations-with-long-labels
Open

fix(relation): Fix ellipsis for long selected relation labels#2878
Koc wants to merge 1 commit into
mainfrom
bugfix/fix-styling-relations-with-long-labels

Conversation

@Koc

@Koc Koc commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

This PR fixes display of the relation fields with long relation label for add/edit modal and inline edit.

🖼️ Screenshots

🏚️ Before 🏡 After note
image image relation field has small width, there is an overlap between text and select icons
image image column has different width when we selected editing 2 or 3 rows

🏁 Checklist

  • ⛑️ Tests (unit and/or integration) are included or not needed
  • 🔙 Backport requests are created or not needed: /backport to stableX.X
  • 📅 Milestone is set
  • 🌸 PR title is meaningful (if it should be in the changelog: is it meaningful to users?)

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@Koc
Koc marked this pull request as ready for review August 8, 2026 22:23
@Koc
Koc requested review from blizzz and enjeck as code owners August 8, 2026 22:23
@samin-z
samin-z self-requested a review August 10, 2026 15:48
@samin-z

samin-z commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

ran 'npm run build' and hard refreshed, but i still see the same bug
in chrome:

Screenshot 2026-08-10 at 17 53 36

in firefox:
Screenshot 2026-08-10 at 17 54 46

@Koc

Koc commented Aug 10, 2026

Copy link
Copy Markdown
Contributor Author

@samin-z I guess because you're using single word. In my test scenario there were multiple words

@samin-z

samin-z commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

@Koc the issue is that the input field is flexible and exapnds with the content (has nothing to do with if its a single words or multiple words), so the input box expands to the width of the 'edit row' box and if it's bigger than that, the text still mixes with the 'dropdown' and 'cancel' icon.
"edit row" maximum box input length:
Screenshot 2026-08-11 at 09 53 47
smaller than "edit row" maximum bix input length:
Screenshot 2026-08-11 at 09 54 07

@enjeck enjeck left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no longer needed since nextcloud-libraries/nextcloud-vue#8829 is merged???

@Koc

Koc commented Aug 25, 2026

Copy link
Copy Markdown
Contributor Author

I will update library and double-check

@Koc
Koc force-pushed the bugfix/fix-styling-relations-with-long-labels branch from 7173fd2 to 328c1a0 Compare August 27, 2026 23:34
@Koc

Koc commented Aug 27, 2026

Copy link
Copy Markdown
Contributor Author

I've updated libraries but it seems like this is not enough. Here what I've achieved in current branch with override:

image image image

@samin-z please test on your side. If everything ok - I will move changes from the current PR back to libraries.

@Koc
Koc force-pushed the bugfix/fix-styling-relations-with-long-labels branch from 328c1a0 to 5606dd3 Compare August 28, 2026 08:14
@Koc
Koc force-pushed the bugfix/fix-styling-relations-with-long-labels branch from 5606dd3 to 9055e54 Compare September 6, 2026 16:56
@samin-z

samin-z commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

he issue is that the input field is flexible and exapnds with the content (has nothing to do with if its a single words or multiple words), so the input box expands to the width of the 'edit row' box and if it's bigger than that, the text still mixes with the 'dropdown' and 'cancel' icon.

@Koc the issue still stands.

@Koc
Koc force-pushed the bugfix/fix-styling-relations-with-long-labels branch from 9055e54 to a19e202 Compare September 13, 2026 14:48
Signed-off-by: Kostiantyn Miakshyn <molodchick@gmail.com>
@Koc
Koc force-pushed the bugfix/fix-styling-relations-with-long-labels branch from a19e202 to d6dae04 Compare September 13, 2026 15:38
@Koc

Koc commented Sep 13, 2026

Copy link
Copy Markdown
Contributor Author

@samin-z theck one more time, please. I've tested in Firefox and Chrome

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