Skip to content

Conversation

@ferranpujolcamins
Copy link
Contributor

New mixer layout with better usable buttons!

esbrandt and others added 30 commits April 29, 2015 13:38
Conflicts:
	res/skins/Deere/deck_overview_row.xml
…toggling the key had no effect/would not update state on change
* Add menu svg icon for skin settings toggle.
* Add variable for custom <Tooltip> to 2state_button template, because a skin setting is unique to Deere (no <existing <TooltipId> available>). Tooltip won't be i18n able though. Move to tooltips.cpp?
…t. Now follows the left->down, right->up convention.
…he icons instead of unicode characters in the loop section.
…ling cue/master mix

After talking to daschuer on lp:1458213, I realize that headphone gain
should not regularly be manipulated while DJing. So, remove that from
the mapping and make the knobs control filters by default. With shift,
the right knob still controls cue/master mix and turning it all the way
to the right toggles split cue mode.
makes a convenient way to quickly seek across large sections of tracks
Before, volume faders could not be moved very quickly
be a little more explicit
@ferranpujolcamins
Copy link
Contributor Author

In the new version the text under the fx knobs has gotten bigger and is interfering with the knobs even more than it already was

I think this text size is what we want at the end. The good solution for me is to fix Bug #1395245. However this means new code and someone will have to review it. :)

The alternative is to revert the fx knobs text size back to the small size. It's ugly, but it's a fast hack.

Which approach shall I take?

@ywwg
Copy link
Member

ywwg commented Aug 21, 2015

no, it is too late to come up with a design for adding short text to all the effects. I think if we make the effect knobs a little smaller and make the box just a few pixels taller we can go back to the old font size and make sure the text is not touching the knobs

@ferranpujolcamins
Copy link
Contributor Author

Ok, here you have a first approach.

I've also changed the position of the super and d/w knobs. Before, they were on the first fx row, taking a lot of space from it. Now they are vertical and in a column next to the three fx rows, so the effects are now the same size.

@ywwg
Copy link
Member

ywwg commented Aug 21, 2015

still seeing this:
knobs

@ferranpujolcamins
Copy link
Contributor Author

What about this?

captura de pantalla de 2015-08-21 14 48 06

@ywwg
Copy link
Member

ywwg commented Aug 21, 2015

much better

@ywwg
Copy link
Member

ywwg commented Aug 21, 2015

heh, try shrinking the skin down to minimal width -- bad things start happening. Change the minimum size to 1024, 700 and I think we're ok

@ywwg
Copy link
Member

ywwg commented Aug 21, 2015

btw here is the selection color issue I mentioned. This probably happens because the skin is not defining specific values for selection colors and the colors are being picked up from the default theme, which may be different on different computers. Check out how latenight themes the library for guidance.
colors

@ferranpujolcamins
Copy link
Contributor Author

Because how I put the knobs of the minimized effects units, it needs to be 1170x700. This can be really optimized but involves a lot of work and re-thinking, lets leave it like this for 1.12.

@ywwg
Copy link
Member

ywwg commented Aug 22, 2015

Is there any way to get down to 1024? We've had a lot of complaints about latenight taking up too much space on small screens so if we can save those 50 pixels somehow it would be a big win.

@ferranpujolcamins
Copy link
Contributor Author

Ok, it was not that hard :)

Now let me check this issue with the library colours.

By the way, my priority right now is to push 1.12, but unless you keep pointing me to more skin issues, I'll put my time to other parts of Mixxx. Is there other 1.12 blockers nobody is taking care of?

@ferranpujolcamins
Copy link
Contributor Author

I have no idea about what's happening with your colours. I see on the image that the selection colour is a gradient, but there's no gradient specified on the QTableView section of style.qss. Maybe you have the skin files somehow messed up?

Or maybe your desktop environment is overriding the colours. Is that possible?

@ywwg
Copy link
Member

ywwg commented Aug 22, 2015

First, that selection color is the unfocused selection color. As I described, I believe that gradient is part of my system's theme. And since the skin doesn't define that color, the system theme version shows through.

Like I said, look at latenight, I even figured out how to style the scrollbar correctly! ferranpujolcamins#1

@ferranpujolcamins
Copy link
Contributor Author

I've just made the scroll bars blue :)

@ywwg
Copy link
Member

ywwg commented Aug 22, 2015

cool, if you can just fix the conflicts that github is reporting I'll merge it in

Conflicts:
	res/skins/Deere/skin.xml
@ferranpujolcamins
Copy link
Contributor Author

Done!

ywwg added a commit that referenced this pull request Aug 22, 2015
@ywwg ywwg merged commit dede93b into mixxxdj:master Aug 22, 2015
@ywwg
Copy link
Member

ywwg commented Aug 29, 2015

didn't realize this was targetted to master. manually merged to 1.12

@ferranpujolcamins
Copy link
Contributor Author

Ooops, my fault. Thank you.

2015-08-29 2:26 GMT+02:00 Owen Williams notifications@github.com:

didn't realize this was targetted to master. manually merged to 1.12


Reply to this email directly or view it on GitHub
#682 (comment).

@ferranpujolcamins ferranpujolcamins deleted the 1.12-Deere-New-Mixer-Buttons branch March 28, 2016 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants