-
Notifications
You must be signed in to change notification settings - Fork 4
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
[CheckboxGroup#864] Fix disable checkbox group issue #867
[CheckboxGroup#864] Fix disable checkbox group issue #867
Conversation
[RadioButton#853] Fix size of radio button
…small [Slider#1977] Updated Slider handle size from 24x24 to 32x32
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.
Some of the changes were done in a separate merge. Could you do a rebase/catch-up merge, so it is clear which changes are only for this PR?
public override var isEnabled: Bool { | ||
didSet{ | ||
guard isEnabled != oldValue else { return } | ||
if isEnabled { |
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.
I'm not sure of the logic here. Why can't all the checkboxes be set to enabled as in the disabled states?
If someone enables the whole component, shouldn't it override the status of each single component?
self.checkboxes.forEach { $0.isEnabled == isEnabled }
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.
ex; Checkbox group has 3 item and first item is disable then consumer wants to disable all and enable all in this case first item will be enable but it should be disable
# Conflicts: # core/Sources/Components/Checkbox/View/SwiftUI/CheckboxGroupView.swift
I fixed disable issue checkbox group.