On Fri, 28 Mar 2014 14:04:13 +0100 Petr Vobornik <[email protected]> wrote:
> Attached patches replace IPA.unauthorized dialog with new Login > Screen. > > To make it happen, a support for standalone facets had to be > developed because current framework was limited by facets dependent > on entities and a container with menu. This new feature was already > used for Load facet which is part of this patchset and also will be a > basis for API browser and OTP sync page. > > Patches should fix these tickets: > https://fedorahosted.org/freeipa/ticket/3903 > https://fedorahosted.org/freeipa/ticket/4017 > > Depends on patches #565-#568. > > > [PATCH] webui: facet container > ------------------------------ > A widget which servers as container for facets. FacetContainer is a > base class. App is specialization. > > Doing this abstraction will allow us to implement various facet > containers. > > [PATCH] webui: FormMixin > ------------------------ > a mixin used for fields validation. Basically implements a logic which > is already in details facet and dialog. > > Now this logic can be used in any component. > > The long term goal is to replace the logic in details facet and dialog > with this mixin. > > [PATCH] webui: ContainerMixin > ----------------------------- > A mixin which implements widget storing logic. Similar logic is > already implemented > in details facet and dialog. > > Long term goal is to replace that with this one. > > Separating the logic into mixin makes it usable in other components. > > [PATCH] webui: standalone facet > ------------------------------- > `facet.Facet` is a new base class for facets. It doesn't have any > dependencies > on entities so it's usable for general purpose facets, e.g., future > API browser, > load facet or login facet. > > [PATCH] webui: activity widget > ------------------------------ > A widget for showing ongoing activity. > > Displays a text with changing dots. > > It listens to `network-activity-start` and `network-activity-end` > topics. > > [PATCH] webui: publish network activity topics > ---------------------------------------------- > Network activity is now published through global topics. It allows > other components like activity_widget to listen to them. > > [PATCH] webui: load page > ------------------------ > Load page is a simple facet which is displayed up to 'runtime' phase. > > On application start it tells the user that there is ongoing activity. > > [PATCH] webui: validation summary widget > ---------------------------------------- > A widget which aggregates warnings and errors and shows them on one > place. > > [PATCH] webui: login screen widget > ---------------------------------- > Reimplementation of unauthorized dialog into separate widget. It uses > RCUE design. > > New features compared to unauthorized dialog: > > - reflects auth methods from `auth` module > - validation summary > - differentiates Kerberos auth failure with session expiration > - Caps Lock warning > - form based method doesn't allow password only submission > > https://fedorahosted.org/freeipa/ticket/4017 > https://fedorahosted.org/freeipa/ticket/3903 > > > [PATCH] webui: login page > ------------------------- > A facet with login sreen widget. > > [PATCH] webui: authentication module > ------------------------------------ > General purpose authentication interface and state. See doc of > 'freeipa/auth' module. > > [PATCH] webui: use asynchronous call for authentication > > Change `IPA.login_password` and `IPA.get_credentials` to use async > AJAX and to return promise instead of blocking the code. > > IPA.get_credentials is still partially blocking because of negotiate > process. > We can't do anything about that. > > It allows activity indicators to do their job. > > [PATCH] webui: fix combobox styles to work with selenium testing > [PATCH] webui-ci: adapt to new login screen > [PATCH] webui: remove IPA.unauthorized_dialog Hi, - Attached patch fixes weird combobox behaviour - opens automatically on facet load - When trying to log in with password only(username field is empty), there is an error message "Authentication with Kerberos failed", which is not the desired behaviour. It should sign that the username field is invalid. - When trying to log in with kerberos credentials, and the realm of the krb ticket is not the same as the realm of freeipa(eg freeipa realm is IPA.TEST.COM, and the ticket's is TEST.COM), firefox goes into an endless cycle calling the kerberos auth url. Currently it seems to me as a browser issue. Anyways, with correct krb ticket, authentication works fine. Although, unit tests ran, integration tests ran as expected, and browsing through the code manually was ok for me, so if that validation issue is corrected, than it will be an ACK. Thanks: Adam
>From 633b162c414ecc8c156d90d5c4c1860b1e418288 Mon Sep 17 00:00:00 2001 From: Adam Misnyovszki <[email protected]> Date: Fri, 4 Apr 2014 16:47:15 +0200 Subject: [PATCH] combobox widget fix this patch fixes the combobox widget, so it doesn't open on page load, just when the user clicks on it. --- install/ui/src/freeipa/widget.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/install/ui/src/freeipa/widget.js b/install/ui/src/freeipa/widget.js index 025d6d1f3375efae1df55a026b08197b18f14359..568696b5b7a3f74572d6102e0cfb454101bb1b7c 100644 --- a/install/ui/src/freeipa/widget.js +++ b/install/ui/src/freeipa/widget.js @@ -3305,7 +3305,7 @@ IPA.combobox_widget = function(spec) { that.list_container = $('<div/>', { 'class': 'combobox-widget-list', - css: { 'z-index': that.z_index }, + css: { 'z-index': that.z_index, 'display':'none' }, keydown: that.on_list_container_keydown }).appendTo(that.input_container); -- 1.9.0
_______________________________________________ Freeipa-devel mailing list [email protected] https://www.redhat.com/mailman/listinfo/freeipa-devel
