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
F15397537: D7810.id17668.diff
Sun, Mar 16, 8:55 PM
F15395906: D7810.id17667.diff
Sun, Mar 16, 10:39 AM
F15392355: D7810.diff
Sat, Mar 15, 2:44 PM
Unknown Object (File)
Feb 17 2025, 2:56 AM
Unknown Object (File)
Feb 17 2025, 1:44 AM
Unknown Object (File)
Feb 9 2025, 10:55 AM
Unknown Object (File)
Feb 9 2025, 10:55 AM
Unknown Object (File)
Feb 9 2025, 10:55 AM

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