<table><tr><td style="">asemke added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D16839">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D16839#360121" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D16839#360121</a>, <a href="https://phabricator.kde.org/p/mikhailru/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@mikhailru</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p><a href="https://phabricator.kde.org/p/asemke/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@asemke</a> , could you please describe the steps to reproduce wrong behavior with default dark worksheet theme? i tried setting a dark theme ("Dark") and axis label color seemed OK to me, screenshots attached (before and after patch).</p></div>
</blockquote>

<p><a href="https://phabricator.kde.org/p/mikhailru/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@mikhailru</a> sorry, I was not very precise in my last comment. The problem that I referred to can be seen on these screenshots:<br />
<a href="https://phabricator.kde.org/F6428869" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6428869: textlabel_white_desktop_white_worksheet.png</a> <a href="https://phabricator.kde.org/F6428871" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6428871: textlabel_white_desktop_black_worksheet.png</a> <a href="https://phabricator.kde.org/F6428873" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6428873: textlabel_black_desktop_white_worksheet.png</a><a href="https://phabricator.kde.org/F6428875" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6428875: textlabel_black_desktop_white_worksheet_theme_loaded.png</a></p>

<p>Observe the colors shown in the textedit in the properties dock widget. Do you also have this? This doesn't look nice and this is what I meant with "We need to re-think how we treat QTextEdit::toHtml() in LabelWidget.cpp and how we plot this rich-text formatted string in TextLabel.cpp via QPainter::drawStaticText()".</p>

<p>On the last screenshot I started with a "white desktop/white (default) worksheet" and applied the theme "BlackOnWhite" on the worksheet - only after this the foreground color of the label is set correctly.</p>

<p>You patch doesn't fix the problem for me, for white worksheet and plot the axis title is still hardly visible and I don't really understand why this should help. All properties of the text like forground and background colors are read from the html text in QTextEdit and set to QStaticText which is then painted in TextLabelPrivate::paint() via QPainter::drawStaticText(). A global color for the pen doesn't make much sense since the different parts of the label string can have different colors - this is what is encoded in the html code that is painted via the QStaticText.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R262 LabPlot</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D16839">https://phabricator.kde.org/D16839</a></div></div><br /><div><strong>To: </strong>mikhailru, LabPlot<br /><strong>Cc: </strong>asemke, cfeck, kde-edu, narvaez, apol<br /></div>