SimpleMobileTools / Simple-Music-Player

A clean music player with a customizable widget, stylish interface and no ads.
https://www.simplemobiletools.com
GNU General Public License v3.0
1.29k stars 408 forks source link

Migrate to View binding, SDK 34 and Version catalogs #592

Closed naveensingh closed 1 year ago

naveensingh commented 1 year ago

Major changes:

Minor changes:

Notes:

tibbi commented 1 year ago

have you also tested upgrading from master branch in release mode?

tibbi commented 1 year ago

selecting items like folders by long press and drag doesnt work, only works at main tracks fragment

naveensingh commented 1 year ago

have you also tested upgrading from master branch in release mode?

No, I did some tests in release mode but not the migration one. Only tested the upgrade with debug builds, will check it now. I'd tested migration in release mode at the time of media3 changes though

naveensingh commented 1 year ago

selecting items like folders by long press and drag doesnt work, only works at main tracks fragment

I'm on it...

naveensingh commented 1 year ago

selecting items like folders by long press and drag doesnt work, only works at main tracks fragment

I broke it :)

naveensingh commented 1 year ago

have you also tested upgrading from master branch in release mode?

did some basic testing, and nothing looks broken when upgrading from the last release in release mode (on android 14 too)

naveensingh commented 1 year ago

Fixed everything, check now, please.

tibbi commented 1 year ago

if I have 1 playlist only and I long press it, it doesnt have the selected background. Albums opened from Artists tab dont have selected backgrounds either.

naveensingh commented 1 year ago

I have 1 playlist only and I long press it, it doesnt have the selected background

have you pulled the latest changes? I broke it during media3 changes and fixed it above https://github.com/SimpleMobileTools/Simple-Music-Player/pull/592/commits/bebd99bf5ff53a77ae0789e746e13fd10a92cf52

Albums opened from Artists tab dont have selected backgrounds either

I remember that one never really worked and I kept ignoring it, I'm going to fix it now. EDIT: Fixed it.

tibbi commented 1 year ago

right, it works well now, but colors arent properly updated in onResume. The placeholder icon and selectiong texts either.

naveensingh commented 1 year ago

Not sure what I need to do reproduce this. I select some items => go to home screen => reopen the app => everything is just as before

tibbi commented 1 year ago

no need to select anything, just open the app settings, change colors and go back at the main screen, you will see placeholder not being changed. Then you can also long press items.

naveensingh commented 1 year ago

Fixed the color issues and resolved some conflicts. I think the colors were not updated in some other apps with lists, will have to check.

tibbi commented 1 year ago

the placeholder icon at tracks that have no image still doesnt toggle between light/dark properly

naveensingh commented 1 year ago

I didn't have any tracks without thumbnails so didn't notice it, added now.

tibbi commented 1 year ago

yeah looks good now, lets see how will it go ;) Thanks