Skip to content

Allow analytics table headings to wrap - #1392

Merged
MikeMcQuaid merged 1 commit into
mainfrom
analytics-header-wrap
Oct 25, 2026
Merged

MikeMcQuaid merged 1 commit into
mainfrom
analytics-header-wrap

Conversation

@MikeMcQuaid

Copy link
Copy Markdown
Member

This will make some of these easier to read and avoid stretching a column with short data for a long header

@MikeMcQuaid
MikeMcQuaid enabled auto-merge October 25, 2026 16:45
@coderabbitai

coderabbitai Bot commented Oct 25, 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 d47e934 and ad33ac9.

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
    • Updated wrapping behavior in analytics tables: body row headers remain unwrapped, while header cells outside the table body can now wrap. Date and version values retain their existing no-wrap behavior.

The .analytics-table.no-stack no-wrap selector now targets tbody th instead of all th elements. The .date-data and .version-data selectors remain unchanged.

Changes: Analytics table styling

Layer / File(s) Summary
No-wrap selector
assets/css/style.scss
The white-space: nowrap rule now applies to tbody th. The .date-data and .version-data selectors remain included.

Priority: Low

Merge Risk: Minimal · up to ad33a

The focused styling change matches the stated objective without introducing an actionable merge risk.

Pre-merge checks: Passed 4
Check name Status Explanation
Title check Passed The title clearly and concisely describes the main change: allowing analytics table headings to wrap.
Description check Passed The description explains that wrapping headings will improve readability and prevent columns from stretching due to long headers.
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 0049213 into main Oct 25, 2026
12 checks passed
@MikeMcQuaid
MikeMcQuaid deleted the analytics-header-wrap branch October 25, 2026 17:15
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