-
Notifications
You must be signed in to change notification settings - Fork 656
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
Web GUI update #141
Web GUI update #141
Conversation
Yeah, looks great. |
👍 |
Merging. Thanks! |
@vernak2539 , I added login button near the logo in 67b4528, but it doesn't quite fit there. Can you take a look to see how it could be improved? |
@rlidwka, yeah I have an idea of what we could do. I'll probably able to look at it in the next few days. Is there a specific branch you're working on? |
No, it's all in the master branch right now. |
okay no worries. Should have something this week |
@vernak2539, thanks again for the update. I wanted to ask, was there any reason to use "position: fixed" for the main header there? It creates an issue with anchors in readmes ( |
I guess the only reason was to keep the header, and subsequently the search bar, always at the top of the page. I didn't even think about the anchors in the readme files, but you're right; this would cause that behavior. I think the static positioning would be just fine. Especially for people contributing in the future. I can change it, but if you got it no worries. |
Updated the Web GUI to use bootstrap and have a cleaner design (fully responsive). Removed the big logo at the top, since most people are interested in the published modules and not a header. Put a static header on top, and it will be easy to add login buttons when it gets to that point.
Put pictures below, but running locally would probably be better.
Desktop + Mobile
(1) Large, (2) Large with vertical scroll
(1) Medium/Small, (2) Medium/Small with vertical scroll
Extra Small