Web apps - Share selected text doesn't work on Android 6+

RESOLVED DUPLICATE of bug 1360658

Status

()

Firefox for Android
Web Apps
P1
normal
RESOLVED DUPLICATE of bug 1360658
8 months ago
7 months ago

People

(Reporter: Oana Horvath, Assigned: nechen)

Tracking

(Blocks: 1 bug)

55 Branch
ARM
Android
Points:
---

Firefox Tracking Flags

(firefox55 affected)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Reporter)

Description

8 months ago
Device: Android 6+ devices.
Build: Nightly 55.0a1 (2017-04-12).

Steps to reproduce:
1. Install a web app (e.g. Twiiter) and open it.
2. Long press on a word to select it.
3. Tap Share in the dialog box opened.

Expected result:
The Share action should open a list of apps to share with.

Actual result:
The Share button doesn't open the sharing apps list.
(Reporter)

Updated

8 months ago
Blocks: 1285858
Priority: -- → P1
Assignee: nobody → cnevinchen
Comment hidden (mozreview-request)
Comment hidden (mozreview-request)
(Assignee)

Comment 3

8 months ago
Comment on attachment 8862791 [details]
Bug 1355847 - Share selected text should work.

Just need to call super method so things in GeckoApp handleMessage can be called.

Comment 4

7 months ago
mozreview-review
Comment on attachment 8862791 [details]
Bug 1355847 - Share selected text should work.

https://reviewboard.mozilla.org/r/134682/#review138102

LGTM
Attachment #8862791 - Flags: review?(max) → review+

Comment 5

7 months ago
mozreview-review
Comment on attachment 8862791 [details]
Bug 1355847 - Share selected text should work.

https://reviewboard.mozilla.org/r/134682/#review138264
Attachment #8862791 - Flags: review?(walkingice0204) → review+
Comment hidden (mozreview-request)
(Assignee)

Updated

7 months ago
Keywords: checkin-needed

Comment 7

7 months ago
mozreview-review
Comment on attachment 8862791 [details]
Bug 1355847 - Share selected text should work.

https://reviewboard.mozilla.org/r/134682/#review138354

So r+, howevet JanH already landed this in https://bugzilla.mozilla.org/show_bug.cgi?id=1360658 so I think you can close as dupe
Attachment #8862791 - Flags: review?(dale) → review+

Comment 8

7 months ago
We're sorry - something has gone wrong while rewriting or rebasing your commits. The commits being pushed no longer match what was requested. Please file a bug.
Keywords: checkin-needed
Status: NEW → RESOLVED
Last Resolved: 7 months ago
Resolution: --- → DUPLICATE
Duplicate of bug: 1360658
You need to log in before you can comment on or make changes to this bug.