953777 - Give the user a way to set a buddy icon and a display name

Status: RESOLVED FIXED

(Instantbird :: Contacts window, enhancement)

Description

[Florian Quèze [:florian]]

*** Original post on bio 334 at 2010-02-16 16:08:00 UTC ***

*** Due to BzAPI limitations, the initial description is in comment 1 ***

Comment 1

[Florian Quèze [:florian]]

Attachment: WIP

*** Original post on bio 334 as attmnt 267 at 2010-02-16 16:08:00 UTC ***

The toolbar at the top of the buddy list should allow the user to view and set a buddy icon and a display name.

The attachment is a XUL mockup/work-in-progress.

Comment 2

[Florian Quèze [:florian]]

Attachment: Screenshot on Mac

*** Original post on bio 334 as attmnt 268 at 2010-02-16 16:10:00 UTC ***

This is a screenshot of how this should look on Mac.
The attached mockup in comment 0 doesn't produce exactly this result, there's an issue with the alignment of the label containing the status text.

Comment 3

[Florian Quèze [:florian]]

Attachment: Screenshot/mockup on Windows XP

*** Original post on bio 334 as attmnt 269 at 2010-02-16 16:11:00 UTC was without comment, so any subsequent comment numbers will be shifted ***

Comment 4

[Florian Quèze [:florian]]

Attachment: WIP 2

*** Original post on bio 334 as attmnt 610 at 2011-05-05 16:20:00 UTC ***

Updated attachment 8352009 [details] [diff] [review] (bio-attmnt 267) and fixed the issue mentioned in comment 1.

Comment 5

[Florian Quèze [:florian]]

Comment on attachment 8352009 [details] [diff] [review]
WIP

*** Original change on bio 334 attmnt 267 at 2011-05-05 16:20:51 UTC was without comment, so any subsequent comment numbers will be shifted ***

Comment 6

[Florian Quèze [:florian]]

Attachment: WIP 3

*** Original post on bio 334 as attmnt 611 at 2011-05-06 15:45:00 UTC ***

Added:
- JS code in the buddy list to open a file picker when the icon place holder is clicked.
- C++ code in purpleCoreService to store the chosen icon in the profile folder, keep track of it, and fire a user-icon-changed notification
- JS code in the buddy list to display the new icon when it's changed.

Next:
- C++ code in purpleAccount to observe the user-icon-changed notification and propagate new icons to the servers of libpurple accounts.
This should resize and convert the new icon to a format acceptable by the protocol plugin, store the new icon in libpurple's buddy icon store, and keep the handle of the stored icon in the account preferences.
- When loading a libpurple account, load the stored icon handle from the account preferences and reapply it.
- Apply the current icon when a new account is created.

Comment 7

[Florian Quèze [:florian]]

Comment on attachment 8352353 [details] [diff] [review]
WIP 2

*** Original change on bio 334 attmnt 610 at 2011-05-06 15:45:29 UTC was without comment, so any subsequent comment numbers will be shifted ***

Comment 8

[Florian Quèze [:florian]]

Attachment: WIP 4

*** Original post on bio 334 as attmnt 617 at 2011-05-10 22:37:00 UTC ***

Done:
Almost all purplexpcom and libpurple changes needed for the buddy icons. Buddy icons work correctly with this patch.

Next:
 - apply the current buddy icon to new accounts.
 - ensure that it's possible to remove the icon (setting the new icon to NULL should remove the existing icon. I haven't verified that this works so I assume it currently doesn't).
 - Check/fix the CSS on Windows/Linux.
 - The display name part.

Comment 9

[Florian Quèze [:florian]]

Comment on attachment 8352354 [details] [diff] [review]
WIP 3

*** Original change on bio 334 attmnt 611 at 2011-05-10 22:37:31 UTC was without comment, so any subsequent comment numbers will be shifted ***

Comment 10

[Florian Quèze [:florian]]

Attachment: WIP 5

*** Original post on bio 334 as attmnt 619 at 2011-05-12 00:29:00 UTC ***

Done:
The "Next" part of comment 5 (buddy icon related items only).

Next:
Setting the display name.

Comment 11

[Florian Quèze [:florian]]

Comment on attachment 8352360 [details] [diff] [review]
WIP 4

*** Original change on bio 334 attmnt 617 at 2011-05-12 00:29:31 UTC was without comment, so any subsequent comment numbers will be shifted ***

Comment 12

[Florian Quèze [:florian]]

Attachment: Patch v6

*** Original post on bio 334 as attmnt 620 at 2011-05-12 17:01:00 UTC ***

I think it's ready (for review?).

Comment 13

[Florian Quèze [:florian]]

Comment on attachment 8352362 [details] [diff] [review]
WIP 5

*** Original change on bio 334 attmnt 619 at 2011-05-12 17:01:39 UTC was without comment, so any subsequent comment numbers will be shifted ***

Comment 14

[Florian Quèze [:florian]]

*** Original post on bio 334 at 2011-05-12 17:21:59 UTC ***

Pushed attachment 8352363 [details] [diff] [review] (bio-attmnt 620) as https://hg.instantbird.org/instantbird/rev/e52ac1260bb5
-> Fixed.

Comment 15

[Benedikt Pfeifer]

*** Original post on bio 334 at 2011-05-14 16:23:16 UTC ***

I just found a wrong case in an id-attribute but I guess nobody will care? ;)

displayNameAndstatusMessageStack

Comment 16

[Florian Quèze [:florian]]

*** Original post on bio 334 at 2011-05-16 14:27:54 UTC ***

Fixed a bug reported by even in the way the "Display Name" empty text is set after finishing an edit without entering anything in the textbox: http://hg.instantbird.org/instantbird/rev/7f1f36534832

Comment 17

[Florian Quèze [:florian]]

*** Original post on bio 334 at 2011-05-19 13:23:40 UTC ***

More or less follow-ups:
* https://hg.instantbird.org/instantbird/rev/13baeefb20d8 - add notifications when a buddy icon for a contact/buddy/conversation is changed.
* Bug 953888 (bio 448) - ability for message themes to display the user's own icon.