Skip to content

Remove unused docs CSS after retiring shared chrome - #1376

Merged
gantoreno merged 4 commits into
masterfrom
gro-905-remove-unused-docs-css-after-retiring-shared-chrome
Oct 9, 2026
Merged

gantoreno merged 4 commits into
masterfrom
gro-905-remove-unused-docs-css-after-retiring-shared-chrome

Conversation

@gantoreno

Copy link
Copy Markdown
Contributor

Fixes GRO-905

  • Removes unused CSS rules after retiring the shared chrome

@linear

linear Bot commented Oct 7, 2026

Copy link
Copy Markdown

GRO-905

@github-actions

github-actions Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

The deployments were removed when this pull request closed. Reopening it deploys again at the same address.

Comment thread _docs Outdated
@@ -0,0 +1 @@
/Users/gabrielmoreno/Developer/metabase/docs.metabase.github.io/_docs No newline at end of file

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Weird... should it be /_docs instead?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe this might be an artifact and shouldn't have been tracked in the first place, removed it! Also simplified the PR a lot

@gantoreno
gantoreno requested a review from losrebellos October 8, 2026 15:49

@losrebellos losrebellos left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Non-blocking comments 🚀

Comment thread public/docs/css/main.css Outdated
Comment on lines +239 to +242
display: box;
display: flex;
display: flexbox;
display: flex;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we just have display: flex;?

Comment thread public/docs/css/main.css Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could it be merged with his best friend at line 140?

@gantoreno
gantoreno merged commit 7db5c7c into master Oct 9, 2026
6 checks passed
@gantoreno
gantoreno deleted the gro-905-remove-unused-docs-css-after-retiring-shared-chrome branch October 9, 2026 14:46
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