-
Notifications
You must be signed in to change notification settings - Fork 125
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
fix: (Platform) add tooltip to the components #3785
Conversation
Deploy preview for fundamental-ngx ready! Built with commit 3f5c0fb |
Deploy preview for fundamental-ngx ready! Built with commit 03e5e55 |
b24ed79
to
ec4962e
Compare
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.
LGTM
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.
Also the PR title should be following the standard we agreed on.
...ct-status/platform-object-status-example/platform-object-status-large-example.component.html
Outdated
Show resolved
Hide resolved
...status/platform-object-status-example/platform-object-status-inverted-example.component.html
Outdated
Show resolved
Hide resolved
...tatus/platform-object-status-example/platform-object-status-icon-text-example.component.html
Outdated
Show resolved
Hide resolved
...tform-object-status-example/platform-object-status-clickable-and-icon-example.component.html
Outdated
Show resolved
Hide resolved
platform tool tip adding
update the tooltip in paltform
adding title to the examples to add tooltip
ea21609
to
75014fd
Compare
...info-label/platform-info-label-example/platform-info-label-aria-label-example.component.html
Outdated
Show resolved
Hide resolved
fixing accessability broken 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.
And please add proper naming for the PR titles. |
platform tool tip adding
Please provide a link to the associated issue.
fixes #3697
Please provide a brief summary of this pull request.
Actual: None of the info label examples have a tooltip.
Expected: info label should have a tooltip on hover.
Please check whether the PR fulfills the following requirements
https://github.com/SAP/fundamental-ngx/blob/master/CONTRIBUTING.md
https://github.com/SAP/fundamental-ngx/wiki/PR-Review-Checklist
Documentation checklist: