Skip to content
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

fix app/window icons #4254

Merged
merged 2 commits into from
Sep 4, 2021
Merged

fix app/window icons #4254

merged 2 commits into from
Sep 4, 2021

Conversation

ronso0
Copy link
Member

@ronso0 ronso0 commented Aug 30, 2021

  • fix regression introduced by 564df77, all dialogs (except warnings) now use the Mixxx icon
    (not 100% sure I caught all, and I didn't figure why the icon is implicitely inherited, for example by the Replace Hotcue Color dialog)
  • make About dialog reachable via Alt+Tab by not parenting it to MixxxMainWindow
    (this was a bit annoying until now, i.e. when trying to jump back and forth between About Mixxx and web browser to report bugs)

@JoergAtGithub
Copy link
Member

The regression is in the 2.3.1 branch, shouldn't the fix go into 2.3.1 too?

@Be-ing Be-ing changed the base branch from main to 2.3 August 30, 2021 21:11
@ronso0
Copy link
Member Author

ronso0 commented Aug 30, 2021

I read this
image
while #4204 actually went into 2.3

will rebase.

@ronso0 ronso0 removed the library label Aug 30, 2021
@uklotzde uklotzde added this to the 2.3.1 milestone Aug 30, 2021
@ronso0
Copy link
Member Author

ronso0 commented Sep 2, 2021

what's left to do @Be-ing ?

@uklotzde
Copy link
Contributor

uklotzde commented Sep 3, 2021

I will perform a test and then merge. Sorry for the delay.

@uklotzde
Copy link
Contributor

uklotzde commented Sep 3, 2021

I guess this cannot be tested properly without creating an installer and actually installing the broken, current version. I am too lazy spending time on all this. But the changes seem reasonable.

@Be-ing Merge anyway?

@Be-ing
Copy link
Contributor

Be-ing commented Sep 4, 2021

thanks

@Be-ing Be-ing merged commit 722f739 into mixxxdj:2.3 Sep 4, 2021
@ronso0 ronso0 deleted the app-icons branch September 4, 2021 11:33
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.

4 participants