Skip to content

Track File Export: add 'Apply to all' checkbox, remove ".. All" buttons - #13614

Merged
acolombier merged 1 commit into
mixxxdj:mainfrom
ronso0:track-export-apply-to-all-checkbox
Feb 21, 2025
Merged

Track File Export: add 'Apply to all' checkbox, remove ".. All" buttons#13614
acolombier merged 1 commit into
mixxxdj:mainfrom
ronso0:track-export-apply-to-all-checkbox

Conversation

@ronso0

@ronso0 ronso0 commented Aug 30, 2024

Copy link
Copy Markdown
Member

based on #13610, so it's only the last commit 1392b33

Looks like this now:
track-export-checkbox

new tr strings:
Replace
Apply to all files

(if you like I can rebase onto 2.5)

Fixes #13613

@daschuer

Copy link
Copy Markdown
Member

Uh that was quick. Thank you. The new texts will likely be untranslated in a 2.5 release, so let's keep this for 2.6.

@ronso0 ronso0 added this to the 2.6-beta milestone Sep 6, 2024
@ronso0
ronso0 force-pushed the track-export-apply-to-all-checkbox branch from 1392b33 to c772e6f Compare September 6, 2024 22:21
@ronso0
ronso0 marked this pull request as ready for review September 6, 2024 22:22
Comment thread src/library/export/trackexportdlg.cpp Outdated
Comment thread src/library/export/trackexportdlg.cpp
Comment thread src/library/export/trackexportdlg.cpp Outdated
@ronso0

ronso0 commented Feb 20, 2025

Copy link
Copy Markdown
Member Author

Thanks for your review!
All done I guess.

@acolombier acolombier left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! I'm tempted to try the /softfix command but scared to make a mess...

@ronso0

ronso0 commented Feb 20, 2025

Copy link
Copy Markdown
Member Author

/softfix

@ronso0

ronso0 commented Feb 20, 2025

Copy link
Copy Markdown
Member Author

Well, let's see...

@github-actions
github-actions Bot force-pushed the track-export-apply-to-all-checkbox branch from 4f74655 to cb67dbe Compare February 20, 2025 23:51
@ronso0

ronso0 commented Feb 20, 2025

Copy link
Copy Markdown
Member Author

Looking good! 🎉

@ronso0

ronso0 commented Feb 20, 2025

Copy link
Copy Markdown
Member Author

Though CI hasn't been triggered (clazy was still running when I use softfix)

@acolombier

acolombier commented Feb 21, 2025

Copy link
Copy Markdown
Member

Though CI hasn't been triggered (clazy was still running when I use softfix)

Ah yeah, this is because we are using the default github token (github-actions, which cannot trigger sub-sequent builds). Might want to create a mixxx-bot app soon!

Edit: created #14386

@acolombier
acolombier merged commit b3da7d3 into mixxxdj:main Feb 21, 2025
@ronso0
ronso0 deleted the track-export-apply-to-all-checkbox branch February 21, 2025 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Align track export replace dialog with the file managers

4 participants