Page MenuHomePhabricator

Add a space after "View Inline" in mail to prevent double-click on the filename from selecting "Inline"
ClosedPublic

Authored by epriestley on Oct 10 2018, 7:44 PM.
Tags
None
Referenced Files
F18205387: D19742.id47171.diff
Mon, Aug 18, 2:46 PM
F18095153: D19742.id.diff
Thu, Aug 7, 10:32 PM
F17818348: D19742.diff
Sat, Jul 26, 12:18 AM
F17796098: D19742.id47171.diff
Fri, Jul 25, 12:43 AM
F17755838: D19742.diff
Jul 22 2025, 1:14 PM
F17743560: D19742.id47171.diff
Jul 21 2025, 2:15 PM
Unknown Object (File)
Jul 3 2025, 10:37 AM
Unknown Object (File)
Jun 2 2025, 2:08 PM
Subscribers
None

Details

Summary

See PHI920. Ref T13210. Since the HTML is just:

<a>View Inline</a><span>filename.txt</span>

..double-clicking "filename.txt" in email selects "Inlinefilename.txt".

Add a space to stop this. At least in Safari, a space between the tags is not sufficient (perhaps because the parent is a <div>?). I couldn't find an authoritative-seeming source on what the rules for this actually are and adding a space here fixes the issue without changing the visual rendering, so just put it here.

Test Plan
  • Made an inline.
  • Used bin/mail show-outbound --id ... --dump-html to dump the HTML.
  • Double-clicked the filename.

Screen Shot 2018-10-10 at 12.40.39 PM.png (956×1 px, 131 KB)

Diff Detail

Repository
rP Phabricator
Branch
select1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 21008
Build 28538: Run Core Tests
Build 28537: arc lint + arc unit