Skip to content

fix: commits button not showing consistently #88

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
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

vinhyan
Copy link

@vinhyan vinhyan commented Dec 10, 2024

This PR fixes issue #81.

Now the Commits button/tab is always available when navigating back from other tabs, or clicking on the browser's back button.

issue-81.mov

@NirmalScaria
Copy link
Owner

Hey, This is awesome. It fixes the issue for the most part.
There is one side effect that came with the fix though. When a user installs the extension for the first time, they are usually taken to a first-run-experience page. It is a custom url that is handled by the extension.
You can try opening the following url with the original extension installed and see it in action.

https://github.com/NirmalScaria/le-git-graph/?fre=true&reason=install

With your changes, this page no longer works. It shows the github with a blank overlay instead. I would be happy to merge the PR if you could get some fix for this issue.

commitsButton keeps re-rendering causing popup tooltip not able to locate its position in the DOM
@vinhyan
Copy link
Author

vinhyan commented Jan 29, 2025

Hi @NirmalScaria! Been long overdue but the FRE has been fixed. Please review. Thanks!

legitgraph-issue81.mov

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