-
-
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
Index page redesign #864
Index page redesign #864
Conversation
Looks like there is a css conflict. |
Apologies for the delay!! I think this design looks great!! We will need to rebase from main to resolve the merge conflict that Sean mentioned. I'm getting a Sassc Syntax error when trying to view the changes locally. I'm trying to figure out if that is something on my end or something we will need to look at before merging.
|
Thanks for checking on the home.scss. I am no longer seeing an error locally. This looks great. I really like the re-organized navigation structure. One question on that - would it be possible to change the "Users" nav item to be "User" since those items pertain to stuff a single user can do as opposed to the admin nav item for managing users? I'm seeing an issue with a few of the buttons. Would you mind taking a look? The CI picked up a couple of failing specs for the homepage statistics. I think those specs are just looking for some of the old styles. Happy to help get those resolved, if you'd like. I'll be available in the morning, if you have any questions or want me to take a look at anything. Thanks again for all the work on this. |
What this PR does:
Final Design -