[Freeipa-devel] [PATCH] 0249-optional-uid.

[Endi Sukma Dewata]

On 6/20/2011 8:25 PM, Adam Young wrote:
> Note that this patch needs a review by UXD in addition to code review

Some issues:

1. The patch tries to find the elements to be hidden using
    span.find('input'). This will not work with all widgets because
    some widgets use other elements or some combination. I think it's
    better to add the link outside the span, then hide the span itself
    to hide the entire widget.

2. Hiding the optional widgets but not the labels might not be enough
    to simplify the screen because they still occupy some space. Another
    solution is to hide both the labels and the widgets (i.e. the entire
    row) then have a link for to hide/show all optional fields somewhere
    else.

-- 
Endi S. Dewata