#7360 closed defect (bug) (fixed)
Can't upload profile images in Microsoft Edge browser
Reported by: | angeljs | Owned by: | hnla |
---|---|---|---|
Milestone: | 2.7.3 | Priority: | normal |
Severity: | normal | Version: | 2.7.2 |
Component: | Core | Keywords: | |
Cc: |
Description
After making this post:
https://buddypress.org/support/topic/cant-upload-profile-images-in-internet-explorer/
I was asked to create a ticket here.
Clicking on the browse image does not work in the Microsoft Edge browser, although it works in Internet Explorer and other browsers. Members using Edge cannot upload profile or cover images.
Attachments (1)
Change History (21)
#2
@
8 years ago
- Milestone changed from Awaiting Review to 2.7.3
I"m provisionally moving this into 2.7.3, we have another ticket that does need to come out prior to 2.8, and breaking a key feature for an entire browser does seem like something we should fix quicker. :)
@hnla What version of WordPress was your test site using? Plupload comes with WP, and I'm wondering if you were on WP 4.7 trunk/beta/rc etc, because they may have updated it there.
#4
@
8 years ago
Confirmed drag 'n' drop works as expected.
@djpaul I'm on bleeding edge for both WP/BP but hadn't updated WP so will check that just in case.
We have an fairly extensive file managing bp-plupload, this has code matching to that which was in bug report so may need to test those suggested changes.
#5
@
8 years ago
@djpaul downloaded and updated the core WP plupload js files `(plupload-full.min.js) v2.2.0 and this appears to correct issue, this is not a conclusive test as the files between WP & plupload package seem to differ slightly.
This WP trac ticket identifies the same issue (or very similar) but looks like it addresses a specific use case?:
https://core.trac.wordpress.org/ticket/37039
This does however appear to be a WP core issue to deal with in terms of updating core vendor files.
#6
@
8 years ago
I've opened a ticket on WP Trac:
https://core.trac.wordpress.org/ticket/38912#ticket
#7
@
8 years ago
And missed the fact we have had an extensive ticket on this and other issues with respect to our plupload implementation:
#6846
This suggested things were fixed!
Not sure my suggestion or WP ticket to update to update Plupload vendor files is valid frankly, we need to digest the fixes we added and comments on this previous BP ticket.
This ticket was mentioned in Slack in #buddypress by slaffik. View the logs.
8 years ago
#9
@
8 years ago
https://wordpress.slack.com/archives/buddypress/p1479932039001391
It would be very interesting to know if it is new in 2.7 (did it work in 2.6?), or whether its dependant on WP 4.6 vs 4.7, or combinations thereof.
#10
follow-up:
↓ 11
@
8 years ago
Just tested versions tracking back to 2.4.0 to see if the upload button worked in Edge but appears not to in all cases, also pretty sure it isn't as simple as latest plupload as reverting that didn't instantly revert to the issue so feels intermittent in nature, also now don't think it's the CSS overlay but feels more to do with binding the click event.
Edit In testing just noticed that the click event area is off set from the button rendering/positioning, click outside the button top edge and file explorer will open or should open.
If this is correct it's the same issue we thought we had fixed in #6846 so we'll need to revist that ticket and look again at the fixes and CSS.
#11
in reply to:
↑ 10
@
8 years ago
#12
@
8 years ago
Adding a rough patch as a workaround.
Issue essentially is stacking contexts are - as always in IE - different to other modern browsers, and the positioning of the shim is not aligned to the actual input button.
Patch manipulates the inline top property applied to the shim to match to our button, left property value with % unit to allow some degree of text resizing and adds widths to our button and shim input.
This works... more or less Edge stacking context is wrong so we lose the hover effect but re-stacking the elements means click event is hidden.
Quick check in Firefox and we still have functionality there, for the moment this is as far as I have time for but honestly to really sort this out the markup needs re-ordering so that elements are children not separated and forced positioned which is fragile.
This ticket was mentioned in Slack in #buddypress by slaffik. View the logs.
8 years ago
#14
@
8 years ago
honestly to really sort this out the markup needs re-ordering so that elements are children not separated and forced positioned which is fragile.
Agreed. Needs a11y fix.
#15
@
8 years ago
@mercime if we do tackle that then we'll likely need a little time on it as this sort of markup and style wrangling is always fraught with issues so we probably ought to take it out of the 2.7.3 tag?
#16
@
8 years ago
@hnla quick fix is fine for 2.7.3 so it's usable for "Clicking on the browse image" as was reported. a11y fix for 2.8.
#17
@
8 years ago
@mercime yep - I'll commit quick fix in a day or two, I'll give it a quick check over see if it can be improved a little.
Confirmed as an issue.
IE < Edge appear to work ok.
On first glance feels like the click event not firing and may be associated with this plupload bug report:
https://github.com/moxiecode/plupload/issues/1343
Going to look further and hope that it's a case of maybe needing to update plupload?