Feeling Lucky
?
?
[#27425] - [4.0] Toggle Password Visibility
- Fixed in Code Base
- 7 Jan 2020
- Medium
- Build: 4.0-dev
- # 27425
- Diff
- brianteeman:toggle
- Pending Hound Hound is busy sniffing around... Details
User tests: Successful: Unsuccessful:
Now that the code has been fixed with #27412 I can update the strings to be more meaningful.
The strings for Screen Readers are now "Show Password" and "Hide Password"
Testing Instructions
Make sure you are the current codebase (ie you have #27412)
Apply this PR
Update the javascript with
npm i
Inspect the password field in the admin user manager, admin login, site login
The strings for Screen Readers are now "Show Password" and "Hide Password"
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration Language & Strings Modules JavaScript Repository NPM Change Layout Front End |
jwaisner
- comment
- 7 Jan 2020
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27425.
| Status | Pending | ⇒ | Ready to Commit |
Quy
- comment
- 7 Jan 2020
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27425.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2020-01-07 10:26:08 |
| Closed_By | ⇒ | wilsonge | |
| Labels |
Added:
?
?
|
||
wilsonge
- comment
- 7 Jan 2020
Thanks!
brianteeman
- comment
- 7 Jan 2020
w00t
I have tested this item✅ successfully on 6220a78
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/27425.