- Issue created by @B_man
- πͺπΈSpain fjgarlin
We can add to show scrollbars:
.node-project-issue table { display: block; overflow: scroll; }
I can't think of a better solution given that it's "table" elements.
Moving this issue to "bluecheese", which is where the CSS happens. - Status changed to Needs review
12 months ago 3:34pm 16 January 2024 - πͺπΈSpain fjgarlin
Adding the above suggestion.
- MR: https://git.drupalcode.org/project/bluecheese/-/merge_requests/44/diffs
- Can be tested here: https://fjgarlin-drupal.dev.devdrupal.org/project/config_notify/issues/3... - πΊπΈUnited States smustgrave
I think that's the best solution probably available. But in this example I didn't know the table was scrollable.
- πͺπΈSpain fjgarlin
If we force scrollbars (
overflow: scroll
), all tables will have them, whether they need them or not.overflow: auto;
seems to be the best fit. In this case, you don't see the scrollbar because the table is really long too. - πΊπΈUnited States drumm NY, US
For this particular table, we have a
@include force-wrap;
that applies:word-break: break-word; hyphens: auto;
Plus some related styles for older browser compatibility. This avoids the scrolling, and I think should be okay for every table in an issue summary. It's used in various other places on the issue page, like for long filenames without word-breaking characters.
- πͺπΈSpain fjgarlin
That works much nicer. Applied the suggestion.
- Test URL: https://fjgarlin-drupal.dev.devdrupal.org/project/config_notify/issues/3...
- MR: https://git.drupalcode.org/project/bluecheese/-/merge_requests/44/diffs -
drumm β
committed fd3bd999 on 7.x-2.x authored by
fjgarlin β
Issue #3415027 by fjgarlin, B_man, smustgrave: Tables in issue...
-
drumm β
committed fd3bd999 on 7.x-2.x authored by
fjgarlin β
- Issue was unassigned.
- Status changed to Fixed
12 months ago 6:00pm 19 January 2024 Automatically closed - issue fixed for 2 weeks with no activity.