persist="true" required on all instances of a control

Bug #1277201 reported by Hendrik
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
Low
Owen Williams

Bug Description

Saves the values of an skin-created control only if the <ConfigKey persist="true"> stands above the dependent Widget. It concerns the order of code in the skin.xml.

Attached an example-skin with two PushButtons an two visible/hidden WidgetGroups.

The values of the first button are saved to user config.
The values of the second button are not saved.

BTW: The developer-tooltip shows me the first button as push and the second one as toggle-button.

Tags: skin 1.12
Revision history for this message
Hendrik (index82) wrote :
Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

Hi Hendrik -- yes this is indeed order-dependent (and pretty janky). The current workaround is to put persist=true on every use of the control.

RJ Skerry-Ryan (rryan)
Changed in mixxx:
milestone: none → 1.12.0
status: New → Confirmed
RJ Skerry-Ryan (rryan)
Changed in mixxx:
importance: Undecided → Low
Owen Williams (ywwg)
summary: - persist="true" does not save all controls in the user config
+ persist="true" required on all instances of a control
Changed in mixxx:
assignee: nobody → Owen Williams (ywwg)
status: Confirmed → Fix Committed
RJ Skerry-Ryan (rryan)
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/7293

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

Bug attachments

Remote bug watches

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