Skip to content
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

Use same name & order for toggler::new and helper #1616

Merged
merged 1 commit into from
Jan 2, 2023

Conversation

13r0ck
Copy link
Member

@13r0ck 13r0ck commented Dec 22, 2022

The helper function for the toggler widget switched the order and name of the arguments passed when creating the toggler widget. This just standardizes the order whether the dev is using the helper or the associated function.

The helper function for the toggler widget switched the order and
name of the arguments passed when creating the toggler widget.
This just standardizes the order whether the dev is using the
helper or the associated function.
Copy link
Member

@hecrj hecrj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

I believe Checkbox could use the same treatment.

@hecrj hecrj added improvement An internal improvement feature New feature or request widget labels Jan 2, 2023
@hecrj hecrj added this to the 0.7.0 milestone Jan 2, 2023
@hecrj hecrj merged commit a5a406f into iced-rs:master Jan 2, 2023
@13r0ck 13r0ck deleted the standardize-toggler-functions branch January 2, 2023 23:04
13r0ck added a commit to pop-os/iced that referenced this pull request Jan 3, 2023
The helper function for the checkbox widget switched the order
and name of the arguments passed when creating the checkbox
widget. This just standardizes the order whether the dev is using
the helper or the associated function.

Continuation of iced-rs#1616
13r0ck added a commit to pop-os/iced that referenced this pull request Jan 4, 2023
The helper function for the checkbox widget switched the order
and name of the arguments passed when creating the checkbox
widget. This just standardizes the order whether the dev is using
the helper or the associated function.

Continuation of iced-rs#1616
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request improvement An internal improvement widget
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants