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

Allow custom character for active dot #4186

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

Conversation

rywall
Copy link

@rywall rywall commented May 2, 2022

I wanted to use an unfilled circle for inactive dots.

image

@markvantilburg
Copy link
Contributor

@ahmadalfy Looks like a nice addition to me, but only includes the sass file and not the less/css file

@pablo67340 pablo67340 self-assigned this Apr 13, 2024
@pablo67340 pablo67340 self-requested a review April 13, 2024 01:58
Copy link
Collaborator

@pablo67340 pablo67340 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rywall

Just as @markvantilburg mentions, excellent additions, would like to see the complete less/css versions, if you're unable to do this, I can PR into your fork prior. Let me know.

@rywall
Copy link
Author

rywall commented Apr 15, 2024

@pablo67340 I added the less version. Hope this helps.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants