You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Feb 25, 2020. It is now read-only.
Describe the bug
If the user has a commit/node selected after clicking it. hovering over the commit/node that is highlighted yellow does not indicate that you can click it when you actually can, bringing up a modal pop up.
To reproduce
Steps to reproduce the behavior:
Launch VisualGit
Select a repository that has two or more commits/nodes on it
Click on a commit/node, Close the modal pop-up
Hover over the node highlighted in green
You do not see any indication that you can click it
Clicking it brings up modal pop up
Expected behavior / Acceptance criteria
Either one of the following:
it must be clear that you click any commit/node on the GUI when you hover over them
Disallow users to click on the yellow highlighted commits/nodes
Screenshots
N/A
Desktop (please complete the following information):
OS: Windows 10
Browser: Chrome
Additional context
Add any other context about the problem here.
Hi @SamuelZheng11 - I've looked into reproducing this as you have said, and the problem seems to be as you have outlined, except the solution appears to be simply clicking out of having selected the node itself? Then the node you previously selected notifies you that it is clickable.
Outside of that however, I agree with the two solutions you have proposed, and am therefore happy to approve this - Approved!
Describe the bug
If the user has a commit/node selected after clicking it. hovering over the commit/node that is highlighted yellow does not indicate that you can click it when you actually can, bringing up a modal pop up.
To reproduce
Steps to reproduce the behavior:
Expected behavior / Acceptance criteria
Either one of the following:
Screenshots
N/A
Desktop (please complete the following information):
Additional context
Add any other context about the problem here.
Related issues
#298
The text was updated successfully, but these errors were encountered: