Skip to content
This repository was archived by the owner on Jul 22, 2022. It is now read-only.

Conversation

@zfett
Copy link

@zfett zfett commented Nov 13, 2019

What's the issue:

No styling for the top navigation links on the Explore page and no styling for the sidebar navigation menu on the Sponsors dashboard.

What's fixed:

  • Added .selected and :hover CSS for .subnav-link on Explore page
  • Added CSS styling for sidebar menu navigation on Sponsors Dashboard.

- Added .selected and :hover CSS for .subnav-link on Explore page  
- Added CSS styling for sidebar menu navigation on Sponsors Dashboard.
@zfett zfett closed this Nov 25, 2019
@snowe2010
Copy link
Contributor

Hi, Sorry for getting to this so late. I've had a bad illness. Why did you close the PR?

@zfett
Copy link
Author

zfett commented Dec 2, 2019

Sorry for that. I thought I PR’d on a dead topic so I closed it. I’ll reopen it for review!

@zfett zfett reopened this Dec 2, 2019
@author Quan You, Tyler Thrailkill, Lance Jordan, Lexa Hall, Jared Allard, seanysean
@license MIT
==/UserStyle== */
@-moz-document domain("github.com") {
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you pull the formatting changes out into a second PR? Also it seems your formatter is removing newlines where I put them so if you could leave those in place that would be good.
You also just copied and pasted the .user.css into the .css. Just don't worry about the .css file, I'm going to deprecate that soon.
Could you also provide some screenshots so I can verify the changes on my end too? Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants