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 669576 - Add contact button
Add contact button
Status: RESOLVED FIXED
Product: gnome-contacts
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: GNOME Contacts maintainer(s)
GNOME Contacts maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2012-02-07 14:09 UTC by Allan Day
Modified: 2013-03-21 10:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch (931 bytes, patch)
2012-11-14 18:20 UTC, Allan Day
none Details | Review
mockup experiments (22.24 KB, image/png)
2012-11-23 17:48 UTC, Allan Day
  Details

Description Allan Day 2012-02-07 14:09:09 UTC
It currently reads 'Add...' but I think it would be better if it just read 'New'. That would be consistent with the other new apps.

The button also needs more internal horizonal padding - it should be much wider. 20px between the text and the internal edges would be good.
Comment 1 Alexander Larsson 2012-02-13 16:18:02 UTC
Changed the text, padding left to do.
Comment 2 Felipe Erias Morandeira 2012-02-22 11:01:52 UTC
I can see "Add..." making some sense, though: "you add a friend to your contact list by creating a new entry".

The decision is whether the button should refer to your friend ("Add...") or the contact entry itself ("New"). I'd go with the first option, but clearly both are understandable.

Consider how this is different from creating a new document: the document doesn't exist before you press the button, but your friend does.
Comment 3 Andreas Nilsson 2012-07-05 23:13:53 UTC
Should the fact that the button it don't line up with the A in Activities of the shell (similar to Web bug #679348) be added to this bug report, or should I create a new one about that?
Comment 4 Alexander Larsson 2012-09-05 08:12:26 UTC
Andreas: Please file a new bug about that.
Comment 5 Allan Day 2012-11-14 18:20:22 UTC
Created attachment 229000 [details] [review]
patch

Here's a one liner to fix the button size.
Comment 6 Alexander Larsson 2012-11-22 15:29:15 UTC
Hmm, that just sets the minimal size to a hardcoded size. What if the translated string is longer than that?
Comment 7 Allan Day 2012-11-23 17:48:47 UTC
Created attachment 229736 [details]
mockup experiments

(In reply to comment #6)
> Hmm, that just sets the minimal size to a hardcoded size. What if the
> translated string is longer than that?

I think we ideally want to combine a minimum width with horizontal padding. That way we ensure that the button never shrinks to an unusable size, and we avoid making the button too huge if the string is long (see the attachment).

I investigated setting internal padding via CSS but it didn't seem to work.
Comment 8 Pierre-Yves Luyten 2012-12-18 01:25:41 UTC
I had the same concern and used gtk_misc_set_padding. Maybe does it fit your need?
Comment 9 Claudio Saavedra 2013-01-23 15:46:13 UTC
While we are at it: am I the only one who thinks that the "New" button being the only item in that horizontal array looks terrible? It seems like a waste of screen estate. I would definitely move it elsewhere, perhaps next to the search entry using only a '+' icon. Whatever you prefer, but where it stands now, it looks pretty bad.
Comment 10 Erick Perez Castellanos 2013-01-23 15:53:15 UTC
(In reply to comment #9)
> While we are at it: am I the only one who thinks that the "New" button being
> the only item in that horizontal array looks terrible? It seems like a waste of
> screen estate. I would definitely move it elsewhere, perhaps next to the search
> entry using only a '+' icon. Whatever you prefer, but where it stands now, it
> looks pretty bad.

The toolbar has changed in master.
Comment 11 Allan Day 2013-03-21 10:47:19 UTC
(In reply to comment #10)
> The toolbar has changed in master.

Yeah, I think we can close this.