-
Notifications
You must be signed in to change notification settings - Fork 742
Add blog link to site switcher #6961
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
Conversation
|
Visit the preview URL for this PR (updated for commit fdd2da3): https://dart-dev--pr6961-misc-move-blog-to-site-switcher-jo7bws34.web.app |
|
Hi, I feel very strongly that we need the blog at the top and not hidden, so this is a change I'd like for us not to make. If there is a project plan somewhere with a larger vision, let's review it. |
@antfitch I went ahead and added it back to the header. Do you think it's fine to live there and in the site switcher? I was hoping to have it there for consistency. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great!
This makes it more consistent with other Dart-related site properties. It is still linked to from the top navigation as well.
Screenshot with the added button:
