Closed
Bug 1492335
Opened 6 years ago
Closed 5 years ago
copying link from source view adds "source-view:" to link
Categories
(Firefox for Android Graveyard :: General, defect, P5)
Tracking
(firefox64 wontfix, firefox65 verified, firefox66 verified)
VERIFIED
FIXED
Firefox 66
People
(Reporter: felix.bau, Assigned: JanH)
Details
Attachments
(1 file)
47 bytes,
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-beta+
|
Details | Review |
User Agent: Mozilla/5.0 (Android 7.0; Mobile; rv:62.0) Gecko/62.0 Firefox/62.0 Build ID: 20180906142540 Steps to reproduce: I opened a page and then opened source view (html) Actual results: I wanted to copy a link's address so I did that the url in my clipboard had "source-view:" add the beginning Expected results: this could be intentional, but I would like it to be different if you click on it, then Firefox should open the link in source view if you copy the link's address over the context menu, then you should only get the link without "source-view:" at the beginning of it what do you think?
Assignee | ||
Comment 1•6 years ago
|
||
https://dxr.mozilla.org/mozilla-central/rev/85b4d2bf888afa32b67638602a3338d0a6935ff9/browser/base/content/nsContextMenu.js#1300-1306 needs porting over to the mobile context menu implementation.
Assignee: nobody → jh+bugzilla
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Unspecified → Android
Hardware: Unspecified → All
Updated•5 years ago
|
status-firefox64:
--- → wontfix
status-firefox65:
--- → fix-optional
status-firefox66:
--- → affected
Priority: -- → P5
Assignee | ||
Comment 2•5 years ago
|
||
Desktop does this when copying link URLs and sharing is just effectively just another form of copying. For completeness, we also apply this when "viewing" the source of images (which just displays the image itself anyway). No special handling is required for other media elements (audio/video), because looking at those in view-source mode does display the raw file contents and thus none of the media-specific context menu entries will show up.
Pushed by mozilla@buttercookie.de: https://hg.mozilla.org/integration/autoland/rev/531e57a2941f Strip view-source prefix when copying/sharing links from context menu. r=snorp
Comment 4•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/531e57a2941f
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 66
Comment 5•5 years ago
|
||
Did you want to nominate this for Beta uplift?
Flags: qe-verify+
Flags: needinfo?(jh+bugzilla)
Assignee | ||
Comment 6•5 years ago
|
||
Comment on attachment 9033329 [details] Bug 1492335 - Strip view-source prefix when copying/sharing links from context menu. r?snorp [Beta/Release Uplift Approval Request] Feature/Bug causing the regression: View source on Android User impact if declined: Copying a link from the context menu while in a view-source: tab copies the link including the view-source:-prefix, which apparently is not consistent with what Desktop does. Is this code covered by automated tests?: No Has the fix been verified in Nightly?: Yes Needs manual test from QE?: No If yes, steps to reproduce: List of other uplifts needed: none Risk to taking this patch: Low Why is the change risky/not risky? (and alternatives if risky): We're simply stripping off the view-source:-prefix in the context menu handler, which is very similar to the way this is done in Desktop Firefox. String changes made/needed: none
Flags: needinfo?(jh+bugzilla)
Attachment #9033329 -
Flags: approval-mozilla-beta?
Comment 7•5 years ago
|
||
I could reproduce this issue on Android Firefox 64.0.1. I verified this bug fix on Android Firefox 66.0a1 (2018-12-28), issue is no longer reproducible. I will verify on beta 65 after the fix gets uplifted to beta.
Comment 8•5 years ago
|
||
Comment on attachment 9033329 [details] Bug 1492335 - Strip view-source prefix when copying/sharing links from context menu. r?snorp [Triage Comment] Avoids copying the view-source: prefix when copying a link while viewing the page source. Approved for 65.0b8.
Attachment #9033329 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 9•5 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/3bfc60606908
Comment 10•5 years ago
|
||
I verified this bug fix on Android Firefox 65.0b13, issue is no longer reproducible.
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•