Skip to content

Support wide analytics tables - #1391

Merged
MikeMcQuaid merged 1 commit into
mainfrom
analytics-tables
Oct 24, 2026
Merged

MikeMcQuaid merged 1 commit into
mainfrom
analytics-tables

Conversation

@MikeMcQuaid

Copy link
Copy Markdown
Member
  • Keep prose narrow while tables use the available viewport width.
  • Share numeric alignment and wrapping with formulae.brew.sh.

- Keep prose narrow while tables use the available viewport width.
- Share numeric alignment and wrapping with formulae.brew.sh.
@coderabbitai

coderabbitai Bot commented Oct 24, 2026

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
Commits

Reviewing files that changed from the base of the PR and between ebe15bd and 0572a26.

Files selected for processing: 1
  • assets/css/style.scss

Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review.

📝 Summary

Summary by CodeRabbit

  • Style
    • Improved the display of wide tables, including horizontal scrolling and clearer alignment for analytics data.
    • Made code in analytics tables wrap more cleanly.
    • Added visual cues and spacing for expandable sections.

The stylesheet adds presentation rules for wide tables, analytics tables, code within analytics tables, and expandable details elements.

Changes: Page styling

Layer / File(s) Summary
Table presentation
assets/css/style.scss
Wide tables are centred, sized to the viewport, and horizontally scrollable. Analytics tables gain alignment and spacing rules. Code in page analytics tables can wrap at any character.
Disclosure presentation
assets/css/style.scss
details elements gain vertical margins. Their summary elements use a pointer cursor.

Priority: Low

Merge Risk: Minimal · up to 0572a

The CSS changes present no established merge-blocking risk.

Pre-merge checks: Passed 4
Check name Status Explanation
Title check Passed The title clearly identifies the main change: support for wide analytics tables.
Description check Passed The description directly relates to the changes. It states the goals for table width, numeric alignment, and wrapping.
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@MikeMcQuaid
MikeMcQuaid merged commit d47e934 into main Oct 24, 2026
12 checks passed
@MikeMcQuaid
MikeMcQuaid deleted the analytics-tables branch October 24, 2026 18:52
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.

2 participants