Closed
Bug 799715
Opened 12 years ago
Closed 12 years ago
[email] UI: make <a> tag hyperlinks in HTML e-mails clickable
Categories
(Firefox OS Graveyard :: Gaia::E-Mail, defect, P1)
Tracking
(blocking-basecamp:+)
People
(Reporter: asuth, Assigned: Daeken)
References
Details
(Keywords: feature, Whiteboard: [fix in bug 799723])
You can't click on hyperlinks that are part of an HTML message right now. The actual links are sanitized onto a separate attribute as part of the sanitization process in order to guarantee that no pre-fetching happens and because this UI step was planned. Per discussion with Casey, the plan is that if you click on a hyperlink we will pop up a confirmation box that asks if you are sure you want to browse to the URL by providing the URL. This: 1) Serves as our anti-phishing protection by showing you the URL. Thunderbird and web-mail tend to have the "you can hover on it to see where it really goes" defense. We have no hover, so that doesn't work for us. 2) Avoids randomly launching the browser app due to mis-touches, mis-interpreted touches, etc. A similar SMS bug is bug 796788. This is distinct from automatic detection of things that look like URLs and turning them into hyperlinks; that would happen in both text and HTML e-mails. I will file that bug now.
Reporter | ||
Comment 1•12 years ago
|
||
Bug 799723 has been filed for auto-linkification of URLs; it will want to reuse this UI even in the case of plaintext emails.
Blocks: 799723
Updated•12 years ago
|
blocking-basecamp: --- → ?
Comment 2•12 years ago
|
||
Nominated for basecamp - seems like a must have.
Updated•12 years ago
|
blocking-basecamp: ? → +
Priority: -- → P2
isn't this a dup of bug 799723?
Updated•12 years ago
|
Priority: P2 → --
Updated•12 years ago
|
Priority: -- → P1
Comment 4•12 years ago
|
||
Assigning to Cody since he's already working on the linkify feature 799723.
Assignee: nobody → cbrocious
Assignee | ||
Comment 5•12 years ago
|
||
I just attached a patch to bug 799723 which covers this issue as well (no way to split it up, really). Needs cleanup and testing, but otherwise fully functional. Can anyone comment on the wording? It currently displays a confirm() dialog that simply says "Browse to $URL$".
Flags: needinfo?
Comment 6•12 years ago
|
||
Casey, can you provide proper wording for the confirmation dialog?
Flags: needinfo? → needinfo?(kyee)
Comment 7•12 years ago
|
||
We're marking this bug with the C1 milestone since it follows the criteria of "unfinished feature work" (see https://etherpad.mozilla.org/b2g-convergence-schedule). If this work is not finished by Nov19, this bug will need an exception and will be called out at the upcoming Exec Review.
Target Milestone: --- → B2G C1 (to 19nov)
Reporter | ||
Comment 8•12 years ago
|
||
(In reply to Cody Brocious [:Daeken] from comment #5) > I just attached a patch to bug 799723 which covers this issue as well (no > way to split it up, really). Needs cleanup and testing, but otherwise fully > functional. Can anyone comment on the wording? It currently displays a > confirm() dialog that simply says "Browse to $URL$". The phrasing seems fine by me, I would add a question mark and then you should use the localization infrastructure. Specifically, it would look like: var prompt = mozL10n.get('prompt-browse-url', { url: url }); where you would then add to the properties file a line like: prompt-browse-url=Browse to {{ url }}? You should be able to find various other examples of using mozL10n throughout the code.
Flags: needinfo?(kyee)
Reporter | ||
Comment 9•12 years ago
|
||
(didn't mean to clear Casey's information request)
Flags: needinfo?(kyee)
Updated•12 years ago
|
Whiteboard: [fix in bug 799723]
Reporter | ||
Comment 10•12 years ago
|
||
back-end patch landed on gaia-email-libs-and-more/master by: https://github.com/mozilla-b2g/gaia-email-libs-and-more/pull/79 front-end patch with back-end changes landed on gaia/master by: https://github.com/mozilla-b2g/gaia/pull/6419
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
Flags: needinfo?(kyee)
Comment 12•11 years ago
|
||
we'll need a testcase for this in moztrap specific to html emails
Flags: in-moztrap?(twalker)
Updated•11 years ago
|
Flags: in-moztrap?(twalker) → in-moztrap?
Comment 13•11 years ago
|
||
Created a test case for interacting with links in the body of an e-mail. https://moztrap.mozilla.org/manage/cases/?filter-id=9276
Flags: in-moztrap?(dwatson) → in-moztrap+
You need to log in
before you can comment on or make changes to this bug.
Description
•