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

LateNight :: follow-up fixes / update #2913

Merged
merged 77 commits into from
Aug 29, 2020
Merged

LateNight :: follow-up fixes / update #2913

merged 77 commits into from
Aug 29, 2020

Conversation

ronso0
Copy link
Member

@ronso0 ronso0 commented Jul 5, 2020

here I collect fixes and improvements for the recent LateNight update.

General

Classic

  • fix crossfader scale, update knob
  • more contrast in fx mixmode icons
  • add indentations in skin settings

PaleMoon

  • unique beatloop icons
  • improve spinny mask with cover art visible
  • fix cropped effect name in WEffectSelectors
  • make BPM tap button more discoverable with hover effect
  • make track context menu more discoverable with hover effect
  • equalize brightness of dark standalone VU meters

Nice to have

@uklotzde
Copy link
Contributor

uklotzde commented Jul 5, 2020

Retarget for 2.3?

@ronso0 ronso0 changed the base branch from master to 2.3 July 5, 2020 14:28
@ronso0
Copy link
Member Author

ronso0 commented Jul 5, 2020

..of course, thx!

@Holzhaus
Copy link
Member

Holzhaus commented Jul 7, 2020

Cool. When you're done with it, can you write blogpost the showcases the massive improvements you made to our new default skin for mixxxdj/website#77?

PaleMoon: unique beatloop icons

can you make then round or do you have something different in mind?

@Holzhaus
Copy link
Member

Holzhaus commented Jul 7, 2020

Another thing: can we make the WCueMenuPopup use the same styles you use for hotcue buttons?

@ronso0
Copy link
Member Author

ronso0 commented Jul 7, 2020

I can try that, sure.
I have difficulties with the current color buttons anyway: with that palette and style it's never WYSIWYG, there always seems to be some overlay that makes the colors more pale.

@ronso0
Copy link
Member Author

ronso0 commented Jul 7, 2020

When you're done with it, can you write blogpost the showcases the massive improvements you made to our new default skin for mixxxdj/website#77?

Yep, I'm already sketching the post.

can you make then round or do you have something different in mind?

I want to get away from the edged look, yes. some slight rounding will be sufficient making the icons blend in with the rest of PaleMoon.

@ronso0 ronso0 added the skins label Jul 8, 2020
@ronso0 ronso0 added this to the 2.3.0 milestone Jul 8, 2020
@JoergAtGithub
Copy link
Member

The new FX meta knob position makes it much easier, to distinguish meta and super knobs!

@ronso0
Copy link
Member Author

ronso0 commented Jul 9, 2020

great! it's the old position, though. toggle & fx selector next to each other was an experiment that failed..

@ronso0
Copy link
Member Author

ronso0 commented Aug 21, 2020

I added the Fx focus bar (underline looks better but wasn't that obvious IMO)
So far I think that works best and I'll test it in tonight's session :)

PaleMoon
image

Classic
image

@ronso0
Copy link
Member Author

ronso0 commented Aug 28, 2020

Besides small tweaks, on of the last changes here was to improve the popups (context menus, Cue menu, color picker).
If there are no objections I suggest we merge this now.
image

Copy link
Member

@Holzhaus Holzhaus left a comment

Choose a reason for hiding this comment

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

LGTM, thank you very much.

@foss-
Copy link
Contributor

foss- commented Aug 29, 2020

Anything blocking a merge here?

@uklotzde
Copy link
Contributor

@Holzhaus I am assuming that you forgot to merge this PR after approval? LGTM, too

@uklotzde uklotzde merged commit 1adb0f7 into mixxxdj:2.3 Aug 29, 2020
@ronso0 ronso0 deleted the lateupdate branch August 29, 2020 14:12
@foss-
Copy link
Contributor

foss- commented Aug 30, 2020

Testing 2.4.0-alpha-pre (build master r7522) and stars in deck view are now cut off and moved one star to the right, meaning only four of the five stars are shown on deck. Mouse behavior in deck for star rating is still missaligned, i.e. mouse is to the left one star.

stars

@ronso0
Copy link
Member Author

ronso0 commented Aug 30, 2020

I have no idea why that simple layout is screwed on macOS.
I created a branch with some debug colors so we can figure out what's wrong. The PR for testing/discussion is ronso0#10

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.

7 participants