[Freeipa-devel] [PATCH] 288 Disable enroll button if nothing selected.
Endi Sukma Dewata
edewata at redhat.com
Thu Sep 29 04:50:57 UTC 2011
On 9/28/2011 7:59 PM, Adam Young wrote:
> On 09/28/2011 06:50 PM, Endi Sukma Dewata wrote:
>> A new IPA.dialog_button class has been added to encapsulate the
>> buttons in the dialog box so they can be managed more easily.
>>
>> The adder dialog has been modified to disable the enroll button if
>> there is no entries selected.
>>
>> Ticket #1856
> Pretty extensive change for such a small piece of functionailty. What is
> the rationale?
The solution is simple, but it requires refactoring which probably
should have been done much earlier. So this patch is like a combination
of several patches.
When creating a jQuery dialog we only specify the button labels and
their handlers. The dialog API doesn't give us access to the button's
DOM elements. In order to disable a button, we have to find the button
element within the dialog. However, these buttons do not have a name/id.
We can search by the label, but it's not ideal. So the code is matching
the button elements with button definitions based on their index.
To avoid repeating the search every time we need the button, the code is
storing the DOM element in the button object. The button objects are
stored in a map so they can be accessed by name.
Once we have the element, enabling/disabling the button is just a matter
of adding/removing a CSS class. We also need to be able to check whether
the button is enabled/disabled. These functionalities are encapsulated
in IPA.dialog_button class.
For consistency with the rest of the code, all button definitions using
add_button(label, handler) are converted into create_button(spec). This
is where the majority of the changes happen. The changes could have been
minimized by keeping a backward compatible interface, but it will
clutter the code and eventually they are going to be replaced by the new
interface anyway.
As a further enhancement, the IPA.dialog_button class probably can be
combined with the IPA.action_button/IPA.button class so we can use icons
or enable/disable buttons in a more consistent way. This is one step in
that direction.
--
Endi S. Dewata
More information about the Freeipa-devel
mailing list