Allow to set <Variable name="???"/> from color scheme

Bug #1328008 reported by Daniel Schürmann
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
Wishlist
ronso0

Bug Description

This could be useful to replace images or tweak for mixing styles.

This is a follow up from https://bugs.launchpad.net/mixxx/+bug/1327614

Tags: skin
Revision history for this message
jus (jus) wrote :

So color scheme support has a future?
I understood support for color schemes slowly fade out.

tags: added: skin
Revision history for this message
Daniel Schürmann (daschuer) wrote :

I like it: The new Shade skin has some, to implement a dark and light version.
But there is room for improvements.

Revision history for this message
jus (jus) wrote :

Yeah, i noticed you like it :)

Just to make it clear for anyone interested in picking up this bug.
What exactly would you use this <Variable name="???"/> feature for, in connection with color schemes.
Would this allow to make something like lp:1256351

Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

I like the idea of color schemes too but I think we should phase out the bitmap-transformation aspect of them (since we are moving to vector images) and maybe focus on using SVG-variables as added in:
https://github.com/mixxxdj/mixxx/pull/302
to implement them.

It would also be so easy to support conditional CSS based on the selected scheme.

Changed in mixxx:
status: New → Confirmed
RJ Skerry-Ryan (rryan)
Changed in mixxx:
importance: Undecided → Wishlist
RJ Skerry-Ryan (rryan)
Changed in mixxx:
milestone: none → 2.1
Revision history for this message
Be (be.ing) wrote :

There has been no progress on this lately, so I am removing the 2.1.0 milestone.

Changed in mixxx:
milestone: 2.1.0 → none
Revision history for this message
ronso0 (ronso0) wrote :
Changed in mixxx:
assignee: nobody → ronso0 (ronso0)
status: Confirmed → Fix Committed
status: Fix Committed → In Progress
status: In Progress → Fix Committed
Revision history for this message
Hendrik (index82) wrote :

The PR #1988 has no function in Release 2.2.3.
I tested it with Shade and extended the scheme.

...
<Scheme>
<Name>Classic</Name>
   <Filters/>
   <SetVariable name="BgColor">#5EBAFF</SetVariable>
  </Scheme>

The variable is not passed. Or I did something wrong.

BTW: Is there a way to view all keys and variables via CLI or Developertools?

Revision history for this message
Hendrik (index82) wrote :

oops, I wasn't finished.

skins/Shade/skin.xml

<Schemes>
  <Scheme>
    <Name>Classic</Name>
     <Filters/>
     <SetVariable name="BgColor">#5EBAFF</SetVariable>
  </Scheme>
  ...
</Schemes>

skins/Shade/deck.xml

<Visual>
  ...
  <BgColor><Variable name="BgColor"/></BgColor>
  ...
</Visual>

Revision history for this message
ronso0 (ronso0) wrote :

That PR was merged to master so the status is Fix Committed.
As soon as it's in a official release (2.3) it's changed to Fix Released.

You can grab master builds from http://downloads.mixxx.org/builds/master/release/

Changed in mixxx:
milestone: none → 2.3.0
Changed in mixxx:
status: Fix Committed → Fix Released
Revision history for this message
Swiftb0y (swiftb0y) wrote :

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/7502

lock status: Metadata changes locked and limited to project staff
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.