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

Minor branding hints #1567

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

alex-r-bigelow
Copy link

A few minor improvements from a new-ish / casual Quarto user, who got lost in the docs this past week because the HTML Theming page is the first hit for many Google searches

Fixes:

  • The HTML Theming page has no direct links to _brand.yml; this should probably be the first thing a new / casual user tries, before (html-only) "Basic Options?"
  • "Basic Options" is written mostly from the perspective of per-document changes, which is a little odd / confusing on a "Theming" page, especially as the rest of the SASS theme variables on the page seem to be describing more site-wide theming
  • For someone out of the loop, it's easy to miss that _brand.yml is a separate project from Quarto, and all the "Limitations" and references to "the spec" are really confusing. Maybe this is a stretch (feel free to reject / critique this change), but it might have helped me to put the Brand.YML logo at the top of its docs page?

@cscheid
Copy link
Collaborator

cscheid commented Feb 27, 2025

For someone out of the loop, it's easy to miss that _brand.yml is a separate project from Quarto, and all the "Limitations" and references to "the spec" are really confusing.

Maybe this is a stretch (feel free to reject / critique this change), but it might have helped me to put the Brand.YML logo at the top of its docs page?

I think it's the correct diagnosis, but maybe not the best fix. We should instead spell out more explicitly what you said in the first paragraph of the overview.

@cscheid cscheid requested a review from cwickham February 27, 2025 13:58
@cscheid
Copy link
Collaborator

cscheid commented Feb 27, 2025

(I'll go over these with @cwickham and get back to you. It might be a week or so, but we appreciate the PR and feedback!)

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