Skip to content

Conversation

@Artoria2e5
Copy link

@Artoria2e5 Artoria2e5 commented Nov 2, 2025

Fixes #3. Admittedly brute-force, but it works.

image

What do a bunch of A100s say about this? (Copilot summary follows, lol it's longer than my change)

This pull request updates the feature table layout and tooltip styling to improve readability and maintainability. The main changes include simplifying the table structure in feature.liquid, refactoring how tooltips are handled in CSS, and adjusting status display formatting.

Fair.

Feature table structure and layout improvements:

  • Simplified the table in feature.liquid by moving year headers to individual <td> elements, removing complex absolute positioning for year and segment labels, and using colspan to align segments with the correct columns. The status segments are now rendered as inline blocks, making the table easier to maintain and more robust.

Yep, I'm proud of the inline blocks and the colspan.

Tooltip and status styling refactor:

  • Updated the tooltip CSS in main.css to use a dedicated .tooltip class inside .has-tooltip, improving selector specificity and display logic. Tooltips now appear on hover using a direct child selector, and the tooltip content is wrapped in a <div> for better control.

Eh not important. Should be minor cleanup.

  • Added a new .status class in main.css to standardize status label appearance, using inline-block display and padding for better alignment.

Minor CSS cleanup:

  • Removed the fixed row height for table rows (tr { height: 3rem; }) in main.css to allow for more flexible row sizing.

Padding actually replaces the 3rem.

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.

[cosmetic, but also bordering on unreadable] Bad vertical alignment

1 participant