Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ensure we load stoplight css first so that base styles get overridden #4348

Merged
merged 3 commits into from
Oct 26, 2023

Conversation

GideonShils
Copy link
Collaborator

@GideonShils GideonShils commented Oct 26, 2023

Closes #4346

What are you changing in this pull request and why?

Previously the header on the API docs pages was using the wrong font. This was because the Stoplight styles used for rendering API docs were loading last, and were therefore overriding the page level font. This PR swaps the css to load using the standard docusaurus pattern so that the styles load first, and are overridden later by the expected font styling.

Checklist

  • Review the Content style guide and About versioning so my content adheres to these guidelines.
  • Add a checklist item for anything that needs to happen before this PR is merged, such as "needs technical review" or "change base branch."

@GideonShils GideonShils requested a review from a team as a code owner October 26, 2023 17:23
@vercel
Copy link

vercel bot commented Oct 26, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs-getdbt-com ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 26, 2023 9:18pm

@github-actions github-actions bot added the size: small This change will take 1 to 2 days to address label Oct 26, 2023
Copy link
Collaborator

@runleonarun runleonarun left a comment

Choose a reason for hiding this comment

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

@GideonShils Looks good, but I don't see any changes in the api fonts. Is there follow-up work that would need to happen?

@GideonShils
Copy link
Collaborator Author

GideonShils commented Oct 26, 2023

@runleonarun ahh yeah, this PR just fixes the header fonts so that we're not overriding anything unexpectedly. The actual docs are still using the default Stoplight CSS. I'd say if we want to change things up there, let's do it as a follow up

https://dbtlabs.atlassian.net/browse/CC-498

Copy link
Collaborator

@runleonarun runleonarun left a comment

Choose a reason for hiding this comment

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

OK let's merge this then, @GideonShils!

@runleonarun runleonarun enabled auto-merge (squash) October 26, 2023 21:13
@runleonarun runleonarun merged commit 8597c72 into current Oct 26, 2023
6 checks passed
@runleonarun runleonarun deleted the gideonshils/load-stoplight-styles-first branch October 26, 2023 21:18
runleonarun added a commit that referenced this pull request Oct 26, 2023
runleonarun added a commit that referenced this pull request Oct 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size: small This change will take 1 to 2 days to address
Projects
None yet
Development

Successfully merging this pull request may close these issues.

APIs: Update font to be consistent with site
2 participants