Skip to content
This repository has been archived by the owner on Mar 4, 2024. It is now read-only.

Add example: Combobox with list store #766

Merged
merged 1 commit into from
Dec 5, 2023

Conversation

ski0090
Copy link
Contributor

@ski0090 ski0090 commented Aug 10, 2022

Add an example that utilizes Liststore in Combobox.

@ski0090 ski0090 force-pushed the feacture/add_example branch 2 times, most recently from c7b5d58 to c249a51 Compare August 10, 2022 05:29
@GuillaumeGomez
Copy link
Member

Apart from my comments and the clippy lint, looks good to me.

@sdroege
Copy link
Member

sdroege commented Aug 10, 2022

the clippy lint

@GuillaumeGomez That's from old code and a new clippy lint, and because we always use latest beta this now triggers for an unrelated PR. Want to send a PR to fix the clippy lint?

@ski0090
Copy link
Contributor Author

ski0090 commented Aug 11, 2022

Apart from my comments and the clippy lint, looks good to me.

@GuillaumeGomez @sdroege thanks for the review

examples/combobox_liststore/README.md Outdated Show resolved Hide resolved
examples/combobox_liststore/main.rs Outdated Show resolved Hide resolved
@ski0090
Copy link
Contributor Author

ski0090 commented Aug 11, 2022

@bilelmoussaoui Opinion applied. thanks for the review

@ski0090
Copy link
Contributor Author

ski0090 commented Nov 29, 2023

@GuillaumeGomez @bilelmoussaoui Can i merge?

@GuillaumeGomez
Copy link
Member

Thanks!

@GuillaumeGomez GuillaumeGomez merged commit e7f0881 into gtk-rs:master Dec 5, 2023
32 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants