<table><tr><td style="">tcanabrava retitled this revision from "One Search Bar Per Tab" to "New layout for the search bar.".<br />tcanabrava edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-rfdps7sec3kux3g/">(Show Details)</a>
</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/D13174">View Revision</a></tr></table><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Remove</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Now, this review is probabely way bigger than I wanted as it touches too many things to make it work.<br />
<br />
The old searchbar had some issues as it made the text jump around if programs that refreshed the screen where used (like top, htop or any curses application) and after a few round of reviews we got to the conclusion that it was better that the search behaved a bit like chrome: be smaller and on top of the TerminalDisplay instead of being added in a QLayout.<br />
<br />
To simplify positioning and maintenance removed</span> the global search bar and <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">allows each tab<br />
to have</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">now every TerminalDisplay has</span> it's own searc<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">h.</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">hbar that's setup only once,</span> <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">F</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">f</span>or thi<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">s I created a new</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">s there's a new class TerminalWidget that holds the TerminalDisplay and the SearchBar and positions them.<br />
<br />
Tests done:</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">class TerminalWidget - that's</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">- Open multiple tabs and open the search bar, test</span> the <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">combination of</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">search in multiple tabs.</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">the TerminalDisplay and SearchBar, and extracted</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">- Open / close tabs while top is running</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">the uses of TermianlDisplay</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">- Delete the konsole configuration</span> and <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">SearchBar that</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">test the window size</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">where scattered around the code, making</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">- Mark "don't save window size" and test</span> the <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">resulting</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">new window size</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">code less tangled.</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">- Drag & drop the terminal from a tab to detach a view</span><br />
<br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Testing:</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Known issues:<br />
- If the tab detached had search enabled, the search will be disabled and you need to search again</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">created</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">- If you have multiple</span> tab<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">s, created search boxes, searched, closed.</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">s and splits ctrl + shift + f will complain about multiple shortcuts<br />
</span></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D13174">https://phabricator.kde.org/D13174</a></div></div><br /><div><strong>To: </strong>tcanabrava, Konsole, hindenburg<br /><strong>Cc: </strong>ahmadsamir, rizzitello, ngraham, konsole-devel, herrold, maximilianocuria, hindenburg<br /></div>