search

nt4f04uNd / sweyer

Music player built with Flutter
BSD 3-Clause "New" or "Revised" License
197 stars 47 forks source link

Update theme_tailor #119

Closed Abestanis closed 3 months ago

Abestanis commented 4 months ago

Hey, it has been a while, I hope you are doing well!

I had a bit of spare time and I decided to work towards getting the widget support merged into Sweyer. But because I updated my flutter version, the project was no longer building. Updating to the newer flutter requires updating some dependencies.

Rather than giving you one big MR, I want to split it up into very small changes, but I can't compile with just these changes, they are cherry picked from my local changes so I can't fully guarantee that they work on their own and hope CI will tell me if I missed something.

This updates theme_tailor from v2 to v3. They removed the @tailor way to define the theme, so I had to migrate to their new way according to their migration guide

nt4f04uNd commented 3 months ago

Hi @Abestanis, absolutely. I hope you are doing well too

I'm sorry this is taking such a long time to review, I will be sure to check your PRs tomorrow. Appreciate your contributions

Abestanis commented 3 months ago

Good to hear you are doing well. I'm doing good too, thank you.

No rush on the review, I know this is a hobby and not a job and I appreciate your time.