Re: [Freeipa-devel] [PATCH] 158 Separate reset password page
On 06/20/2012 07:00 PM, Endi Sukma Dewata wrote: On 6/19/2012 1:22 AM, Petr Vobornik wrote: Updated patch attached. Sorry for late update, I was working on trust UI and didn't want to switch context. No problem. It would be nicer if this page can use the same fonts as the login dialog in patch #157, but that can be fixed separately. ACK. Pushed to master. Fonts: https://fedorahosted.org/freeipa/ticket/2861 -- Petr Vobornik ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
Re: [Freeipa-devel] [PATCH] 158 Separate reset password page
On 6/19/2012 1:22 AM, Petr Vobornik wrote: Updated patch attached. Sorry for late update, I was working on trust UI and didn't want to switch context. No problem. It would be nicer if this page can use the same fonts as the login dialog in patch #157, but that can be fixed separately. ACK. -- Endi S. Dewata ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
Re: [Freeipa-devel] [PATCH] 158 Separate reset password page
On 06/13/2012 07:22 PM, Endi Sukma Dewata wrote:
On 6/13/2012 8:32 AM, Petr Vobornik wrote:
On 06/13/2012 01:25 AM, Endi Sukma Dewata wrote:
On 6/8/2012 10:54 AM, Petr Vobornik wrote:
This patch adds separate reset password page. It is a complement to
separate login page. It differentiate from reset password capabilities
in Web UI's anauthorized dialog by not performing login. This is useful
for users who wants only to reset the password and not to use Web UI.
And also for users who are using the separate login page.
https://fedorahosted.org/freeipa/ticket/2755
It also works with mkosek 274-2 (but needs rebase).
If the password reset is successful, the page shows a link to the login
page, but it also still shows the password reset form. I think it's very
unlikely that the user will want to change the password again, so it
might be better not to show the form, but provide a link to the form
just in case the user needs to do it again.
So after successful reset user will remain on reset_password.html and
will see something like this:
http://pvoborni.fedorapeople.org/ux/resetpwd_success.png ?
Yes, I think it looks much cleaner. Also make sure the password fields
are cleaned too.
Updated patch attached. Sorry for late update, I was working on trust UI
and didn't want to switch context.
--
Petr Vobornik
From 8518b770f9f44f4388403f063b16dd11f4b8fb0a Mon Sep 17 00:00:00 2001
From: Petr Vobornik pvobo...@redhat.com
Date: Fri, 8 Jun 2012 16:38:17 +0200
Subject: [PATCH] Separate reset password page
This patch adds separate reset password page. It is a complement to separate login page. It differentiate from reset password capabilities in Web UI's anauthorized dialog by not performing login. This is useful for users who wants only to reset the password and not to use Web UI. And also for users who are using the separate login page.
https://fedorahosted.org/freeipa/ticket/2755
---
freeipa.spec.in|5 +
install/ui/Makefile.am |2 +
install/ui/ipa.css | 15 -
install/ui/jsl.conf|1 +
install/ui/login.html |3 +-
install/ui/reset_password.html | 66
install/ui/reset_password.js | 163
7 files changed, 252 insertions(+), 3 deletions(-)
create mode 100644 install/ui/reset_password.html
create mode 100644 install/ui/reset_password.js
diff --git a/freeipa.spec.in b/freeipa.spec.in
index ce8e1e69fa0953013f06736394800e8797883323..1d70a7c58cc1235d77c44400f265ec4c59f7b348 100644
--- a/freeipa.spec.in
+++ b/freeipa.spec.in
@@ -585,6 +585,7 @@ fi
%{_usr}/share/ipa/ui/index.html
%{_usr}/share/ipa/ui/login.html
%{_usr}/share/ipa/ui/logout.html
+%{_usr}/share/ipa/ui/reset_password.html
%{_usr}/share/ipa/ui/*.ico
%{_usr}/share/ipa/ui/*.css
%{_usr}/share/ipa/ui/*.js
@@ -712,6 +713,10 @@ fi
%ghost %attr(0644,root,apache) %config(noreplace) %{_sysconfdir}/ipa/ca.crt
%changelog
+
+* Fri Jun 18 2012 Petr Vobornik pvobo...@redhat.com - 2.99.0-33
+- Add Web UI reset password page
+
* Fri Jun 8 2012 Martin Kosek mko...@redhat.com - 2.99.0-32
- Add directory /var/lib/ipa/sysupgrade for package upgrade metadata
- Set min for bind-dyndb-ldap to 1.1.0-0.12.rc1 to pick up persistent search
diff --git a/install/ui/Makefile.am b/install/ui/Makefile.am
index e4256de913b7d178c529390732075c183a14f247..ea3a2029535c4ca73318779de2579ee6034bca95 100644
--- a/install/ui/Makefile.am
+++ b/install/ui/Makefile.am
@@ -53,6 +53,8 @@ app_DATA =\
overpass_regular-web.ttf \
overpass_regular-web.woff \
policy.js \
+ reset_password.js \
+ reset_password.html \
rule.js \
search.js \
selinux.js \
diff --git a/install/ui/ipa.css b/install/ui/ipa.css
index c69fc80ded3dfef8a9fa645dbe74ac96d069f73a..2ce8494ab8e49154c6ab7b82e6e76bdb956b6d4d 100644
--- a/install/ui/ipa.css
+++ b/install/ui/ipa.css
@@ -1569,6 +1569,10 @@ span.sshkey-status, a.sshkey-set {
width: 450px;
}
+.form-box {
+padding: 1em 2em;
+}
+
#error-box, .error-box {
-moz-border-radius: 0.3em 0.3em 0.3em 0.3em;
-webkit-border-radius: 0.3em 0.3em 0.3em 0.3em;
@@ -1578,6 +1582,15 @@ span.sshkey-status, a.sshkey-set {
padding: 1em 2em;
}
+#success-box, .success-box {
+-moz-border-radius: 0.3em 0.3em 0.3em 0.3em;
+-webkit-border-radius: 0.3em 0.3em 0.3em 0.3em;
+background-color: #FFEB80;
+border: 1px solid #FFDE2E;
+margin: 0 2em 1em 2em;
+padding: 1em 2em;
+}
+
#formwindow h2 {
background-color: #F0F0F0;
font-size: 1.6em;
@@ -1619,7 +1632,7 @@ span.sshkey-status, a.sshkey-set {
box-shadow: inset 0 1px 1px rgba(0, 0, 0, 0.2);
margin-left: 15px;
padding: 2px 10px;
-width: 248px;
+width: 210px;
}
#login li label, #modal li label {
diff --git a/install/ui/jsl.conf b/install/ui/jsl.conf
index 3eebda4e8f6f6cb9ed9b3b502feec3558d723bc4..97789b2f08e96b06f94c4d212df7122fb7dff49d 100644
--- a/install/ui/jsl.conf
+++
Re: [Freeipa-devel] [PATCH] 158 Separate reset password page
On 06/13/2012 01:25 AM, Endi Sukma Dewata wrote: On 6/8/2012 10:54 AM, Petr Vobornik wrote: This patch adds separate reset password page. It is a complement to separate login page. It differentiate from reset password capabilities in Web UI's anauthorized dialog by not performing login. This is useful for users who wants only to reset the password and not to use Web UI. And also for users who are using the separate login page. https://fedorahosted.org/freeipa/ticket/2755 It also works with mkosek 274-2 (but needs rebase). If the password reset is successful, the page shows a link to the login page, but it also still shows the password reset form. I think it's very unlikely that the user will want to change the password again, so it might be better not to show the form, but provide a link to the form just in case the user needs to do it again. So after successful reset user will remain on reset_password.html and will see something like this: http://pvoborni.fedorapeople.org/ux/resetpwd_success.png ? -- Petr Vobornik ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
Re: [Freeipa-devel] [PATCH] 158 Separate reset password page
On 6/13/2012 8:32 AM, Petr Vobornik wrote: On 06/13/2012 01:25 AM, Endi Sukma Dewata wrote: On 6/8/2012 10:54 AM, Petr Vobornik wrote: This patch adds separate reset password page. It is a complement to separate login page. It differentiate from reset password capabilities in Web UI's anauthorized dialog by not performing login. This is useful for users who wants only to reset the password and not to use Web UI. And also for users who are using the separate login page. https://fedorahosted.org/freeipa/ticket/2755 It also works with mkosek 274-2 (but needs rebase). If the password reset is successful, the page shows a link to the login page, but it also still shows the password reset form. I think it's very unlikely that the user will want to change the password again, so it might be better not to show the form, but provide a link to the form just in case the user needs to do it again. So after successful reset user will remain on reset_password.html and will see something like this: http://pvoborni.fedorapeople.org/ux/resetpwd_success.png ? Yes, I think it looks much cleaner. Also make sure the password fields are cleaned too. -- Endi S. Dewata ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
Re: [Freeipa-devel] [PATCH] 158 Separate reset password page
On 6/8/2012 10:54 AM, Petr Vobornik wrote: This patch adds separate reset password page. It is a complement to separate login page. It differentiate from reset password capabilities in Web UI's anauthorized dialog by not performing login. This is useful for users who wants only to reset the password and not to use Web UI. And also for users who are using the separate login page. https://fedorahosted.org/freeipa/ticket/2755 It also works with mkosek 274-2 (but needs rebase). If the password reset is successful, the page shows a link to the login page, but it also still shows the password reset form. I think it's very unlikely that the user will want to change the password again, so it might be better not to show the form, but provide a link to the form just in case the user needs to do it again. -- Endi S. Dewata ___ Freeipa-devel mailing list Freeipa-devel@redhat.com https://www.redhat.com/mailman/listinfo/freeipa-devel
[Freeipa-devel] [PATCH] 158 Separate reset password page
This patch adds separate reset password page. It is a complement to
separate login page. It differentiate from reset password capabilities
in Web UI's anauthorized dialog by not performing login. This is useful
for users who wants only to reset the password and not to use Web UI.
And also for users who are using the separate login page.
https://fedorahosted.org/freeipa/ticket/2755
--
Petr Vobornik
From 597c567114d9913acb3cad0924b1eeba6d367573 Mon Sep 17 00:00:00 2001
From: Petr Vobornik pvobo...@redhat.com
Date: Fri, 8 Jun 2012 16:38:17 +0200
Subject: [PATCH] Separate reset password page
This patch adds separate reset password page. It is a complement to separate login page. It differentiate from reset password capabilities in Web UI's anauthorized dialog by not performing login. This is useful for users who wants only to reset the password and not to use Web UI. And also for users who are using the separate login page.
https://fedorahosted.org/freeipa/ticket/2755
---
freeipa.spec.in|4 +
install/ui/Makefile.am |2 +
install/ui/ipa.css | 11 ++-
install/ui/jsl.conf|1 +
install/ui/login.html |3 +-
install/ui/{login.html = reset_password.html} | 27 +++--
install/ui/reset_password.js | 136
7 files changed, 170 insertions(+), 14 deletions(-)
copy install/ui/{login.html = reset_password.html} (56%)
create mode 100644 install/ui/reset_password.js
diff --git a/freeipa.spec.in b/freeipa.spec.in
index de93aecb6142f73528e6aefe89d6bfcb48fc036f..ac61ce8464ddaae84ced189d5e6802a8d500fbb6 100644
--- a/freeipa.spec.in
+++ b/freeipa.spec.in
@@ -564,6 +564,7 @@ fi
%{_usr}/share/ipa/ui/index.html
%{_usr}/share/ipa/ui/login.html
%{_usr}/share/ipa/ui/logout.html
+%{_usr}/share/ipa/ui/reset_password.html
%{_usr}/share/ipa/ui/*.ico
%{_usr}/share/ipa/ui/*.css
%{_usr}/share/ipa/ui/*.js
@@ -684,6 +685,9 @@ fi
%ghost %attr(0644,root,apache) %config(noreplace) %{_sysconfdir}/ipa/ca.crt
%changelog
+* Fri Jun 8 2012 Petr Vobornik pvobo...@redhat.com - 2.99.0-30
+- Add Web UI reset password page
+
* Fri May 11 2012 Martin Kosek mko...@redhat.com - 2.99.0-29
- Replace used DNS client library (acutil) with python-dns
diff --git a/install/ui/Makefile.am b/install/ui/Makefile.am
index e4256de913b7d178c529390732075c183a14f247..ea3a2029535c4ca73318779de2579ee6034bca95 100644
--- a/install/ui/Makefile.am
+++ b/install/ui/Makefile.am
@@ -53,6 +53,8 @@ app_DATA =\
overpass_regular-web.ttf \
overpass_regular-web.woff \
policy.js \
+ reset_password.js \
+ reset_password.html \
rule.js \
search.js \
selinux.js \
diff --git a/install/ui/ipa.css b/install/ui/ipa.css
index 5371070a1c875c0bab4d058eebc5a353df29a79f..1793e7f75616332417a0c4b1822b9dc75b51e862 100644
--- a/install/ui/ipa.css
+++ b/install/ui/ipa.css
@@ -1578,6 +1578,15 @@ span.sshkey-status, a.sshkey-set {
padding: 1em 2em;
}
+#success-box, .success-box {
+-moz-border-radius: 0.3em 0.3em 0.3em 0.3em;
+-webkit-border-radius: 0.3em 0.3em 0.3em 0.3em;
+background-color: #FFEB80;
+border: 1px solid #FFDE2E;
+margin: 0 2em 1em 2em;
+padding: 1em 2em;
+}
+
#formwindow h2 {
background-color: #F0F0F0;
font-size: 1.6em;
@@ -1619,7 +1628,7 @@ span.sshkey-status, a.sshkey-set {
box-shadow: inset 0 1px 1px rgba(0, 0, 0, 0.2);
margin-left: 15px;
padding: 2px 10px;
-width: 248px;
+width: 210px;
}
#login li label, #modal li label {
diff --git a/install/ui/jsl.conf b/install/ui/jsl.conf
index 3eebda4e8f6f6cb9ed9b3b502feec3558d723bc4..97789b2f08e96b06f94c4d212df7122fb7dff49d 100644
--- a/install/ui/jsl.conf
+++ b/install/ui/jsl.conf
@@ -159,3 +159,4 @@
+process automember.js
+process webui.js
+process login.js
++process reset_password.js
diff --git a/install/ui/login.html b/install/ui/login.html
index 9902466a70b3deb7e4c7ec5168abc803935d0c32..69e3dea7f0f09906a7586c745e5df8c326803580 100644
--- a/install/ui/login.html
+++ b/install/ui/login.html
@@ -30,7 +30,7 @@
div id=expired class=error-box style=display:none
pstrongPassword expired/strong/p
-pPlease run kinit to reset the password and then try to login again./p
+pPlease a href=reset_password.htmlreset the password/a and then try to login again./p
pIf the problem persists, contact your administrator./p
/div
@@ -52,7 +52,6 @@
/form
/div
/div
-
/body
/html
diff --git a/install/ui/login.html b/install/ui/reset_password.html
similarity index 56%
copy from install/ui/login.html
copy to install/ui/reset_password.html
index 9902466a70b3deb7e4c7ec5168abc803935d0c32..af191cc0780fec6ee023b8dd16626b5196bf1c5d 100644
--- a/install/ui/login.html
+++ b/install/ui/reset_password.html
@@ -8,7 +8,7 @@
script
