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 724028 - Icons for recent/favourite in entrycompletion
Icons for recent/favourite in entrycompletion
Status: RESOLVED FIXED
Product: gnome-maps
Classification: Applications
Component: general
git master
Other Linux
: Normal normal
: ---
Assigned To: gnome-maps-maint
gnome-maps-maint
Depends on: 723895
Blocks:
 
 
Reported: 2014-02-10 12:44 UTC by Jonas Danielsson
Modified: 2015-08-17 12:15 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Icons for entrycompletion (2.34 KB, patch)
2014-02-10 12:45 UTC, Jonas Danielsson
none Details | Review
Icons for entrycompletion, symbollic (2.37 KB, patch)
2014-02-10 12:46 UTC, Jonas Danielsson
none Details | Review
non-symbolic (637.93 KB, image/png)
2014-02-12 11:47 UTC, Jonas Danielsson
  Details
symbolic (632.64 KB, image/png)
2014-02-12 11:47 UTC, Jonas Danielsson
  Details
searchPopup: move treeview creation to ui file (3.91 KB, patch)
2014-03-02 12:36 UTC, Jonas Danielsson
none Details | Review
placeStore: Make exists method public (1.87 KB, patch)
2014-03-02 12:37 UTC, Jonas Danielsson
none Details | Review
Add favorite icon to search popover (4.00 KB, patch)
2014-03-02 12:37 UTC, Jonas Danielsson
none Details | Review
Add favorite icon to search completion (3.31 KB, patch)
2014-03-02 12:37 UTC, Jonas Danielsson
none Details | Review

Description Jonas Danielsson 2014-02-10 12:44:51 UTC
So, what icons should we use? Do we need new ones? 

In GNOME right now we have 'document-open-recent' and 'document-open-recent-symbolic', of which I think the symbolic one looks nicer.

And we also have 'starred' and 'non-starred' and also the symbolic forms.

I tried taking some screenshots but when I press print-screen the completion-popup disappears :)

Will append a patch with the symbolic and the non-symbolic versions of the icons mentioned above. In order to see the favorite one you need to edit the ~/.local/share/maps-places.json file and change a type: 0 to a type: 1, since we can't add favorites yet.
Comment 1 Jonas Danielsson 2014-02-10 12:45:44 UTC
Created attachment 268666 [details] [review]
Icons for entrycompletion
Comment 2 Jonas Danielsson 2014-02-10 12:46:04 UTC
Created attachment 268667 [details] [review]
Icons for entrycompletion, symbollic
Comment 3 Jonas Danielsson 2014-02-10 12:46:22 UTC
So either use these, or create our own. What do you think?
Comment 4 Zeeshan Ali 2014-02-11 12:02:26 UTC
(In reply to comment #3)
> So either use these, or create our own. What do you think?

screenshots will help better than patches to compare in this case. :)
Comment 5 Jonas Danielsson 2014-02-11 12:04:02 UTC
(In reply to comment #4)
> (In reply to comment #3)
> > So either use these, or create our own. What do you think?
> 
> screenshots will help better than patches to compare in this case. :)

Agreed, can you tell me a way to take screenshots? If I press anything the dropdownbox will dissapear. I guess I could take a picture with my phone :)
Comment 6 Jonas Danielsson 2014-02-12 11:46:40 UTC
Ok, I will do a mockup by adding them to the searchPopup so you can see what they look like.
Comment 7 Jonas Danielsson 2014-02-12 11:47:07 UTC
Created attachment 268900 [details]
non-symbolic
Comment 8 Jonas Danielsson 2014-02-12 11:47:25 UTC
Created attachment 268901 [details]
symbolic
Comment 9 Zeeshan Ali 2014-02-12 13:30:03 UTC
(In reply to comment #7)
> Created an attachment (id=268900) [details]
> non-symbolic

Meh, that looks like this very ugly DE that I don't want to mention by name. :) Symbolic is the outright winner I think.
Comment 10 Zeeshan Ali 2014-02-12 13:30:51 UTC
Having said that, I'm not at all sure we want to show an icon for 'recent'. How about just 'favorite'.
Comment 11 Jonas Danielsson 2014-02-14 13:25:29 UTC
If we are talking about the search-popup I agree, and I'm not really sure we even should show favorites there.

But in the EntryCompletion my thinking was that it would be nice to show recent/favorite icons as a hint to users. To avoid giving the impression that it is completing on server-side search results.
Comment 12 Zeeshan Ali 2014-02-14 14:22:49 UTC
(In reply to comment #11)
> If we are talking about the search-popup I agree, and I'm not really sure we
> even should show favorites there.
> 
> But in the EntryCompletion my thinking was that it would be nice to show
> recent/favorite icons as a hint to users. To avoid giving the impression that
> it is completing on server-side search results.

I don't see why user should care, really if its client or server-side. IMHO showing the favorite icon makes sense though.
Comment 13 Jonas Danielsson 2014-03-02 12:36:32 UTC
Created attachment 270694 [details] [review]
searchPopup: move treeview creation to ui file
Comment 14 Jonas Danielsson 2014-03-02 12:37:07 UTC
Created attachment 270695 [details] [review]
placeStore: Make exists method public
Comment 15 Jonas Danielsson 2014-03-02 12:37:12 UTC
Created attachment 270696 [details] [review]
Add favorite icon to search popover
Comment 16 Jonas Danielsson 2014-03-02 12:37:17 UTC
Created attachment 270697 [details] [review]
Add favorite icon to search completion