--===============0453195412975845577== Content-Type: multipart/alternative; boundary="===============3386045351707878529==" --===============3386045351707878529== MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123888/ ----------------------------------------------------------- (Updated July 29, 2015, 8:57 a.m.) Status ------ This change has been marked as submitted. Review request for Plasma, KDE Usability and Vishesh Handa. Changes ------- Submitted with commit 2d2a79390d8bafeda8dc1ce30fd4bc40eafaea57 by Kai Uwe Broulik to branch master. Bugs: 335731 https://bugs.kde.org/show_bug.cgi?id=335731 Repository: plasma-workspace Description ------- This turns KRunner's TextField into an editable ComboBox to provide a history. When a result is invoked, the query string is prepended to the history, query strings are only added once. ComboBox provides letter-by-letter auto completion. Diffs ----- krunner/view.h 45f4d28 krunner/view.cpp 208c88c lookandfeel/contents/runcommand/RunCommand.qml 4c6eb30 Diff: https://git.reviewboard.kde.org/r/123888/diff/ Testing ------- Somehow I have a feeling it doesn't always save the history or nukes it at times. It also has some shortcomings due to ComboBox: 1.) You cannot use the arrow keys to cycle between entries (when the popup's not opened) because arrow keys navigate through results 2.) forceActiveFocus() on the ComboBox will not activate the embedded TextField - when you had opened the popup there's a slight chance the input field won't get focussed I'll prepare a Qt patch for this. 3.) Before Qt 5.4.2 (not sure if my patch ended up in 5.4.1) pressing space in the edit combobox will open the popup, not insert a space (nasty show stopper) 4.) Plasma's edtiable ComboBox looks a bit strange imho 5.) Plasma's editable ComboBox doesn't support clearButtonShown 6.) Plasma's ComboBox has strange bullets and margins in it, that's probably a bug in Plasma Style (need to look what Desktop style does differently from us) 7.) ComboBox doesn't have a cursorPosition, I'll prepare a Qt patch for this. File Attachments ---------------- History popup https://git.reviewboard.kde.org/media/uploaded/files/2015/05/23/7ad7e5eb-4874-4f9f-9796-738fa2ac9ed5__krunnerhistory.png Auto completion https://git.reviewboard.kde.org/media/uploaded/files/2015/05/23/18714844-ef28-4cdd-af00-e6685caece9b__krunnerautocompletion.png History popup (new) https://git.reviewboard.kde.org/media/uploaded/files/2015/07/28/ad685fde-7d37-48c6-8b01-ac938a450618__krunnerhistory2.png Thanks, Kai Uwe Broulik --===============3386045351707878529== MIME-Version: 1.0 Content-Type: text/html; charset="utf-8" Content-Transfer-Encoding: 7bit
This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/123888/

This change has been marked as submitted.

Review request for Plasma, KDE Usability and Vishesh Handa.
By Kai Uwe Broulik.

Updated July 29, 2015, 8:57 a.m.

Changes

Submitted with commit 2d2a79390d8bafeda8dc1ce30fd4bc40eafaea57 by Kai Uwe Broulik to branch master.
Bugs: 335731
Repository: plasma-workspace

Description 

This turns KRunner's TextField into an editable ComboBox to provide a history.


When a result is invoked, the query string is prepended to the history, query strings are only added once. ComboBox provides letter-by-letter auto completion.

Testing 

Somehow I have a feeling it doesn't always save the history or nukes it at times. It also has some shortcomings due to ComboBox:


1.) You cannot use the arrow keys to cycle between entries (when the popup's not opened) because arrow keys navigate through results
2.) forceActiveFocus() on the ComboBox will not activate the embedded TextField - when you had opened the popup there's a slight chance the input field won't get focussed I'll prepare a Qt patch for this.
3.) Before Qt 5.4.2 (not sure if my patch ended up in 5.4.1) pressing space in the edit combobox will open the popup, not insert a space (nasty show stopper)
4.) Plasma's edtiable ComboBox looks a bit strange imho
5.) Plasma's editable ComboBox doesn't support clearButtonShown
6.) Plasma's ComboBox has strange bullets and margins in it, that's probably a bug in Plasma Style (need to look what Desktop style does differently from us)
7.) ComboBox doesn't have a cursorPosition, I'll prepare a Qt patch for this.

Diffs

  • krunner/view.h (45f4d28)
  • krunner/view.cpp (208c88c)
  • lookandfeel/contents/runcommand/RunCommand.qml (4c6eb30)

View Diff

File Attachments

  • History popup
  • Auto completion
  • History popup (new)
    		•
    --===============3386045351707878529==-- --===============0453195412975845577== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KUGxhc21hLWRl dmVsIG1haWxpbmcgbGlzdApQbGFzbWEtZGV2ZWxAa2RlLm9yZwpodHRwczovL21haWwua2RlLm9y Zy9tYWlsbWFuL2xpc3RpbmZvL3BsYXNtYS1kZXZlbAo= --===============0453195412975845577==--