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

Added home, settings and explore menu entries back into sidebar #336

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Tywele
Copy link
Contributor

@Tywele Tywele commented Jun 20, 2024

I think the new navigation layout is too different from the old layout and messes up muscle memory and the frontpage button is also too far away and leads to too much mouse movement for a button that is regularly used.

I have moved the frontpage, settings, and explore buttons back to the sidebar for easier access and kept everything else as is.

Copy link

vercel bot commented Jun 20, 2024

@Tywele is attempting to deploy a commit to the Xylight's projects Team on Vercel.

A member of the Team first needs to authorize it.

@sevonj
Copy link
Contributor

sevonj commented Jun 20, 2024

I'm not sure what kind of plan Xylight has for the sidebar, but I agree on having all fundamental navigation there.

Screenshot of this pr change

image

There are more things that could be changed in the sidebar. I'm not sure if they're all good ideas, but I'll just throw them here anyway.

  • I use the All/Subscribed buttons in voyager a lot. Maybe we could do that?
    • A more radical option would be to move all the location items here. It would free up some precious horizontal space; my phone is too narrow to show the view selectbox.
      • Actually, the sidebar or subbed communities even accessible on narrow screens?
  • Move settings down. Now they are roughly categorized to a "browsing group" and an "account/settings group".
vger

image

Something like this?
image

@Xyphyn
Copy link
Owner

Xyphyn commented Jun 22, 2024

The issue with this is that it removes location cohesion. Currently, the sidebar is associated with everything related to your account. (Profile, inbox, saved, account switcher, communities), and the navbar is associated with navigation to common pages. If I were to add these into the sidebar, I'd probably have to commit hard and completely remove the regular navbar, but this would ruin people's muscle memory.

@Tywele
Copy link
Contributor Author

Tywele commented Jun 22, 2024

To be fair with the change to the navbar you've already ruined people's muscle memory once.
You also used to have an option for legacy navigation layout which would be perfect for this as this is the real legacy navigation that Photon once had.
With a bit of tinkering I could implement it as an option. Maybe I would need some assistance as this pull request is the first time ever I've dealt with Svelte.

@Xyphyn
Copy link
Owner

Xyphyn commented Jun 22, 2024

The legacy navigation layout still exists? You just set both panel mode and position to "adaptive"

@Tywele
Copy link
Contributor Author

Tywele commented Jun 22, 2024

But that doesn't bring back the menu entries in the sidebar. That's the point. Even with both set to adaptive my muscle memory (and probably lot's of other people's) is completely useless with any version past 1.28

@Tywele
Copy link
Contributor Author

Tywele commented Jun 22, 2024

The option for this could be called "Legacy Sidebar" or something.

@Tywele
Copy link
Contributor Author

Tywele commented Jun 24, 2024

I added a setting for the legacy sidebar navigation:

Setting enabled

Setting disabled

The setting also disables the "Settings" menu entry when you are not logged in since with the setting enabled you would've duplicate menu entries of "Settings".

The setting can also be set via an ENV and is set to false by default.

@Xyphyn
Copy link
Owner

Xyphyn commented Jun 24, 2024

image

What if I added completely custom pins?

@Tywele
Copy link
Contributor Author

Tywele commented Jun 24, 2024

That's an interesting idea that I would like to spin a bit further: Since every menu entry in the sidebar is accessible some other way anyway (mainly through the Profile Picture menu in the top right). You could take this concept and make a setting to select which menu entries in the sidebar are shown (and in which order) instead of pins that are on top of it. The default could either be the layout it is right now or the sidebar would only contain Favorites, Moderates and Subscribed.

What do you think of that?

I would also take a shot at it if you don't want to invest the time in it.

@Xyphyn
Copy link
Owner

Xyphyn commented Jun 24, 2024

The pins system is easier to code, but I could try that.

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.

3 participants