-
-
Notifications
You must be signed in to change notification settings - Fork 319
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
Show a visible link to the docs of each package #2309
Conversation
I'll open a PR this week-end (ideally sunday afternoon) -- probably loosely based on the stimulus bundle page "menu", with some simplification in markup + CSS, and some things to help us bring the next chapter :) But I see no problem to make this change in the meantime! Do you think we could keep the current (flat) icon, with the color theme of the package ? |
@javiereguiluz I'm on it and pretty confident i'll open the PR soon... (@kbond can confirm if needed) But depending on tonight, i may miss the deadline by a day or two... Poke me if you want to have an "avant-première" ;) |
I'm working on it 👀 |
Thank you Javier. |
@Kocal "Since nothing moved since October/November" ... this is not fair 😆 You were in the slack discussions about the redesign, and saw how the scope changed 😅 |
Of course, but I mean on the live website / pull-requests :) |
I know that you mentioned that there's a work-in-progress effort to update some things in the current design (#2273 (comment)) but I think we should fix this issue quickly because it's a problem in many long package pages.
If you don't like the proposed changes here it's fine; let's iterate over them. Tell me what to fix or change and I'll do it. Thanks!
This is how this PR looks at the moment:
#SymfonyHackday :)