<table><tr><td style="">dfaure 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/D7387" rel="noreferrer">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/D7387#137153" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;" rel="noreferrer">D7387#137153</a>, <a href="https://phabricator.kde.org/p/dkurz/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;" rel="noreferrer">@dkurz</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>What happens if the user sets the color back to default? We wouldn't get past line 760, and disableColor() wouldn't be called, so the color would not be saved immediately, right?</p></div>
</blockquote>

<p>If you want to set it back to default, you call disableColor().<br />
myColor.isValid() is about checking for "cancel" in the color dialog, and myColor != defaultColor is about not making any change if the user selected the same color again (no-op). I don't see how assignColor() can ever have the meaning of "setting the color back to default".</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R210 KOrganizer</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D7387" rel="noreferrer">https://phabricator.kde.org/D7387</a></div></div><br /><div><strong>To: </strong>dfaure, smartins<br /><strong>Cc: </strong>dkurz, KDE PIM, dvasin, winterz, vkrause, mlaurent, knauss, dvratil<br /></div>