<table><tr><td style="">nibags 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/D20958">View Revision</a></tr></table><br /><div><div><p>Thanks for the comments, in a while I will update the diff with the suggestions/corrections.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>The only thing I would like to have is an auto generated "most often used" list. That was also suggested somewhere. This way you could quickly change between preferred styles instead to search for them.</p></blockquote>

<p>I could add a "Recent" section to the beginning of the menu, it would be useful.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>When the menu is opened, clear any text in the search field left over from a prior search</p></blockquote>

<p>Also I could keep all the search text as selected, to delete it with a single button.<br />
I hope more comments about it, I'm not sure which will be the best option: S</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Single-clicking on an item in the list should select it and close the menu; double-click doesn't gain us anything here</p></blockquote>

<p>To me the double-click works correctly. It may also be necessary to add a special case for double-click to avoid problems.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>.cp does not show c++, same for cp</p></blockquote>

<p>I checked the XML file of "C++", and <tt style="background: #ebebeb; font-size: 13px;">*.cp</tt> isn't an extension associated with that language, so it isn't displayed.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>These ctrl-return is handy! Would be good to have this on right or middle too or in conjunction with some modifier key alt/ctrl/shift</p></blockquote>

<p>Good idea! I will also add shift and alt and combinations.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>The blue frame (in dark theme) looks compared to the other plain menus slightly strange, any possibility to remove this?</p></blockquote>

<p>You can erase the edge, but I'm not sure if the brightness. I will try, although I don't know if it will be a good idea, since that brightness indicates that the widget is in focus.<br />
As <a href="https://phabricator.kde.org/p/ngraham/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@ngraham</a> says, it might be better to reduce the brightness in the Breeze Dark theme.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>Is your new great menu easy usable for other parts? Like the encoding?</p></blockquote>

<p>It could be done. I would have to separate the menu into two classes that are inherited, one with the menu only, so that it can be implemented elsewhere.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>As first feedback: much better than current state :)</p></blockquote>

<p>;-D</p></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/D20958">https://phabricator.kde.org/D20958</a></div></div><br /><div><strong>To: </strong>nibags, KTextEditor, Kate, VDG<br /><strong>Cc: </strong>cullmann, ngraham, loh.tar, kwrite-devel, kde-frameworks-devel, domson, michaelh, bruns, demsking, sars, dhaumann<br /></div>