5
11
Fork
You've already forked tomato
1

Respect the screen size #14

Merged
edwardloveall merged 1 commit from el-too-wide-too-furious into main 2025年08月13日 22:18:51 +02:00

Table cells containing items with long names would not wrap around to the next line which would make the table very wide. This was not great for smaller, or even medium screens.

Turns out material's default is to use white-space: nowrap on tables. I've disabled that except for cells that have buttons in them .mdc-data-table__cell--action. Otherwise, the action button text would wrap which looked bad.

Before

before

After

after

Table cells containing items with long names would not wrap around to the next line which would make the table very wide. This was not great for smaller, or even medium screens. Turns out material's default is to use `white-space: nowrap` on tables. I've disabled that except for cells that have buttons in them `.mdc-data-table__cell--action`. Otherwise, the action button text would wrap which looked bad. ## Before ![before](/attachments/97c91cca-3baf-47d9-b61f-b07ed83853f9) ## After ![after](/attachments/050d8d64-bb9a-42a5-8ce6-034b2a15a688)
edwardloveall force-pushed el-too-wide-too-furious from 19aab38f6d
Some checks failed
Setup Successful
Jest Successful
Static Analysis Successful
RSpec Failed
to 0cc33c72e1
Some checks failed
Setup Successful
Jest Successful
RSpec Failed
Static Analysis Successful
2025年08月13日 16:53:19 +02:00
Compare
First-time contributor
Copy link

Simple fix, huge visual upgrade. Nice one!

Simple fix, huge visual upgrade. Nice one!
edwardloveall force-pushed el-too-wide-too-furious from 0cc33c72e1
Some checks failed
Setup Successful
Jest Successful
RSpec Failed
Static Analysis Successful
to 48b2428877
All checks were successful
Setup Successful
Jest Successful
Static Analysis Successful
RSpec Successful
2025年08月13日 22:07:24 +02:00
Compare
edwardloveall deleted branch el-too-wide-too-furious 2025年08月13日 22:19:06 +02:00
Sign in to join this conversation.
No reviewers
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
rootable/tomato!14
Reference in a new issue
rootable/tomato
No description provided.
Delete branch "el-too-wide-too-furious"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?