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

make sidebar content slightly wider and add background colour #507

Merged
merged 4 commits into from
Feb 22, 2024

Conversation

mansona
Copy link
Member

@mansona mansona commented Feb 15, 2024

This moves some of the tweaks that we were doing in https://github.com/ember-learn/guidemaker-ember-template "upstream" into styleguide 👍

Edit: This is a breaking change because any consumers of <EsSidebar /> that didn't want to have a grey background will need to add .bg-light or reset the style themselves 👍

Copy link

netlify bot commented Feb 15, 2024

Deploy Preview for ember-styleguide ready!

Name Link
🔨 Latest commit abeaf35
🔍 Latest deploy log https://app.netlify.com/sites/ember-styleguide/deploys/65d77981888a920008b8301f
😎 Deploy Preview https://deploy-preview-507--ember-styleguide.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@mansona mansona requested a review from a team February 15, 2024 17:04
Copy link
Contributor

@MinThaMie MinThaMie left a comment

Choose a reason for hiding this comment

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

Love it :) <3

@MinThaMie MinThaMie mentioned this pull request Feb 22, 2024
3 tasks
@mansona mansona merged commit 59d9572 into master Feb 22, 2024
15 checks passed
@mansona mansona deleted the fix-sidebar branch February 22, 2024 16:49
@github-actions github-actions bot mentioned this pull request Feb 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants