Skip to content

fix(NcFormBox): use text-wrap: pretty in items#7855

Draft
ShGKme wants to merge 1 commit intomainfrom
fix/NcFormBoxItems--text-wrap-pretty
Draft

fix(NcFormBox): use text-wrap: pretty in items#7855
ShGKme wants to merge 1 commit intomainfrom
fix/NcFormBoxItems--text-wrap-pretty

Conversation

@ShGKme
Copy link
Contributor

@ShGKme ShGKme commented Nov 13, 2025

☑️ Resolves

🏁 Checklist

  • ⛑️ Tests are included or are not applicable
  • 📘 Component documentation has been extended, updated or is not applicable
  • 2️⃣ Backport to stable8 for maintained Vue 2 version or not applicable

Signed-off-by: Grigorii K. Shartsev <me@shgk.me>
@ShGKme ShGKme added this to the 9.3.0 milestone Nov 13, 2025
@ShGKme ShGKme self-assigned this Nov 13, 2025
@ShGKme ShGKme added bug Something isn't working 3. to review Waiting for reviews labels Nov 13, 2025
@ShGKme
Copy link
Contributor Author

ShGKme commented Nov 13, 2025

/backport to stable8

@codecov
Copy link

codecov bot commented Nov 13, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 51.75%. Comparing base (eac1b94) to head (8ba3974).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #7855   +/-   ##
=======================================
  Coverage   51.75%   51.75%           
=======================================
  Files          98       98           
  Lines        3169     3169           
  Branches      871      871           
=======================================
  Hits         1640     1640           
  Misses       1279     1279           
  Partials      250      250           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Member

@kra-mo kra-mo left a comment

Choose a reason for hiding this comment

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

Copy-pasting from the issue:

@kra-mo Only the interactive items, or anything else as well?

Such us the groups headings/description, shortcuts labels etc.

No strong opinions. I found the wrapping odd for interactive elements only because of the trailing icon/switch.

At least for shortcuts labels, that would also apply because of the keys on the right side. For other things, it's your call.

@susnux susnux modified the milestones: 9.3.0, 9.3.1 Nov 20, 2025
@Antreesy Antreesy modified the milestones: 9.3.1, 9.4.0 Dec 11, 2025
@susnux susnux modified the milestones: 9.4.0, 9.4.1 Jan 22, 2026
@susnux susnux removed this from the 9.4.1 milestone Feb 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews backport-request bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[NcFormBox] Use text-wrap: pretty for FormBox* components where supported

4 participants