Skip to content
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

Random Style Improvements #239

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

tandpfun
Copy link

Heyo!

I've made some random style improvements on the website. I'll write a bit about what I've changed below:

Guild List Improvements:

  • I cut off long guild names to prevent them from overlapping as seen below:
    Before:

    After:
  • I added a "Not Allowed" cursor on the disabled buttons:
  • I added transition to the config button

Navbar Improvements

  • Added transitions to the links & "go to dashboard"/"go to docs" buttons
  • Added a hover opacity for the dashboard title

Guild Config Page

  • Added a transition to the save button

Format Index Files
The index app file & stuff weren't inside the npm run format command's reach so they were never formatted before. I formatted em.

tandpfun added 4 commits July 23, 2021 16:41
- Cut off long guild names
- Not allowed cursor on disabled buttons
- Add transition to buttons
- Added transitions to stuff
@almeidx
Copy link
Collaborator

almeidx commented Jul 24, 2021

IMO the Guild Name should be truncated with an ellipsis instead of not rendering the overflow

@tandpfun
Copy link
Author

Good idea, I'm not the best at css lol and couldn't use the ellipsis without it overlapping with the next guild. I can try to do it today unless you know how.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants