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.

Sign up for GitHub

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

Jump to bottom

x/pkgsite: Jump To breaks the page when selecting it after clicking a page link #43804

Open
katiehockman opened this issue Jan 20, 2021 · 3 comments
Open

x/pkgsite: Jump To breaks the page when selecting it after clicking a page link #43804

katiehockman opened this issue Jan 20, 2021 · 3 comments
Labels
NeedsFix The path to resolution is known, but the work has not been done. pkgsite/frontend Issues related to pkgsite HTML/CSS/JavaScript and frontend development pkgsite
Milestone
pkgsite/frontend

Comments

@katiehockman
Copy link
Contributor

This bug is kind of hard to explain, but can be repro'd with these steps:

  1. Navigate to https://pkg.go.dev/crypto/rand
  2. Click something in the Index, e.g Prime (the URL should now be https://pkg.go.dev/crypto/rand#Prime)
  3. Click the "Jump to..." (a dialogue should appear)
  4. Press the back button
  5. Note that the box has disappeared but the rest of the page is still greyed-out and non-interactable

/cc @julieqiu
@katiehockman katiehockman added the NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. label Jan 20, 2021
@gopherbot gopherbot added this to the Unreleased milestone Jan 20, 2021
@julieqiu julieqiu modified the milestones: Unreleased, pkgsite/frontend Jan 20, 2021
@julieqiu julieqiu added NeedsFix The path to resolution is known, but the work has not been done. and removed NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. labels Jan 20, 2021
@julieqiu
Copy link
Member

I think the issue is that clicking the back button doesn't close the input box. It just takes me to the top of the page:

Screen Shot 2021-01-20 at 10 20 55 AM

When I scroll to the bottom, the input box is still there:

Screen Shot 2021-01-20 at 10 21 14 AM

/cc @jamalc

@Omkar76
Copy link

Omkar76 commented Jan 27, 2021

@julieqiu , There have been few changes in css code since this issue was opened.The stylesheet was modified for issue #43840. Now both the .JumpDialog and .ShortcutsDialog stays on viewport. They don't get scrolled away when the screen is scrolled.

So now my question is - is it still necessary to close the input boxes when back button is pressed? because user can now see the boxes even after pressing back button and will know they need to close input box in order to interact with rest of the page.

@georgehu
Copy link

Thanks for flagging this issue @Omkar76. I agree that clicking back should return you to the previous state without a grey overlay or the "jump to" dialog.

CC @julieqiu

@hyangah hyangah added the pkgsite/frontend Issues related to pkgsite HTML/CSS/JavaScript and frontend development label May 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
NeedsFix The path to resolution is known, but the work has not been done. pkgsite/frontend Issues related to pkgsite HTML/CSS/JavaScript and frontend development pkgsite
Projects
None yet
Milestone
pkgsite/frontend
Development

No branches or pull requests
6 participants
@georgehu @hyangah @julieqiu @gopherbot @katiehockman @Omkar76