-
Notifications
You must be signed in to change notification settings - Fork 649
frontend: Document category proposal system and categorization process #2185
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
The wording, of course, is negotiable. Personally I like the style of "[Condition one might be in]? [How to get oneself out of it]!". Signed-off-by: Kristofer Rye <[email protected]>
I feel bad adding another entire style file, but the overhead should still be pretty dang minimal since this just adjusts a couple of styling bits and bobs. Signed-off-by: Kristofer Rye <[email protected]>
r? @smarnach (rust_highfive has picked a reviewer for you, use r? to override) |
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.
Thanks for the PR! One suggested change below.
Per review feedback and discussion in the meeting, it seems like a good idea to hold off on adding this just yet. Signed-off-by: Kristofer Rye <[email protected]> Tested-by: Kristofer Rye <[email protected]>
Thanks for the update! Visual diff approved in Percy. @bors r+ |
📌 Commit 2662550 has been approved by |
frontend: Document category proposal system and categorization process This PR aims to close #544, and just adds a div and some styles for it. <details><summary>📸 Before</summary><img width="1036" alt="image" src="https://user-images.githubusercontent.com/1566689/74396257-0aa34100-4dd7-11ea-9f77-c27cca629da6.png"></details> <details><summary>📸 After</summary><img width="1040" alt="image" src="https://user-images.githubusercontent.com/1566689/74396288-260e4c00-4dd7-11ea-9c48-04bf3f9c814b.png"></details> Wording and stylesheet placement is absolutely negotiable!
☀️ Test successful - checks-travis |
This PR aims to close #544, and just adds a div and some styles for it.
📸 Before
📸 After
Wording and stylesheet placement is absolutely negotiable!