Skip to content

Missing navigation arrows on diff for Safari #581

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

Open
joleecl opened this issue May 19, 2025 · 5 comments
Open

Missing navigation arrows on diff for Safari #581

joleecl opened this issue May 19, 2025 · 5 comments
Labels
Accepted Accepted issue on our roadmap

Comments

@joleecl
Copy link

joleecl commented May 19, 2025

Details

Document file tree diff navigation arrows are not showing up for Safari browser, works fine on Chrome. Please check the link below:
https://readthedocs-landing--12114.org.readthedocs.build/platform/12114/guides/connecting-git-account.html?readthedocs-diff=true&readthedocs-diff-chunk=1

Expected Result

Screenshot from Chrome Version 136.0.7103.114 (Official Build) (arm64)
Image

Actual Result

Screenshot from Safari Version 18.5 (20621.2.5.11.8)
Image

@ericholscher ericholscher added the Accepted Accepted issue on our roadmap label May 19, 2025
@github-project-automation github-project-automation bot moved this to Planned in 📍Roadmap May 19, 2025
@humitos humitos transferred this issue from readthedocs/readthedocs.org May 19, 2025
@humitos
Copy link
Member

humitos commented May 19, 2025

Thanks for reporting this. I'm transferring the issue to the addons repository 👍🏼

@humitos
Copy link
Member

humitos commented Jun 3, 2025

@ericholscher can you test this on your side? I'm not sure how to debug this issue. Is there a way to run Safari on Linux?

@ericholscher
Copy link
Member

@humitos probably need to emulate it or something, but not sure there's a good way.

@humitos
Copy link
Member

humitos commented Jun 4, 2025

@ericholscher yeah, I'm not going to emulate Safari on Linux. It's too complicated. I was asking if you experience the same issue in your side.

@ericholscher
Copy link
Member

Yea, it can reproduce it. The spans with the arrows have 0 computed height:

Image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted Accepted issue on our roadmap
Projects
Status: Planned
Development

No branches or pull requests

3 participants