Skip to content

Added sticky desktop navigation#916

Open
ralphocdol wants to merge 2 commits intoglanceapp:devfrom
ralphocdol:sticky-desktop-navigation
Open

Added sticky desktop navigation#916
ralphocdol wants to merge 2 commits intoglanceapp:devfrom
ralphocdol:sticky-desktop-navigation

Conversation

@ralphocdol
Copy link
Contributor

pages:
  - name: Playground
    sticky-desktop-navigation: true

Screencast

@ralphocdol ralphocdol changed the title Added sticky desktop navigation property Added sticky desktop navigation Dec 31, 2025
@benny01000010
Copy link

the code works but
as a feature I just don't like this. screen real estate is valuable and most people's dashboards aren't that long that scrolling to the top is a problem.

also - does it save where you are at if you swap over to another dashboard? or does it start you back up from the top? if that's the case this seems totally useless.

@ralphocdol
Copy link
Contributor Author

ralphocdol commented Jan 25, 2026

screen real estate is valuable and most people's dashboards aren't that long that scrolling to the top is a problem.

This is opt-in/optional, you don't need to enable it if you don't want/need to.

also - does it save where you are at if you swap over to another dashboard? or does it start you back up from the top?

Glance by design, reloads the entire page when switching tabs. I'm not sure why you even thought it's a good idea to preserve the position when you switch tabs.

The goal here is to not have to scroll up when you quickly need to switch tabs when you happen to expand a long widget. But then again, this is not enabled by default so you can just forget it even exist.

Copy link

@benny01000010 benny01000010 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for the context!
I supposed it would've made sense to look at the code before making a snarky comment lol.
now that I actually have reviewed it everything looks good. you implemented this nicely.

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