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

Revert to raster level meters in LateNight and Tango to fix severe GUI lag on macOS #1573

Merged
merged 3 commits into from
Apr 1, 2018

Conversation

Be-ing
Copy link
Contributor

@Be-ing Be-ing commented Apr 1, 2018

Rendering SVGs for the meters causes the whole GUI to become severely laggy on older Macs:
https://bugs.launchpad.net/mixxx/+bug/1759433

Be-ing added 2 commits March 31, 2018 20:03
Rendering SVG level meters caused the whole GUI to become severely
unresponsive on older Macs:
https://bugs.launchpad.net/mixxx/+bug/1759433
Rendering SVGs for the level meters makes the GUI severely laggy
on old Macs:
https://bugs.launchpad.net/mixxx/+bug/1759433
@Be-ing Be-ing changed the base branch from master to 2.1 April 1, 2018 01:54
@Be-ing
Copy link
Contributor Author

Be-ing commented Apr 1, 2018

I just tested with 250% scaling on my 3840 x 2160 screen with GNU/Linux and the meters still look good to me. I don't notice any blurriness.

@Be-ing Be-ing added this to the 2.1.0 milestone Apr 1, 2018
@Be-ing Be-ing changed the title Revert to raster level meters in LateNight and Tango Revert to raster level meters in LateNight and Tango to fix severe GUI lag on macOS Apr 1, 2018
<PathVu>graphics/vumeter_floating_over.svg</PathVu>
<PathBack>graphics/vumeter_floating.svg</PathBack>
<PathVu>skin:/graphics/vumeter_floating_over.png</PathVu>
<PathBack>skin:/graphics/vumeter_floating.</PathBack>
Copy link
Member

Choose a reason for hiding this comment

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

PNG extension is missing

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch, thanks

@ronso0
Copy link
Member

ronso0 commented Apr 1, 2018

... the rest LGTM

@ronso0
Copy link
Member

ronso0 commented Apr 1, 2018

I just noticed the Aux mute/play button icons in LateNight is reverted.
@Be-ing could you fix this here as well? Just swap the <Number>s at https://github.com/mixxxdj/mixxx/blob/master/res/skins/LateNight/aux_unit.xml#L42

@Be-ing
Copy link
Contributor Author

Be-ing commented Apr 1, 2018

FYI the SVG files are still in the Git repository. I just rendered them to PNG.

@Be-ing
Copy link
Contributor Author

Be-ing commented Apr 1, 2018

I double checked this on the Macbook Air that I am borrowing and it works. 🎉

@Be-ing Be-ing merged commit 7a1750d into mixxxdj:2.1 Apr 1, 2018
@Be-ing
Copy link
Contributor Author

Be-ing commented Apr 1, 2018

I just noticed the Aux mute/play button icons in LateNight is reverted.

Please open another PR for that.

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.

2 participants