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

Added white hex outline on hovering round counter. #2491

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dheerajd5
Copy link

This fixes issue #2240

Note:
As of now I am calling the displayVisualState() function everytime for each hex in the grid, everytime showGrid()is called.
I didn't notice much of a performance difference but I've put the results in the comments related to this issue.

This is what it looks like after adding the white hex.
image

@vercel
Copy link

vercel bot commented Aug 27, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
ancientbeast ✅ Ready (Inspect) Visit Preview Aug 27, 2023 5:52pm

@ghost
Copy link

ghost commented Aug 27, 2023

👇 Click on the image for a new way to code review

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map legend

@DreadKnight
Copy link
Member

@dheerajd5 Heya! By "white outline" I didn't mean a new white hexagon (they tend to get edges overlapped at times), but css text effect for the letter-number coordinate, kinda like in this tut over here -> https://kinsta.com/blog/css-text-outline

@DreadKnight DreadKnight marked this pull request as draft October 1, 2023 23:01
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.

None yet

2 participants