Skip to content

Conversation

ritorhymes
Copy link

Description

On screen sizes of approximately 700px width or less, tables on at least (4) pages overflow and break the right page margin causing the entire page to increase its width. Normal behavior with table content exceeding the page width should have tables contained with a scrollable overflow or wrap down behavior localized to the tables and not the entire page. This harms the UX for users, especially on mobile, as it makes it more difficult to navigate the page and mars the appearance of the header and footer.

Pages I found with the issue:
/docs/internal-statistics/
/docs/internal-events/
/docs/constants/
/docs/alerts/

Fix

  • Add CSS to _page.scss to enable scrollable overflow within the table itself instead of the entire page.
  • No side-effects noted

Before and After

Before
tables-before

After
tables-after

After - Scrolling the Table Content
tables-after-1

@psiinon
Copy link
Member

psiinon commented Sep 18, 2025

Logo
Checkmarx One – Scan Summary & Details3791ea5b-50be-4ed3-b1a0-dd7693ec80b9

Great job! No new security vulnerabilities introduced in this pull request


Use @Checkmarx to reach out to us for assistance.

Just send a PR comment with @Checkmarx followed by a natural language request.

Examples: @Checkmarx how are you able to help me? @Checkmarx rescan this PR

@kingthorin
Copy link
Member

Signed-off-by: Rito Rhymes <83614463+ritorhymes@users.noreply.github.com>
@ritorhymes ritorhymes force-pushed the fix/table-responsive-scroll branch from 274bac2 to 46cb499 Compare September 18, 2025 11:05
@ritorhymes
Copy link
Author

@kingthorin got it, thanks! I just took care of them.

@thc202 thc202 changed the title fix(table): prevent broken page layout from table overflow Prevent broken page layout from table overflow Sep 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants