Bug 457596 - FEAT: Catalog should disable all the input field and "Save Change" button in the certification test plan on people who are on the CC list 's view
Summary: FEAT: Catalog should disable all the input field and "Save Change" button in ...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Hardware Certification Program
Classification: Retired
Component: Hardware Catalog
Version: 5.2
Hardware: All
OS: Linux
low
low
Target Milestone: ---
: ---
Assignee: XINSUN
QA Contact:
URL: http://hardware.redhat.com
Whiteboard:
Depends On: 428008
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-08-01 17:37 UTC by XINSUN
Modified: 2008-10-28 05:48 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-10-28 05:48:49 UTC
Embargoed:


Attachments (Terms of Use)
Patch Fix: (3.42 KB, patch)
2008-08-01 19:28 UTC, XINSUN
no flags Details | Diff
Screenshot for the testplan on cc 's view. (149.69 KB, image/png)
2008-08-01 19:31 UTC, XINSUN
no flags Details

Description XINSUN 2008-08-01 17:37:22 UTC
The testplan have been shown to the people on cc list(This was fixed on bug
428008).but for user experience,catalog should disable all the input field and
"Save Change" button in the certification test plan on people who are on the CC
list 's view.

Comment 1 XINSUN 2008-08-01 19:28:39 UTC
Created attachment 313223 [details]
Patch Fix:

pls review.

Nicho

Comment 2 XINSUN 2008-08-01 19:31:03 UTC
Created attachment 313224 [details]
Screenshot for the testplan on cc 's view.

Comment 3 Rob Landry 2008-08-25 16:02:55 UTC
Screenshot looks good; patch looks reasonable clean, from what I see basically it both defines and calls a function called disabled onload which then sets the disabled property of every element it finds in the form to true?  Presuming this is correct and it tests well for all users roles please commit to CVS.


Note You need to log in before you can comment on or make changes to this bug.