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

Remove Clone bound on map_tabs and filter_map_tabs #241

Merged
merged 4 commits into from
Jul 3, 2024

Conversation

Ved-s
Copy link
Contributor

@Ved-s Ved-s commented May 30, 2024

No point of having a FnMut if it can't mutably reference stuff outside

Copy link
Owner

@Adanos020 Adanos020 left a comment

Choose a reason for hiding this comment

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

This seems like a good change. I noticed you haven't applied it to {DockState,Surface,Tree}::{filter_tabs,retain_tabs} and {Surface,Tree}::map_tabs, is there a particular reason for this?

Also, since this is an API change, I'll move it to release-0.13.

@Adanos020 Adanos020 changed the base branch from main to release-0.13 June 2, 2024 10:00
@Ved-s
Copy link
Contributor Author

Ved-s commented Jun 2, 2024

I noticed you haven't applied it to {DockState,Surface,Tree}::{filter_tabs,retain_tabs} and {Surface,Tree}::map_tabs, is there a particular reason for this?

I guess i just removed it from the stuff i used and didn't check for other instances of it.
Fixed

@Adanos020 Adanos020 changed the base branch from release-0.13 to release-0.14 July 3, 2024 18:33
Copy link
Owner

@Adanos020 Adanos020 left a comment

Choose a reason for hiding this comment

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

Sorry for the massive delay, my irl stuff has been insane in the last 2 months.

The changes look good, merging them soon after I've updated the changelog. Thank you!

@Adanos020 Adanos020 merged commit f3a709f into Adanos020:release-0.14 Jul 3, 2024
4 checks passed
@Adanos020 Adanos020 mentioned this pull request Oct 2, 2024
Adanos020 added a commit that referenced this pull request Oct 2, 2024
* Remove `Clone` bound on `map_tabs` and `filter_map_tabs` (#241)

* remove Clone bound on map_tabs and filter_map_tabs

* remove Clone bound from all functions with Clone + FnMut bounds

* Update changelog

* Add missing period in changelog

---------

Co-authored-by: Adanos020 <adanos020@gmail.com>

* Update crate version and add MSRV change to changelog

* Fix heading type in changelog

* Update duplicate to 2.0 (#248)

* Egui 0.29 (#251)

* Upgrade to egui 0.29.

* Update changelog and readme

* Update release date in changelog

---------

Co-authored-by: Ved_s <vedomir.samarin.03@gmail.com>
Co-authored-by: Tau Gärtli <git@tau.garden>
Adanos020 added a commit that referenced this pull request Oct 19, 2024
… on each tab bar (#237)

* Correct comment typos

* Add two buttons on tab bars

- Close all tabs: "×"
- Collapse all tabs: "▼"

* Add some utility for recursive node collapse checking

* Preliminary implementation of collapsing windows

* Handle tab collapsing layout

* Resize windows when subnodes collapsed

* Implements leaf close all button

* Update changelog

* Allow dragging to a collapsed leaf

* Fix collapsed state checking after dnd splits

* Update examples

* Update changelog

* Fix up changelog after rebase

* Fix failing test and rename `Translations::window` to `Translations::leaf`.

* Remove `Clone` bound on `map_tabs` and `filter_map_tabs` (#241)

* remove Clone bound on map_tabs and filter_map_tabs

* remove Clone bound from all functions with Clone + FnMut bounds

* Update changelog

* Add missing period in changelog

---------

Co-authored-by: Adanos020 <adanos020@gmail.com>

* Update crate version and add MSRV change to changelog

* Fix heading type in changelog

* Fix tab bar scroll bar width on window surfaces

* Add close window button in context menu

* Add minimize window button

* Update CHANGELOG.md

* Fix cargo test

* Add tooltips to right-clickable tab bar buttons

* Add minimization activation with modifiers

* Add close window with modifiers

* Correct tooltip display conditions

* Upgrade to egui 0.29

* Add secondary button feature toggle

* Cleanup code

* Update examples

* Fix cursor icon logic

* Update examples

* Update changelog

* Make Japanese translations in the example more consistent with each other

---------

Co-authored-by: Adanos020 <adanos020@gmail.com>
Co-authored-by: Ved_s <vedomir.samarin.03@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants