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

Hint overlays do not appear on GitHub Enterprise #4446

Closed
bent0b0x opened this issue Mar 28, 2024 · 6 comments
Closed

Hint overlays do not appear on GitHub Enterprise #4446

bent0b0x opened this issue Mar 28, 2024 · 6 comments

Comments

@bent0b0x
Copy link

bent0b0x commented Mar 28, 2024

Describe the bug

Pressing f to trigger link/button hints does not yield any visible overlays on GitHub Enterprise. This is probably due to 412cc37.

To Reproduce

Steps to reproduce the behavior:

  1. Go to any github enterprise page
  2. Press f

I would expect the overlays to appear so that I can press character keys to open links, but they do not appear.

Browser and Vimium version
Vimium: 2.1.0
Chrome: Version 123.0.6312.87 (Official Build) (arm64)

@philc
Copy link
Owner

philc commented Mar 30, 2024

I'm surprised -- I'm not sure what would be so different about github enterprise vs. the regular site that would break link hints.

Unfortunately, I don't have access to any github enterprise instances. Could someone with access to one take a closer look?

@bent0b0x
Copy link
Author

bent0b0x commented Mar 30, 2024

It looks like the hint markers are all being rendered, but they are not visible due to a css rule that gets included in GHE (at least the version my org is on, which is GitHub Enterprise Server 3.9.4):

@supports not selector([popover]:open)
[popover]:not(.\:open) {
    display: none;
}

My css chops are a bit stale, but it looks like this is ultimately saying "hide all popover elements unless they are explicitly open"?

I think adding display: 'block' here would fix the issue, I can PR that to discuss

EDIT: PR here

@roachsinai
Copy link

I'm surprised -- I'm not sure what would be so different about github enterprise vs. the regular site that would break link hints.

Unfortunately, I don't have access to any github enterprise instances. Could someone with access to one take a closer look?

No. I cannot use / to search a word in regular site now. Type / and after type target word, vimium show it found matches, but can't jump to it by press Enter.

@bent0b0x
Copy link
Author

bent0b0x commented Apr 3, 2024

FYI, here's a temporary config-level fix until a new version of Vimium is released w/ a patch

#4451 (comment)

@roachsinai
Copy link

Thanks for quick reply!

@philc
Copy link
Owner

philc commented Apr 4, 2024

v2.1.2 includes this fix and I've submitted it to the stores today.

@philc philc closed this as completed Apr 4, 2024
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

No branches or pull requests

3 participants