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

Fill space with navbar items for wide mobile view. #696

Merged
merged 4 commits into from
Mar 4, 2022

Conversation

mtbc
Copy link
Contributor

@mtbc mtbc commented Feb 7, 2022

Description

Mobile view of popup navigation items from hamburger now spreads out horizontally across the screen if it is still quite wide.

Motivation and Context

Previously it looked rather a waste of screen for @danielskatz's view.

Checklist:

  • I have posted the link for the PR in the usrse slack (#website) to ask for reviewers
  • I have previewed changes locally
  • I have updated the README.md if necessary

cc @usrse-maintainers

@vsoch
Copy link
Member

vsoch commented Feb 7, 2022

Preview: https://1942-120906408-gh.circle-artifacts.com/0/usrse.github.io/index.html

huh I'm not sure I like this better? It looks a little messy / weird. What do others think?

image

vs.

image

The latter is much more natural for a mobile layout.

@danielskatz
Copy link
Contributor

I agree about mobile, but since this is what I get on my desktop, I like the new version there.

@mtbc
Copy link
Contributor Author

mtbc commented Feb 7, 2022

It may work for @danielskatz to have that odd mode only on the intermediate widths, I'll push that as an alternative then.

@danielskatz
Copy link
Contributor

See attached screenshots of how this looks to me in various widths - the 4th one is very odd, since it is missing the headers
Screen Shot 2022-03-02 at 10 15 45 AM
Screen Shot 2022-03-02 at 10 15 54 AM
Screen Shot 2022-03-02 at 10 16 02 AM
Screen Shot 2022-03-02 at 10 16 10 AM

@mtbc
Copy link
Contributor Author

mtbc commented Mar 2, 2022

@danielskatz: Looks as if you don't have the latest commit?

Yeah, I found it odd too that the headers are visible only on mouseover. That preexists my PR so I assume somebody liked it like that, I've no strong opinion either way.

@danielskatz
Copy link
Contributor

The headers was not related to mouse position, but just to window width. But I'll check the newest preview and see if this is still the case.

@danielskatz
Copy link
Contributor

Ok, the new part is consistent, though I do think not having the header show in the multi-column view is odd. I would much rather see them than not see them, particularly when the info is somewhat complex.

Three screen shots follow, which seems reasonably otherwise.
Screen Shot 2022-03-02 at 10 33 03 AM
Screen Shot 2022-03-02 at 10 33 11 AM
Screen Shot 2022-03-02 at 10 33 20 AM

@mtbc
Copy link
Contributor Author

mtbc commented Mar 2, 2022

Ohh, interesting, yes - there is probably still a bit of CSS that needs tweaking to make them appear. I'll try to figure which is the culprit.

@mtbc
Copy link
Contributor Author

mtbc commented Mar 2, 2022

... I've found it but will add a bit of extra styling too to help make it clear which down-arrow symbol matches which heading.

@mtbc mtbc force-pushed the spread-navbar-dropdowns-across branch 2 times, most recently from e2b6fbd to db77fb5 Compare March 2, 2022 18:27
@mtbc mtbc force-pushed the spread-navbar-dropdowns-across branch from db77fb5 to 42cced0 Compare March 2, 2022 18:27
@mtbc
Copy link
Contributor Author

mtbc commented Mar 2, 2022

@danielskatz: I hope that this is nicer for you.

@vsoch
Copy link
Member

vsoch commented Mar 2, 2022

This still feels/looks a little wonky to me. It looks fine as a single listing and then I get to a width and there are dropdown headers that probably shouldn't be dropdown because the links are expanded below them.
image

@mtbc
Copy link
Contributor Author

mtbc commented Mar 2, 2022

Ah, I was just mirroring what the current site does, I can get rid of those down arrows in both of the narrow cases though - sounds like a good suggestion, @vsoch.

@mtbc
Copy link
Contributor Author

mtbc commented Mar 2, 2022

Maybe this one's good. 🤞

@vsoch
Copy link
Member

vsoch commented Mar 2, 2022

That's definitely better! 👍

@danielskatz
Copy link
Contributor

This looks good to me - thanks @mtbc for following up and @vsoch for helping

@mtbc mtbc merged commit aca516f into USRSE:main Mar 4, 2022
@mtbc mtbc deleted the spread-navbar-dropdowns-across branch March 4, 2022 16:39
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.

None yet

3 participants