After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 694475 - Pressing space in the overview causes the search box to flash
Pressing space in the overview causes the search box to flash
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: overview
3.7.x
Other Linux
: Normal normal
: ---
Assigned To: gnome-shell-maint
gnome-shell-maint
Depends on:
Blocks:
 
 
Reported: 2013-02-22 19:05 UTC by Hashem Nasarat
Modified: 2013-02-23 08:43 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
viewSelector: Don't focus search entry on whitespace (1.94 KB, patch)
2013-02-22 20:16 UTC, Florian Müllner
committed Details | Review

Description Hashem Nasarat 2013-02-22 19:05:14 UTC
This seems to be a regression caused by removing the ability to search for a word preceded by spaces.

The flash is distracting though.

A better alternative would be to:
1. not focus the search box when space is pressed and the search box isn't already focused.
2. Keep the search box focused, but don't input a space character (this is what happens if the box is highlighted with nothing in it and enter is pressed).
Comment 1 Florian Müllner 2013-02-22 20:16:24 UTC
Created attachment 237211 [details] [review]
viewSelector: Don't focus search entry on whitespace

We recently started to trim leading and trailing whitespace from
the search string, and not to trigger a search when the resulting
string was empty. However we still allow whitespace to trigger
type-ahead-find, so that the key focus is moved briefly to the
search entry and back to the stage, resulting in a disruptive
flickering of the entry.
Fix this by excluding whitespace from triggering type-ahead-find.


(In reply to comment #0)
> A better alternative would be to:
> 1. not focus the search box when space is pressed and the search box isn't
> already focused.
> 2. Keep the search box focused, but don't input a space character (this is what

(1) made slightly more sense to me.
Comment 2 Cosimo Cecchi 2013-02-22 23:30:09 UTC
Review of attachment 237211 [details] [review]:

Looks good to me.
Comment 3 Florian Müllner 2013-02-23 08:43:01 UTC
Attachment 237211 [details] pushed as fb0cf64 - viewSelector: Don't focus search entry on whitespace