Closed Bug 989530 Opened 10 years ago Closed 10 years ago

Quickshare should use text/plain type for links

Categories

(Firefox for Android Graveyard :: General, defect)

x86_64
Linux
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED
Firefox 31

People

(Reporter: wesj, Assigned: wesj)

References

Details

Attachments

(1 file)

We currently use text/html as our default, which leads to a whole slew of sharing apps not appearing. We should use text/plain.
Attached patch PatchSplinter Review
Attachment #8398816 - Flags: review?(mark.finkle)
Blocks: 942270
Comment on attachment 8398816 [details] [diff] [review]
Patch

>diff --git a/mobile/android/base/prompts/PromptListItem.java b/mobile/android/base/prompts/PromptListItem.java

>+    public final static String DEFAULT_MIME_TYPE = "text/plain";

Not needed. You are pulling it from GeckoActionProvider.

r+, but remove the unneeded addition
Attachment #8398816 - Flags: review?(mark.finkle) → review+
I fixed this in bug 990364.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Assignee: nobody → wjohnston
Depends on: 990364
Target Milestone: --- → Firefox 31
Based on comment 6 from Bug 989532 

->Context menu links and the main menu now use text/plain for their mimetype, and all use the default history file.
Verified on 32 Nightly 05/26, 31 Aurora 05/26 and 30 Beta 6.

I will mark this bug as verified fixed
Status: RESOLVED → VERIFIED
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: