Refactor table styles to _tables.scss and fix AI page tables
#1260
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This is a great refactoring that centralizes duplicated table styles into a new _tables.scss partial, improving maintainability by reducing code duplication. This change also fixes an issue with an unstyled table on the AI page by using a more general main table selector. I have one suggestion regarding the font-family for table cells in the new partial, which seems to have been missed during the refactoring and could cause an unintended visual change. Overall, excellent work on cleaning up the SCSS! Why did the coffee file a police report? It got mugged!
|
Visit the preview URL for this PR (updated for commit 72b7cf9): https://doradotdev--pr1260-drafts-off-4o3enmih.web.app (expires Sun, 18 Jan 2026 22:31:42 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 7ad2b3cf9cceb558b493931176f998ae46924361 |
Extracts common table styles into a new partial `_tables.scss` and imports it globally. This replaces the duplicated `table` style blocks previously found in `ai.scss`, `research.scss`, and `capabilities.scss`. This change ensures consistent styling for all tables within the main content area (`main table`) and specifically fixes the unstyled "Four Mindsets" table on the AI Research page, which was previously missing styles due to a CSS selector mismatch. Fixes #1236
nathenharvey
force-pushed
the
push-ztluxuzpstzn
branch
from
ca8a4d3 to
a21ecfc
Compare
Extracts common table styles into a new partial
_tables.scssand imports it globally.This replaces the duplicated
tablestyle blocks previously found inai.scss,research.scss, andcapabilities.scss.This change ensures consistent styling for all tables within the main content area (
main table) and specifically fixes the unstyled "Four Mindsets" table on the AI Research page, which was previously missing styles due to a CSS selector mismatch.The change also removes another instance of
font-family: $font-roboto;from the site.Preview links:
Fixes #1236