<table><tr><td style="">mglb created this revision.<br />mglb added reviewers: Konsole, VDG.<br />mglb added a project: Konsole.<br />mglb requested review of this revision.
</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/D17244">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><h3 class="remarkup-header">Overall changes</h3>
<ul class="remarkup-list">
<li class="remarkup-list-item">Use a dialog with category buttons on the left, which is used in most KDE applications</li>
<li class="remarkup-list-item">Apply KDE HIG as much as possible</li>
<li class="remarkup-list-item">Align layout columns in multiple group boxes</li>
<li class="remarkup-list-item">Move some settings to another groups</li>
</ul>
<h3 class="remarkup-header">General page</h3>
<p><a href="https://phabricator.kde.org/F6447280" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F6447280: general.png</a></p>
<p>Profile name and icon, and settings related to session/application<br />
initialization.</p>
<ul class="remarkup-list">
<li class="remarkup-list-item">Move "Show hint for terminal size after resizing" to Appearance page</li>
<li class="remarkup-list-item">Move "Dim the colors when the window loses focus" to Appearance page</li>
</ul>
<h3 class="remarkup-header">Tabs page, rename tab dialog</h3>
<p><a href="https://phabricator.kde.org/F6447281" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F6447281: tabs.png</a></p>
<p>Tabs settings</p>
<ul class="remarkup-list">
<li class="remarkup-list-item">Only minor UI changes</li>
</ul>
<h3 class="remarkup-header">Appearance page</h3>
<p><a href="https://phabricator.kde.org/F6447282" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F6447282: appearance1.png</a> <a href="https://phabricator.kde.org/F6447283" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F6447283: appearance2.png</a> <a href="https://phabricator.kde.org/F6447284" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F6447284: appearance3.png</a><br />
<a href="https://phabricator.kde.org/F6447290" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F6447290: fontselect.png</a></p>
<p>Settings related to basic appearance.</p>
<ul class="remarkup-list">
<li class="remarkup-list-item">Add additional tabs<ul class="remarkup-list">
<li class="remarkup-list-item">Cursor - cursor settings from Advanced page</li>
<li class="remarkup-list-item">Miscellaneous<ul class="remarkup-list">
<li class="remarkup-list-item">Add "Line spacing" from Advanced page</li>
<li class="remarkup-list-item">Add previously missing terminal margins and terminal center settings (4 years old config-only feature)</li>
<li class="remarkup-list-item">Add "Show hint for terminal size after resizing" from General page</li>
<li class="remarkup-list-item">Add "Dim the colors when the window loses focus" from General page</li>
</ul></li>
</ul></li>
<li class="remarkup-list-item">Use customized font selection dialog<ul class="remarkup-list">
<li class="remarkup-list-item">Show all printable ASCII characters and look-alike character sets as a preview</li>
<li class="remarkup-list-item">Live preview for changes in the dialog</li>
</ul></li>
<li class="remarkup-list-item">Move "Show all fonts" to the font selection dialog</li>
<li class="remarkup-list-item">Remove "text size" (it is replaced with live preview in the font dialog)</li>
<li class="remarkup-list-item">Add live preview for cursor settings</li>
<li class="remarkup-list-item">Add live preview for "Line spacing"</li>
</ul>
<h3 class="remarkup-header">Scrolling page, history size dialog</h3>
<p><a href="https://phabricator.kde.org/F6447285" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F6447285: scrolling.png</a></p>
<p>Settings related to scrolling and history.</p>
<ul class="remarkup-list">
<li class="remarkup-list-item">Replace popping-in warning frames in "Scrollback" group with warning buttons which show floating warning after click. The controls does not change position anymore when switching the scrollback options. Applies also to history size dialog.</li>
<li class="remarkup-list-item">Replace scrollbar "hide"/"show on left side"/"show on right side" options with "visible" checkbox and "show on left side"/"show on right side" options enabled after checking the checkbox.</li>
</ul>
<h3 class="remarkup-header">Keyboard page</h3>
<p><a href="https://phabricator.kde.org/F6447286" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F6447286: keyboard.png</a></p>
<ul class="remarkup-list">
<li class="remarkup-list-item">Removed redundant group box</li>
</ul>
<h3 class="remarkup-header">Mouse page</h3>
<p><a href="https://phabricator.kde.org/F6447287" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F6447287: mouse1.png</a><br />
<a href="https://phabricator.kde.org/F6447288" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F6447288: mouse2.png</a></p>
<ul class="remarkup-list">
<li class="remarkup-list-item">Shorten "Characters considered part of a word..." label</li>
<li class="remarkup-list-item">Replace "triple-click selects" drop-down with option buttons</li>
<li class="remarkup-list-item">Split settings to "Text interaction" and "Miscellaneous" tabs</li>
</ul>
<h3 class="remarkup-header">Advanced page</h3>
<p><a href="https://phabricator.kde.org/F6447289" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F6447289: advanced.png</a></p>
<p>More advanced settings or settings regular user don't care about.</p>
<ul class="remarkup-list">
<li class="remarkup-list-item">Replace "Show URL hints when these keys are pressed" checkboxes with toggle buttons which are easier to associate visually with hardware keys</li>
<li class="remarkup-list-item">Move "Line spacing" to Appearance tab</li>
<li class="remarkup-list-item">Move cursor settings to Appearance tab</li>
<li class="remarkup-list-item">Show "Default character encoding" value directly on drop-down button</li>
</ul>
<h3 class="remarkup-header">Preview for: breeze (dark colors), Oxygen, QtCurve</h3>
<p><a href="https://phabricator.kde.org/F6447339" style="background-color: #e7e7e7;
border-color: #e7e7e7;
border-radius: 3px;
padding: 0 4px;
font-weight: bold;
color: black;text-decoration: none;">F6447339: editprofiledialog-other-themes.png</a></p>
<p>TESTING</p></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Check visually with light/dark color scheme, Breeze, Fusion, Oxygen, QtCurve widget styles, normal/large font, QT_SCALE_FACTOR set to 1 and 2</li>
<li class="remarkup-list-item">Change every possible control to check UI logic</li>
<li class="remarkup-list-item">Change as much settings as possible and see if they are applied</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br /><div><strong>BRANCH</strong><div><div>arc/Edit-Profile-Dialog-UI-redesign (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17244">https://phabricator.kde.org/D17244</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/CMakeLists.txt<br />
src/CharacterColor.h<br />
src/EditProfileAdvancedPage.ui<br />
src/EditProfileAppearancePage.ui<br />
src/EditProfileDialog.cpp<br />
src/EditProfileDialog.h<br />
src/EditProfileDialog.ui<br />
src/EditProfileGeneralPage.ui<br />
src/EditProfileKeyboardPage.ui<br />
src/EditProfileMousePage.ui<br />
src/EditProfileScrollingPage.ui<br />
src/EditProfileTabsPage.ui<br />
src/FontDialog.cpp<br />
src/FontDialog.h<br />
src/HistorySizeDialog.cpp<br />
src/HistorySizeDialog.h<br />
src/HistorySizeDialog.ui<br />
src/HistorySizeWidget.cpp<br />
src/HistorySizeWidget.h<br />
src/HistorySizeWidget.ui<br />
src/RenameTabWidget.ui</div></div></div><br /><div><strong>To: </strong>mglb, Konsole, VDG<br /><strong>Cc: </strong>konsole-devel, ngraham, maximilianocuria, hindenburg<br /></div>