Skip to content

fix: replace old selection reset logic in app_ui.rs#16

Merged
Mjoyufull merged 1 commit intoMjoyufull:devfrom
walldmtd:main
Oct 29, 2025
Merged

fix: replace old selection reset logic in app_ui.rs#16
Mjoyufull merged 1 commit intoMjoyufull:devfrom
walldmtd:main

Conversation

@walldmtd
Copy link
Contributor

@walldmtd walldmtd commented Oct 29, 2025

Summary

It looks like some of the selection reset logic was left behind in app_ui.rs. This fix makes it match the rest.

  • I did basic linting
  • I'm a clown who can't code 🤡

Changes

  • Fixed a piece of app_ui.rs that was still using the old selection reset logic

Testing

  1. Build with cargo build --release
  2. Run fsel and highlight anything but the first app
  3. Start typing input and verify the selection resets

Breaking Changes

None

Related Issues

Continuation of PR #11

@Mjoyufull
Copy link
Owner

thanks again for the notice, have i met you before like on disc or somthing?

@Mjoyufull Mjoyufull self-requested a review October 29, 2025 17:03
@Mjoyufull Mjoyufull merged commit c1c871d into Mjoyufull:dev Oct 29, 2025
@walldmtd
Copy link
Contributor Author

thanks again for the notice, have i met you before like on disc or somthing?

Maybe? I'm in a couple of Linux/WM servers, I'm not super active though.

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