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
F13062833: D7810.diff
Sat, Apr 20, 2:46 AM
Unknown Object (File)
Wed, Apr 10, 4:21 AM
Unknown Object (File)
Feb 8 2024, 6:20 AM
Unknown Object (File)
Jan 24 2024, 9:35 AM
Unknown Object (File)
Jan 24 2024, 9:35 AM
Unknown Object (File)
Jan 24 2024, 9:35 AM
Unknown Object (File)
Jan 20 2024, 5:39 PM
Unknown Object (File)
Jan 18 2024, 12:41 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

Branch
T4246
Lint
Lint Passed
Unit
No Test Coverage