<table><tr><td style="">acrouthamel edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-c7eglzpx6iqf64f/">(Show Details)</a><br />acrouthamel edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-yd5yz7vyxfbhikh/">(Show Details)</a><br />acrouthamel added reviewers: Kate, KTextEditor, Framework: Syntax Highlighting.<br />acrouthamel added a project: KTextEditor.
</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/D15337">View Revision</a></tr></table><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;">Solarized Light contrast was low. Upon investigation it was determined<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"> much of the color scheme did not follow Solarized base 16 colors or rules. Color scheme for both Dark and Light variants were adjusted to match the latest Solarized color scheme release palette and [[ https://ethanschoonover.com/solarized/#usage-development | color interaction rules ]]. Color schemes were also matched closely to the [[ https://github.com/altercation/vim-colors-solarized/blob/master/colors/solarized.vim | Vim color scheme ]] created by the original Solarized author.<br />
<br />
This is my first "frameworks" patch, so please let me know if I missed anything.<br />
<br />
BUG: 382075<br />
<br />
Solarized Light Before:<br />
{F6241106}<br />
Solarized Light After:</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">much of the color scheme did not follow Solarized base 16 colors or</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">{F6241107}</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">rules. Color scheme for both</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Solarized</span> Dark <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">and Light variants were adjusted to</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Before:</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">match the latest Solarized color scheme release palette and color</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">{F6241108}</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">interaction rules. Color schemes were also matched closely to the</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Solarized Dark After:</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Vim color scheme created by the original Solarized author.</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">{F6241109}</span></div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;">Open Kate with new rc files and inspect the Color settings<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">. Open a CPP file to review syntax highlighting</span>.</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R39 KTextEditor</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D15337">https://phabricator.kde.org/D15337</a></div></div><br /><div><strong>To: </strong>acrouthamel, Kate, KTextEditor, Framework: Syntax Highlighting<br /><strong>Cc: </strong>kwrite-devel, kde-frameworks-devel, michaelh, kevinapavew, ngraham, bruns, demsking, head7, cullmann, kfunk, sars, dhaumann<br /></div>