Page MenuHomePhorge

Drag & Drop: set a link as external
ClosedPublic

Authored by valerio.bozzolan on Mar 17 2023, 12:20.
Tags
None
Referenced Files
F3334967: D25077.1743556031.diff
Tue, Apr 1, 01:07
F3331902: D25077.1743514829.diff
Mon, Mar 31, 13:40
F3330417: D25077.1743488333.diff
Mon, Mar 31, 06:18
F3329080: D25077.1743468340.diff
Mon, Mar 31, 00:45
F3328377: D25077.1743455587.diff
Sun, Mar 30, 21:13
F3327510: D25077.1743440597.diff
Sun, Mar 30, 17:03
F3327483: D25077.1743440181.diff
Sun, Mar 30, 16:56
F3322871: D25077.1743379815.diff
Sun, Mar 30, 00:10

Details

Summary

Rest assured: external links remain evil, by default.

Don't adopt them randomly by induction.

Whether you believe it or not, this specific external
link merited some deep thoughts on Phorge:

So, whenever you use a mouse, a finger, or whenever we have
a confirmation dialog or not to prevent onblur disasters,
this change is probably consistent with common expectations.

Having said, external links remain evil - by default.

Closes T15172

Test Plan
  • Drag & Drop a File on a Remarkup text
  • click on the link inside the popup
  • it opens in a new tab (without risk of form loss)

Diff Detail

Repository
rP Phorge
Branch
T15172-drag-and-drop-popup-target-blank
Lint
Lint Warnings
SeverityLocationCodeMessage
Warningwebroot/rsrc/js/core/FileUpload.js:127TXT3Line Too Long
Unit
No Test Coverage
Build Status
Buildable 138
Build 138: arc lint + arc unit

Event Timeline

fix lint error "Line Too Long"

This revision is now accepted and ready to land.Mar 17 2023, 12:54