-
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: (core) added selected button with group #1543
Conversation
Deploy preview for fundamental-ngx ready! Built with commit 21d91f2 |
60801a4
to
3560243
Compare
from the changes I see, this is not just docs change |
Changed! |
@stefanoScalzo Can you add some description on the button group examples - the 2 use cases we support? |
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.
see my previous comment
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.
In button group example there are different sizes example. I think it has been deprecated on styles for some time. Maybe we should remove them in this PR.
this.calendarI18n.i18nChange.next(); | ||
} | ||
|
||
} |
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.
Do we need to add this file ?
this.calendarI18nLabels.labelsChange.next(); | ||
} | ||
|
||
} |
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.
Do we need to add this file ?
} | ||
} | ||
|
||
} |
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.
Do we need to add this file ?
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.
they shouldnt be there ill remove them
Hi @stefanoScalzo could you try to rebase this branch to |
7f926d3
to
36b5ba4
Compare
36b5ba4
to
21d91f2
Compare
Please provide a link to the associated issue.
fixes:#1542
fixes: #1211
Please provide a brief summary of this pull request.
Allow the button group to have one option selected and toggle as each button gets clicked
Please check whether the PR fulfills the following requirements
https://github.com/SAP/fundamental-ngx/blob/master/CONTRIBUTING.md
Documentation checklist: