Task #1193
closed
Redesign displaying messages for password policies
100%
Description
I've set up password policy for production use. The password policy rules are so long that I need to scroll to see form for password change.
Some messages are absolutely pointless (Show password policy name to users).
Most others overlap or duplicate different message.
The best solution for me will be a possibility to show message from new property instead of this.
The accuracy of the message will be responsibility of administrator.
Files
Related issues
Updated by Radek Tomiška over 5 years ago
- Subject changed from Fix display of password policies to Redesign displaing messages for password policies
Updated by Petr Michal over 5 years ago
- unnecessary points will be deleted: policy name, info about form, etc
- duplications will be merged - special characters, enhanced control infromations, etc
- all points will be merged into one box
- box will be displayed under some help button
Updated by Patrik Stloukal over 5 years ago
- Assignee changed from Ondřej Kopr to Patrik Stloukal
Updated by Patrik Stloukal over 5 years ago
- File pwdPreValidation.png pwdPreValidation.png added
- File pwdPreValidation2.png pwdPreValidation2.png added
- Subject changed from Redesign displaing messages for password policies to Redesign displaying messages for password policies
- Status changed from New to Needs feedback
- Assignee changed from Patrik Stloukal to Radek Tomiška
- % Done changed from 0 to 80
So i redesign prevalidation messages, i fixed result code of prevalidation answer, deleted two lines (name of pwd policies and min rules to fulfil), then i redesign other rules to popup panel into one alert (bubble).
I created new component PasswordPreValidation, for easier use of this new feature.
I added this new component to pwd change, create new user and public pwd change.
Only i am not sure if anyone will notice this link, because it is blue on blue and it is almost as normal text... maybe add underline or different color?
Please look at this implementation and if you could send me some ideas, i'd be glad.
commits: https://github.com/bcvsolutions/CzechIdMng/commit/02aab236ec5fc50aa7eb6fe6c188cba2752da14c
https://github.com/bcvsolutions/CzechIdMng/commit/000f03703b6efb0e4388d27f6dd7e47c1e597765
https://github.com/bcvsolutions/CzechIdMng/commit/ad2d3524cc089b0945e4a7570945bb86a432a044
Updated by Radek Tomiška over 5 years ago
- Related to Task #1227: Allow use characters bases in password validation added
Updated by Radek Tomiška over 5 years ago
- Status changed from Needs feedback to In Progress
- Assignee changed from Radek Tomiška to Patrik Stloukal
I did test, review and refactoring (layout), notes consulted personally, summary:
- added prevalidation component will be removed (only one validation component will be used)
- raw html used in component will be refactored to react (usage of raw html was in component before, but i noticed it now)
- forbidden characters base will be shown in prevalidation (because is validated)
- Special characters (the same for forbidden characters) will be shown as one item in rule list (~li).
Updated by Patrik Stloukal over 5 years ago
- File preValidation.png preValidation.png added
- Status changed from In Progress to Needs feedback
- Assignee changed from Patrik Stloukal to Radek Tomiška
- % Done changed from 80 to 90
ok so I resolved issues from last feedback, I removed raw html even in validation message.
Special and forbidden characters have different design for one pwd policy and more password policies, because merge especially for special chars is not ideal, so in this more pwd policies scenario there is list of special chars baseses and name of it's pwd policy. (preValidation.png)
Please look on this modified solution, I think it is now nice feature :D
commit: https://github.com/bcvsolutions/CzechIdMng/commit/1fc5f243b564bb83774c90e970cc6808c2bf956d
https://github.com/bcvsolutions/CzechIdMng/commit/ad476548bd9ceb285cffa4ef500df2a374cdc6cd
https://github.com/bcvsolutions/CzechIdMng/commit/08aced3cb8670a9ab5b541362d5316e2a568477f
Updated by Radek Tomiška over 5 years ago
- Status changed from Needs feedback to Closed
- Assignee changed from Radek Tomiška to Patrik Stloukal
- Target version set to Morganite (9.2.0)
- % Done changed from 90 to 100
I like it, thx! It is much more nicer.
I merged it into develop.