-
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) addressed list header label issue #3457
Conversation
Deploy preview for fundamental-ngx ready! Built with commit 6306c37 |
a5e0ecf
to
ce6451e
Compare
LGTM, but could you wait until #3400 is merged? So you would be able to just put |
ce6451e
to
de948a4
Compare
Hi @JKMarkowski , Updated with 3400. |
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.
Looks good.
added new line at end of the file fix icons in button breaking because of latest 0.12
* fix to issues 3266 * fix to list issue 3423 added new line at end of the file fix icons in button breaking because of latest 0.12 * rebase with 3400 changess
Please provide a link to the associated issue.
#3423
#3266
#3416
Please provide a brief summary of this pull request.
labels with special characters in toolbar are not supported in rtl to ltr mode. So removed special charters in the label text as per current toolbar styles support.
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: