D16839: Force black pen when painting unstyled TextLabel

[Alexander Semke]

asemke added a comment.


  In D16839#370476 <https://phabricator.kde.org/D16839#370476>, @asemke wrote:
  
  > In D16839#365584 <https://phabricator.kde.org/D16839#365584>, @mikhailru wrote:
  >
  > > This text without formatting is painted with the default color, taken from the Qt theme. And this is the case which my original patch addressed.
  >
  >
  > Ok. This makes sense. But let's use maybe the color QApplication::palette().color(QPalette::Active,QPalette::Text ) ) instead of hard-coded Qt::black.
  
  
  Having thought more about this, for the unformatted text a better idea would to use Qt::black for the foreground color in case no theme is active and to use the foreground color from the theme, otherwise. Furthermore, for unformatted texts we shouldn't add any formatting when we apply a theme. At the moment we convert plain text to a html-formatted string which causes problems and is not required for plain text. It should be enough if we only set the proper pen color.

REPOSITORY
  R262 LabPlot

REVISION DETAIL
  https://phabricator.kde.org/D16839

To: mikhailru, #labplot
Cc: asemke, cfeck, kde-edu, narvaez, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20181203/dec3581a/attachment.html>