-
Notifications
You must be signed in to change notification settings - Fork 81
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
Addresses issue #694 #859
Addresses issue #694 #859
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for getting this ready @shruti862. Overall looks great and describes all scenarios requested in the issue. I suggest few tweaks regarding casing, conciseness, and clarity (writing guide has more information). You can submit most of them here in GitHub UI if you'd like.
@MisRob I have done all the suggested changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @shruti862! I pulled in the latest develop
and re-linted because of our recent Node version upgrade.
Description
I added Overview and Usage section on KTooltip documentation page
Issue addressed
Fixes issue #694
Addresses PR #853
Before/after screenshots
Changelog
KTooltip
pageKTooltip
Steps to test
(optional) Implementation notes
At a high level, how did you implement this?
Does this introduce any tech-debt items?
Testing checklist
Reviewer guidance
Comments