Re: [LMMS-devel] Proposal: Fitting UI elements to be used in Qt Designer

[Vesa <di...@nb...>]

On 04/06/2014 11:12 PM, Tobias Doerffel wrote:
> 2014-04-05 13:44 GMT+02:00 Vesa <di...@nb...>:
>>>     QColor lineColor() const;
>>>     void setlineColor( const QColor & _c );
>>>     ...
>>>     QColor * m_lineColor; //!< unused yet
>>>     ...
>>>     Q_PROPERTY(QColor lineColor READ lineColor WRITE setlineColor)
>>>
>>> So this is the same as for outerColor. Is this what you need?
>> Yeah. That works.
> No, m_lineColor shouldn't be a pointer but a regular class instance
> instead. (Besides this: coding style! -> setLineColor( const QColor& c
> );
>
> The rest looks good, go ahead! :-)

Why not a pointer? All the other QColor qproperties are stored as
pointers to QColor...

That's how it was done for the outerColor property of the knob class, so
I also used the same way for the track properties...

Should these be changed?