-
Notifications
You must be signed in to change notification settings - Fork 142
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
Sort by play count #1080
Sort by play count #1080
Conversation
Decimate1405
commented
Mar 4, 2025
•
edited
Loading
edited
- Feature request
marquee effect on song title
Merge pull request #1 from Decimate1405/marquee
fixed non marquee text alignment issue
…that's a temp fix for marquee after flutter 3.2.2
…r based on toggle switch and title size
…ine marquee duplicates for short titles
…or on the now playing bar adapts to the theme. Fixed bug where with the one line marquee enabled, even non-overflowing text uses a marquee on the player screen.
…ttings and renamed it "Allow one line break before using scrolling text"
…es instead of using marquee
…y trigger for 3 or more
…eshold switches to marquee
…d toggle title and description
@Decimate1405 thanks for the PR! But something got messed up, the diff is massive, with tons of JSON files, and there are commits from your last PR in there. |
f6e9325
to
b030f33
Compare
i apologize all this mess, git is sometimes confusing for me. I added the changes to the redesign branch but when I try to create a new PR with that branch I still see these old changes instead of just the new one. Trying to figure this out now |
This is most likely because I squashed all the commits from your last PR into a new commit that contained everything. That helps to keep the commit history clean and uncluttered. But now Git still doesn't see your original commits in the upstream branch, so it tries to merge them in once again, which causes the conflict. To fix this, you should reset the branch on your fork to the base branch, by checking it out ( |
Thank you very much for explaining! Will work on this |