On 10/21/2010 06:48 AM, Jakub Steiner wrote:
I don't like this. The change has broken the login dialog (fields
way too narrow - 10em as opposed to 20em). The issue here is that .modalbox wrapper should
not be used for the edit page. It's only used for login and for self service creating
a new account by the user. Once logged in, the form should not be wrapped in .modalbox
cheers
This is not true. The 'new user' form is always wrapped in the modal box
regardless of whether or not it is the self-service case. eg even if an
admin is logged in creating a new user for someone else, the form is
currently wrapped in the modalbox. I don't see what the objection to
using the modalbox on the edit user form is, everything falls into place
after that, errors are not being displayed otherwise.
The real problem isn't that the form is in a modal box, its that the
input field size was set too small. Granted I'll admit I should've
considered other elements on the site affected by this change and simply
added a new style rule which just applies to the large buttons on the
edit user form (without the style change they are broken up into
multiple lines). But I do object to reverting my whole patch, especially
when I wasn't around to discuss this. Your new patch doesn't work in
solving the original problem, errors on the edit user form are now not
being displayed again.
https://bugzilla.redhat.com/show_bug.cgi?id=643121
Lets try to work together to come up with a solution that works for
everyone, perhaps through the combination of both our patches.
-Mo