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 706300 - Do not use GtkComboBox for search.
Do not use GtkComboBox for search.
Status: RESOLVED FIXED
Product: gnome-maps
Classification: Applications
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gnome-maps-maint
gnome-maps-maint
Depends on:
Blocks:
 
 
Reported: 2013-08-19 12:14 UTC by Jonas Danielsson
Modified: 2013-08-20 12:18 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Add search popup and use it to show search results (11.72 KB, patch)
2013-08-20 11:01 UTC, Jonas Danielsson
none Details | Review
Add search popup and use it to show search results V2 (11.73 KB, patch)
2013-08-20 11:44 UTC, Jonas Danielsson
committed Details | Review

Description Jonas Danielsson 2013-08-19 12:14:27 UTC
Zeeshan Ali had some comments when adding the current way of searching:

"[...] Don't think we want a combobox here. For searches, we still want to use a
search entry and search results/suggestions shown appear in an overlay popup as
shown in mockup:
 
https://wiki.gnome.org/Design/Apps/Maps?action=AttachFile&do=get&target=Maps.png
 
The main issue I see the combobox' drop down, using that for 'execute search'
seems very odd from UX POV."
Comment 1 Jonas Danielsson 2013-08-20 11:01:00 UTC
Created attachment 252370 [details] [review]
Add search popup and use it to show search results

This patch tries to follow the mockups of displaying the search results closer.
Comment 2 Andreas Nilsson 2013-08-20 11:25:18 UTC
I have never before seen such a ui element anywhere else, so I'm not sure how it's supposed to work exactly.

I agree that it the search is a bit odd today, mainly because we can't autocomplete correctly due to technical limitations, but apart from that the current combobox isn't much different from Firefox awesomebar or the combobox in Google Maps (web version).
Comment 3 Jonas Danielsson 2013-08-20 11:44:12 UTC
Created attachment 252381 [details] [review]
Add search popup and use it to show search results V2

Bugfix events.get_coords returns [bool, x, y] not [x, y] which made it all _almost_ work. :)
Comment 4 Zeeshan Ali 2013-08-20 12:08:22 UTC
Review of attachment 252381 [details] [review]:

Really good stuff man! That was very quick. I'll spare you the nitpicks to get this in for the release. :)
Comment 5 Zeeshan Ali 2013-08-20 12:10:27 UTC
Pushing this now as its already a big improvement IMHO and more closer to mockup. Also to get this in release I'm going to roll out soon.
Comment 6 Andreas Nilsson 2013-08-20 12:18:58 UTC
uh, ok.