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

Tango :: polish Play buttons & FX unit expand toggle #1531

Merged
merged 12 commits into from Mar 8, 2018

Conversation

ronso0
Copy link
Member

@ronso0 ronso0 commented Mar 3, 2018

  • Play/Pause icons appear blurred when using 'image' porperty in qss, although svg height matches the buttons' height > distinct 1:1 sized icons for every instance, also I removed separate icon definitions for 'pressed'/'unpressed' state where unnecessary
  • minimized FX unit would flicker when changing FX focus > fix size for that toggle's container
  • emphasize Pfl button in FX units & sampler by adding a border to it
  • repair right-click function of Key_down & rate_down buttons (fixes lp:1752671)

ronso0 added 2 commits March 3, 2018 13:45
… definition (except 0-state buttons like Eject), make Play/Pause icons match the button size(s)

Also, in style.qss I recently split 'background' property into 'background-color' and 'image' to make the button icons scale when scaling the skin.
Drawback was that some icons appeared blurred afterwards, although at least their height matched the buttons.
For affected buttons (basically all Play buttons) I switched to inline definition in xml ('state_X_pressed' etc.) and created icons
for every instance (deck, sampler, previewdeck). There's no hover effect for icons anymore, but the icons look great in every situation.
…prevent flickering when altering effect focus
@ronso0 ronso0 mentioned this pull request Mar 3, 2018
@daschuer
Copy link
Member

daschuer commented Mar 3, 2018

Play/Pause icons appear blurred when using 'image' porperty in qss, although svg height matches the buttons' height > distinct 1:1 sized icons for every instance, also I removed separate icon definitions for 'pressed'/'unpressed' state where unnecessary

Is that a bug?

Is there anything to test here? This skin still looks good

@ronso0
Copy link
Member Author

ronso0 commented Mar 3, 2018

Is that a bug?

I can't tell...maybe border is subtracted from button size and the icons get scaled down, maybe it's just a horizontal offset, idk yet. Here it is fixed but I'll investigate this sometime.

Testing is welcome, of course. Maybe something slipped through..

@ronso0
Copy link
Member Author

ronso0 commented Mar 3, 2018

don't merge yet, I'm working on https://bugs.launchpad.net/mixxx/+bug/1753112

@ronso0
Copy link
Member Author

ronso0 commented Mar 3, 2018

I'm done, please test.
Especially Spinny and CoverArt, scaled to 200% and more

@ronso0
Copy link
Member Author

ronso0 commented Mar 6, 2018

While I was vectorizing the remaining buttons and VU meters for LateNight, I did this for Tango, as well.

@Be-ing Be-ing added the skins label Mar 8, 2018
@Be-ing Be-ing added this to the 2.1.0 milestone Mar 8, 2018
@Be-ing
Copy link
Contributor

Be-ing commented Mar 8, 2018

Everything looks crisp at 2.5x scale on my 3840 x 2160 screen. I am appreciating the design of this skin more now that I have a 4 deck controller. Great job 👍

@Be-ing Be-ing merged commit 3ed7563 into mixxxdj:2.1 Mar 8, 2018
@ronso0
Copy link
Member Author

ronso0 commented Mar 8, 2018

Great, thanks!
Do you get along well with how the waveforms are stacked with 4 decks? I'm still unsure what to do about B. of lp:1750902 and if my suggestion to change that order when toggling symmetric overviews makes sense.

@Be-ing
Copy link
Contributor

Be-ing commented Mar 8, 2018

I think the best waveform layout depends on the user's controller. Refer to the discussion I pinged you in on Zulip.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants