-
-
Notifications
You must be signed in to change notification settings - Fork 86
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
Fix sidebar-weirdness #486
Comments
Hey I was willing to work on this issue, can you reiterate where the guidemaker-template is? |
So the guidemaker-template lives here: https://github.com/ember-learn/guidemaker-ember-template |
I'll pick this up! |
I know it's 3 months later but I'm picking this back up 💀
This point should likelly become "word-wrapping for API menu items that don't include dashes" as the word wrapping happens on dashes but not slashes. |
@MinThaMie @mansona do either of you have an idea on the best way to about the word wrapping? Maybe a naïve approach, but could we just increase the sidebar width and decrease its padding to create more space? e.g. the following screenshot uses a padding of If you are okay with this approach, I propose to change this directly in the ember-styleguide. Let me know what you think |
Should be fixed by #507 |
We will need to update to ember-styleguide@10 to get the new stuff 👍 |
I upgraded to ember-styleguide@9 and cleaned up the redundant styles in ember-learn/guidemaker-ember-template#172. Ignace has already approved this but it could use another pair of eyes. I upgraded to ember-styleguide@10 and removed the negative margin hack in ember-learn/guidemaker-ember-template#171 if either of you could have a look 🙏 (which I based on PR 172, but I can't figure out how to base my PR onto that one so it shows changes for both PRs) |
do we still need this issue now that #171 is merged? |
turns out not! we can close this 🎉 |
All of the fixes for these cases should go in the sidebar or table of contents CSS here in the styleguide (if we can)
The text was updated successfully, but these errors were encountered: