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 Tooltip Helper info in every label for easy to understand #315

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from

Conversation

aancw
Copy link

@aancw aancw commented Apr 23, 2021

Added Tooltip Helper info in every label for easy to understand #277

image

@cobbr
Copy link
Owner

cobbr commented Apr 23, 2021

Hey @aancw , this looks really cool, could be a great addition!

Haven't taken a deep dive into the code yet, but I'm wondering if we could break out the descriptions/tips out in a better way, where they aren't just static inside the component.

@aancw
Copy link
Author

aancw commented Apr 23, 2021

Hi @cobbr, that's cool idea. I'm not familiar with Razor, but I got what you mean. It's using defined dictionary like this?

Keyname: tips description

And showing that tips using class, id or something that unique, isn't?

For now, I'm doing a static way because that's what can I do right now and if you want, you can merge it if you like it :)

I will learn Razor later, so I can implement it as dynamic way.

Thanks

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