Use "readonly" instead of "disabled" for "Install Key" input.

Review Request #7380 — Created June 4, 2015 and submitted

Information

Review Board
release-1.7.x
64b636d...

Reviewers

Firefox has a terrible, horrible, no good, very bad bug whereby a text input
which is disabled cannot be selected. This is especially problematic for our
install key field, which is a 40-character long hash which we want people to be
able to copy and paste.

It turns out that the "readonly" property works equally well for disabling
changes, but Firefox allows selection on those. This property is technically
HTML5 but is well-supported across all browsers.

Verified that "readonly" did the right thing in both Firefox and Chrome.

reviewbot
  1. Tool: Pyflakes
    Processed Files:
        reviewboard/admin/forms.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        reviewboard/admin/forms.py
    
    
  2. 
      
chipx86
  1. Ship It!
  2. 
      
david
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-1.7.x (fe2152e)
Loading...