[buddypress-trac] [BuddyPress Trac] #6290: Avatars, an extensible UI

[buddypress-trac]

#6290: Avatars, an extensible UI
------------------------------------+------------------
 Reporter:  imath                   |       Owner:
     Type:  idea                    |      Status:  new
 Priority:  normal                  |   Milestone:  2.3
Component:  API - Avatars           |     Version:
 Severity:  normal                  |  Resolution:
 Keywords:  has-patch dev-feedback  |
------------------------------------+------------------

Comment (by imath):

 6290.02.patch is :
 - applying the suggestion boonebgorges made about
 `bp_attachments_get_plupload_l10n()` : use our own strings instead of
 relying on wp_plupload_l10n ones
 - applying the suggestion boonebgorges made about improving the way Ajax
 feedback messages are set. It's now happening in the
 `BP_Attachment_Avatar()->script_data()` function and each case (group
 profile photo, or user profile photo...) is no more a result of a
 concatenation/sprintf operation.
 - fixing a bug hnla discovered testing the camera feature due to a device
 resolution lower than mine. Now we are checking the videoWidth and
 videoHeight to be sure it's upper than 150 and the capture dimensions is
 now dynamically built.
 - making sure the canvas style will not be applied a max-width
 (twentyfourteen theme was messing the layout) < thanks to hnla for this
 one!

 I haven't changed anything about the location of the javascript templates,
 as i understood with yesterday's dev-chat that we still need some time to
 decide where they should live.

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/6290#comment:18>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac