Closed
Bug 639497
Opened 13 years ago
Closed 13 years ago
Implement sharing for mobile site
Categories
(Websites :: demos.mozilla.org, defect, P3)
Websites
demos.mozilla.org
Tracking
(Not tracked)
VERIFIED
FIXED
2.0
People
(Reporter: ozten, Assigned: grlicky)
Details
Build Twitter and Facebook sharing which is mobile compatible.
Reporter | ||
Updated•13 years ago
|
OS: Linux → All
Hardware: x86 → All
Target Milestone: --- → 2.0
Reporter | ||
Updated•13 years ago
|
Assignee: nobody → jgrlicky
Assignee | ||
Comment 1•13 years ago
|
||
TLDR: - Should we turn the "Like" button into a "Share" button for mobile to make it work in Fennec when the user isn't logged in? - Demo sharing seems to be popup-blocked on iPhone. I'm going to try and get around this by tweaking the JS sharing function. UNABRIDGED: Here are my findings so far about mobile sharing on WoW: 1) Page-level tweeting seems to work fine on Fennec, but page-level liking only seems to work on Fennec if you are logged in (although the hit area is tiny). If you aren't logged in, it does not seem to respond to clicks. It doesn't seem like we have any options for getting a bigger (or different) "like" button from Facebook, unless I'm misunderstanding our options. We might be better off turning this into a "share" button like the demo-level sharing. 2) Sharing a demo via twitter works on Fennec, but is popup-blocked on iPhone. If it is important, we could probably get around this by making the JS share function directly open a window instead of calling a function first. 3) Sharing a demo via facebook is the same story as sharing via twitter. Additionally, the popup launched is not mobile-friendly. I don't know that much about the Facebook API, but it looks like getting a mobile-friendly dialog will require using the Javascript SDK (http://developers.facebook.com/docs/reference/javascript/) to pop up a Feed Dialog (http://developers.facebook.com/docs/reference/dialogs/feed/). In order to do this, as mentioned at http://developers.facebook.com/docs/reference/dialogs/ , it seems like we'll need a Facebook "App ID" for our site. Do we have one of these yet? This might not be a worthwhile investment of time, given the tight schedule.
Assignee | ||
Comment 2•13 years ago
|
||
#2 is fixed in commit 86068fbefae18462d7bf Because of the schedule, I'm not planning on making the popups mobile friendly (#3), and #1 seems low-priority, so I will work on other tasks for now and see if I can get to this later.
Priority: -- → P3
Assignee | ||
Comment 3•13 years ago
|
||
Austin and I agree that for this release, it looks like the return on investment for the like button and mobile-friendly popups isn't worth it. Let's spend more time on this later, and see if a better solution presents itself at that time.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•