Shifted text in skins because of hardcoded font properties

Marking fixed, Jus correct me if I'm wrong.

Back from the death, bug still stands in current trunk.

Hardcoded alignments override custom stylesheets. It is not possible to align fonts right/center with stylesheets.
The hardcoded font size is somehow no problem with the stylesheet.

My vote for scrapping alignment & fontsize if there is not a good reason to keep.

Affected files:
wnumber.cpp
wlabel.cpp

Leaving Alignment and FontSize in to old ancient skins that still have them, but making FontSize not set the size unless it was present in the skin. This should allow you to leave out FontSize and Alignment and the stylesheet should work.

Hmm, RJ this is marked as fixed in trunk. Could you please check again?
Either the fix is incomplete or not working properly. Looks like there are still hardcoded alignments left, see lp:661493.

Looks like this is still broken. Judging from the Qt stylesheet reference, the reason text-align doesn't work is that it isn't supported on anything but QProgressBar and QPushButton, which isn't used in any of Mixxx's widgets.

http://doc.qt.nokia.com/latest/stylesheet-examples.html
"This property is currently supported only by QPushButton and QProgressBar."

As far as I can tell, there is no way at all to customize the alignment of text in a QLabel via Qt style sheets.

A potential workaround is to use fixed padding amounts -- e.g. padding-left: 10px; to make the text 10px off from the left.

Better late then never, found this interesting bit in http://qt-project.org/forums/viewreply/95684/
"
You can use style sheets not only to set CSS properties, but also to set QObject properties, given that you prefix them with qproperty- (so for example to set the QLabel::alignment property you will have to use qproperty-alignment) so the engine knows you are referring to a QObject property, not a CSS property.
"
E.g. In the skin.xml:

<NumberBpm>
  <ObjectName>TextBpm</ObjectName>
  <Channel>1</Channel>
  <Size>120f,min</Size>
  <Connection>
    <ConfigKey>[Channel1],visual_bpm</ConfigKey>
  </Connection>
</NumberBpm>

In the external skin.qss

#TextBpm {
  qproperty-alignment: AlignCenter;
}

Also see http://qt-project.org/doc/qt-4.8/qlabel.html#alignment-prop
Tested with latest Github master.

So at least for 1.12+ we can override the hardcoded values.
I think it is save to drop them at all from /src/widget/wlabel.cpp & /src/widget/wnumber.cpp

Cool stuff. Looking over WLabel and WNumber -- we don't have any hard-coded alignments. We have code that is only triggered with an <Alignment> tag so you can just not provide the <Alignment> tag. Same goes for <FontSize> -- that's only triggered if that tag is present. Are you seeing hard-coded values that take effect without any special tags?

Marking fix committed because none of our label widgets have hard-coded font properties anymore! They only trigger on <Alignment> and <FontSize>.

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