Skip to content

Optimize link handler code#1400

Merged
Chaphasilor merged 1 commit intoredesignfrom
link-handler-tweaks
Oct 5, 2025
Merged

Optimize link handler code#1400
Chaphasilor merged 1 commit intoredesignfrom
link-handler-tweaks

Conversation

@Maxr1998
Copy link
Collaborator

@Maxr1998 Maxr1998 commented Oct 3, 2025

Changes

As suggested by @Komodo5197, I replaced the retry loop with a SchedulerBinding frame callback to ensure the app is ready before handling links. Tested locally with the Linux build.

Related Issues

Originally discussed in 254dc41.

@Maxr1998 Maxr1998 requested a review from Komodo5197 October 3, 2025 21:22
@Maxr1998 Maxr1998 added refactoring Maintenance and code improvements, that don't change the behavior or add new features redesign-beta Issues related to the beta/redsigned version of Finamp labels Oct 4, 2025
@Maxr1998 Maxr1998 force-pushed the link-handler-tweaks branch from a16e24a to 384b7f0 Compare October 4, 2025 01:03
@Chaphasilor
Copy link
Collaborator

Neat. Does clicking a finamp:// link on Linux work if there's already a Window opened? Seems like that broke at some point (for Windows at least)...
If not, mind throwing that in?

@Chaphasilor Chaphasilor added hacktoberfest-accepted Issues accepted to count as a Hacktoberfest contribution hacktoberfest Issues available for Hacktoberfest labels Oct 4, 2025
@Maxr1998
Copy link
Collaborator Author

Maxr1998 commented Oct 4, 2025

Neat. Does clicking a finamp:// link on Linux work if there's already a Window opened? Seems like that broke at some point (for Windows at least)...

I'll have to check again, but I thought it should work already.

@Maxr1998 Maxr1998 force-pushed the link-handler-tweaks branch from 384b7f0 to 04b660d Compare October 4, 2025 23:07
@Maxr1998
Copy link
Collaborator Author

Maxr1998 commented Oct 4, 2025

@Chaphasilor yes, still works great!

@Chaphasilor Chaphasilor merged commit a72d787 into redesign Oct 5, 2025
5 checks passed
@Chaphasilor Chaphasilor deleted the link-handler-tweaks branch October 5, 2025 09:24
@Chaphasilor
Copy link
Collaborator

awesome, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hacktoberfest Issues available for Hacktoberfest hacktoberfest-accepted Issues accepted to count as a Hacktoberfest contribution redesign-beta Issues related to the beta/redsigned version of Finamp refactoring Maintenance and code improvements, that don't change the behavior or add new features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants