Page MenuHomePhabricator

Render the clone URI input field as readonly.
ClosedPublic

Authored by wotte on Dec 20 2013, 4:16 PM.
Tags
None
Referenced Files
F19540953: D7810.id17667.diff
Thu, Jan 22, 1:06 AM
F18817810: D7810.id17667.diff
Oct 21 2025, 6:37 PM
F18780769: D7810.id.diff
Oct 12 2025, 7:44 PM
F18768415: D7810.diff
Oct 8 2025, 3:28 AM
F18616569: D7810.id.diff
Sep 14 2025, 6:10 PM
F18601781: D7810.diff
Sep 13 2025, 1:28 PM
F18091412: D7810.id17668.diff
Aug 6 2025, 9:37 PM
F18091331: D7810.id.diff
Aug 6 2025, 9:08 PM

Details

Summary

There's no particular reason to allow the user to edit the clone URI field in Diffusion; editing it has no meaning and if you fat finger the keyboard, it's quite possible that the user will either accidentally clear and/or modify the URI before copying (bit me this morning).

Adding a readonly attribute to the input field allows the same benefit (URI is easily selectable) while preventing such accidental input. Fixes T4246.

Test Plan

Verified that the desired behavior is present in both Chrome, Safari, and Firefox. Field remains selectable with one click, but field is not editable.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped