Skip to:
Content

Opened 23 months ago

Closed 20 months ago

Last modified 17 months ago

#4253 closed enhancement (fixed)

Avatar cropper on touch devices

Reported by: modemlooper Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: Core Keywords:
Cc:

Description

iOS 6 includes uploads via mobile safari. BuddyPress images upload great but the cropper fails. I replaced jcrop and made it work. I checked to see what version WordPress was using and its an older version, however a patch was created to update jcrop in 3.4 http://core.trac.wordpress.org/ticket/20728

If its not included in 3.4 maybe the latst version of jcrop should be bundled with BP
http://deepliquid.com/content/Jcrop.html

Change History (7)

comment:1 DJPaul23 months ago

  • Keywords dev-feedback removed
  • Milestone changed from Awaiting Review to Future Release

I'd rather see if WP core will put this into a future 3.4.1 release. If it's not in WordPress by the time 1.7 rolls around, we can think about adding it into BuddyPrss. We have a couple of months before iOS 6.0 is due out.

comment:2 modemlooper23 months ago

I'll post on that jcrop ticket to let them know it's a needed update for BP avatars. I'd like to add avatar and image uploads to BP mobile

comment:3 modemlooper20 months ago

  • Resolution set to fixed
  • Status changed from new to closed

@nacin committed the patch to WP 3.5

comment:4 boonebgorges20 months ago

(In [6330]) Use correct argument swapping in BP_Activity_Activity::save() for item_id and secondary_item_id

Since BP 1.6, the item_id and secondary_item_id columns of the activity table
have been bigint rather than varchar. This changeset fixes the argument
swapping in BP_Activity_Activity::save() to use the appropriate %d rather than
%s, so that null values turn into zeroes. This helps to avoid errors on certain
strict MySQL setups.

Fixes #4253

comment:5 boonebgorges20 months ago

(In [6331]) Use correct argument swapping in BP_Activity_Activity::save() for item_id and secondary_item_id

Since BP 1.6, the item_id and secondary_item_id columns of the activity table
have been bigint rather than varchar. This changeset fixes the argument
swapping in BP_Activity_Activity::save() to use the appropriate %d rather than
%s, so that null values turn into zeroes. This helps to avoid errors on certain
strict MySQL setups.

Fixes #4253

comment:6 boonebgorges20 months ago

Referenced the wrong ticket number, oops.

comment:7 johnjamesjacoby17 months ago

  • Milestone Future Release deleted
Note: See TracTickets for help on using tickets.