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.

Sign up for GitHub

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

Jump to bottom

Shade VU-Meter fix #11598

Merged
merged 4 commits into from
May 31, 2023
Merged

Shade VU-Meter fix #11598

merged 4 commits into from
May 31, 2023

Conversation

daschuer
Copy link
Member

See commits.

@daschuer daschuer added this to the 2.3.6 milestone May 29, 2023
@github-actions github-actions bot added the skins label May 29, 2023
@daschuer daschuer mentioned this pull request May 30, 2023
Merged
ronso0
ronso0 approved these changes May 30, 2023
View reviewed changes
Copy link
Member

@ronso0 ronso0 left a comment

Choose a reason for hiding this comment

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

LGTM, just the two tooltipIds should be replaced

res/skins/Shade/preview_deck.xml Outdated Show resolved Hide resolved
@daschuer @ronso0
Fix previev VU-Meter tooltips
8e56db5 
Co-authored-by: ronso0 <ronso0@mixxx.org>
@daschuer
Copy link
Member Author

Done

@ronso0
Copy link
Member

ronso0 commented May 31, 2023

Thank you!

@ronso0 ronso0 merged commit 88a6708 into mixxxdj:2.3 May 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ronso0 ronso0 ronso0 approved these changes

Assignees
No one assigned
Labels
skins
Projects
None yet
Milestone
2.3.6
Development

Successfully merging this pull request may close these issues.
2 participants
@daschuer @ronso0