Skip to content

Conversation

@timlandolt
Copy link
Contributor

@timlandolt timlandolt commented Oct 24, 2025

TICKET-24653

Previously, the whole table row would just overflow.

Before:

Screen.Recording.2025-10-24.at.13.49.32.mov

After

Screen.Recording.2025-10-24.at.13.46.56.mov

@cb341
Copy link
Contributor

cb341 commented Oct 24, 2025

Please provide screen grab of the fixed version too.

Feel free to use a table:

Before After
vid1 vid2

Copy link
Contributor

@cb341 cb341 left a comment

Choose a reason for hiding this comment

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

As discussed, use hyphenation instead of overflow auto with scrollbar

@timlandolt timlandolt requested a review from cb341 October 24, 2025 11:55
Copy link
Contributor

@cb341 cb341 left a comment

Choose a reason for hiding this comment

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

Nice PR!
Thanks 🙌

table-layout: fixed;
}

.hyphens-auto {
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice use of utility class!

Comment on lines +8 to +12
.timer-session-table-comments {
overflow: auto;
overflow-wrap: normal;
}

Copy link
Contributor

Choose a reason for hiding this comment

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

Don't see it being used..

Suggested change
.timer-session-table-comments {
overflow: auto;
overflow-wrap: normal;
}

@timlandolt timlandolt merged commit 1a268b3 into main Oct 30, 2025
2 checks passed
@timlandolt timlandolt deleted the feature/24653-fix-wide-cell-styling branch October 30, 2025 06:57
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