Opened 4 years ago
Closed 13 months ago
#49182 closed enhancement (invalid)
Unmask password button
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | normal | Version: | 2.7 |
Component: | Posts, Post Types | Keywords: | close has-patch |
Focuses: | Cc: |
Description
Hello,
I would like to add an “eye” (connection protected pagefile) of visibility of the Password because many people without a numeric keypad Keyboard believe entering numbers while entering lowercase type [in French : é ”' (-è_çà] the fact that only * appear when connecting to protected pages (protected page file)
Regards
Attachments (2)
Change History (18)
#1
follow-up:
↓ 2
@
4 years ago
- Component changed from General to Posts, Post Types
- Keywords needs-patch good-first-bug added
- Version changed from 5.3.2 to 2.7
#2
in reply to:
↑ 1
@
4 years ago
Replying to johnbillion:
This functionality exists in a few places within the WordPress admin area. It should be possible to add it to the password protected post form too.
I've been looking into a patch, and adding the markup to get_the_password_form() in wp-includes/post-template.php looks to be relatively straightforward. However, it doesn't seem right to load the same Javascript functionality as is loaded for the admin views, such as the bindToggleButton() function in wp-admin/js/user-profile.js, since this will be for a front end post view.
For something like this, does it make sense to create a new script file in wp-includes/js/ ?
If the markup for the admin functionality is emulated, there's also the issue of dashicons not being enqueued by default on the front end (unless I'm mistaken), and whether or not we would want to add some default styling of the password toggle button similar to the admin style, as well.
#3
@
4 years ago
At @johnbillion and @apieschel.
Thank you for your quick responses.
"For something like this, does it make sense to create a new script file in wp-includes/js/"
I think this suggestion is a good idea.
Note: I am not a champion of JS scripts at all and and it would be better to have the file in its entirety.
Thanks you anywhere
#4
@
4 years ago
Hello, first time contributor here.
@bernardroux I'm trying to find out which password entry you'd like to see updated. I'm running 5.3.2 and wp-login.php has a password field with an eye button to toggle password visibility.
So, perhaps this enhancement has already been delivered? Or perhaps there's another place where the password field can be found in wordpress where you'd like to see the eye toggler.
Let me know if I need to clarify or if I have misunderstood the enhancement request.
-Carrigan
#5
@
4 years ago
Hello Carrigan,
When you have a publication (page) which is password protected, you need to fill the password field. It should be nice to have an eye button to toggle password visibility.
I don't speak for about the password field of the wp-login.php.
Thanks a lot
Bernard
#6
@
4 years ago
Thank you @bernardroux I'll take a look and see if I can't find where the publication password appears.
Thanks,
-Carrigan
#7
@
4 years ago
@bernardroux OK I found what you mean. The password on the published pages that are password protected. I think I understand the problem... Now to fix it!
#8
@
4 years ago
To Carrigan,
That exactly what I meaned.
Thank you for your consideration
I wait you fix it!
Thank's a lot
Bernard
@
4 years ago
updated post password frontend form field with show\hide password toggle, js\css files are included as well as modified get_the_password_form() and get_the_content() functions of /wp-includes/post-template.php
#9
follow-up:
↓ 10
@
4 years ago
Good day @bernardroux seeing no one in several weeks has proposed a patch for it I've done it.
See code in added__show_hide_toggle_to_password_field_of_password_protected_post_form_.patch
I've used trunk (5.5.0 as a base).
#10
in reply to:
↑ 9
;
follow-up:
↓ 11
@
4 years ago
Replying to mikhailroot:
Good day @bernardroux seeing no one in several weeks has proposed a patch for it I've done it.
See code in added__show_hide_toggle_to_password_field_of_password_protected_post_form_.patch
I've used trunk (5.5.0 as a base).
Hello @mikhailroot,
Thank you very much for your volunteer contribution.
As we are confined to France at home because of the Sars-cod 2 / Covid 19, I found the time to try to modify the different files offered.
- Currently I use mysql 5.6
- src/js/_enqueues/lib/wp-post-pwd.js, I found the folder "/js" in www/wp-incudes/js
- src/wp-includes/css/wp-post-pwd.css no problem to include it in the right folder
- src/wp-includes/post-template.php idem
- src/wp-includes/script-loader.php idem
- But I didn't find "Gruntfile.js" anywhere
Thanks if you can edit your files again so I can fix them.
Thank you in advance
Bernard Roux
@
4 years ago
Built js\css and updated PHP files meant to be merged into non development WordPress version.
#11
in reply to:
↑ 10
@
4 years ago
- Keywords needs-patch removed
The thing is patch is applied to SVN repository (trunk) and it has Gruntfile.js and a bit different sources file structure (as it's meant to run Grunt to build\copy css\js files into necessary folders via npm run build
).
See https://make.wordpress.org/core/handbook/tutorials/installing-wordpress-locally/ on how to install development version of WordPress which has files and directory structure I've used with this patch.
I've built files and here's zip file with this update which reside in wp-includes, zip is created from WordPress root directory (not a dev version of WordPress).
https://core.trac.wordpress.org/attachment/ticket/49182/built_files.zip
So you can merge files from this zip with your non development WordPress version.
Replying to bernardroux:
Replying to mikhailroot:
Good day @bernardroux seeing no one in several weeks has proposed a patch for it I've done it.
See code in added__show_hide_toggle_to_password_field_of_password_protected_post_form_.patch
I've used trunk (5.5.0 as a base).
Hello @mikhailroot,
Thank you very much for your volunteer contribution.
As we are confined to France at home because of the Sars-cod 2 / Covid 19, I found the time to try to modify the different files offered.
- Currently I use mysql 5.6
- src/js/_enqueues/lib/wp-post-pwd.js, I found the folder "/js" in www/wp-incudes/js
- src/wp-includes/css/wp-post-pwd.css no problem to include it in the right folder
- src/wp-includes/post-template.php idem
- src/wp-includes/script-loader.php idem
- But I didn't find "Gruntfile.js" anywhere
Thanks if you can edit your files again so I can fix them.
Thank you in advance
Bernard Roux
#13
@
4 years ago
Hello @mikhailroot,and Good easter
Thanks a lot for the new elements
I forgot to tell you that I have a web host called "OVHcloud" well known in Europe and I have access to my database either by "phpMyadmin" tools from my host or directly by FTP with Filezilla and can modify or create files on my database.
Also if I understood correctly I can directly insert or replace the modified "built_files" files in the appropriate folders without using gruntfile, (ie wp-includes - wp-includes/css - wp-includes/js)
Regards
Bernard Roux
#14
@
4 years ago
- Keywords close added; good-first-bug has-patch removed
Hello @mikhailroot
After several hesitations, I downloaded the different files to my database.
Excellent result see the example below
https://www.evab.fr/?page_id=3250.
We can close the ticket. (can you tell me how to do?)
Thank you for your help - see you soon
Спасибо за ваше сотрудничество. До скорой встречи, может быть!
#15
@
4 years ago
- Keywords has-patch added
Good day @bernardroux !
I'm happy it worked for you. Yes you can just replace files in your WordPress directory with ones from built_files.zip
The thing is these changes will be overwritten upon your next WordPress update. We need to wait so that patch will be agreed to be applied to WordPress Core by WordPress core team. So I consider ticket doesn't need to be closed, otherwise when it gets closed Core Team probably will not merge this update.
This functionality exists in a few places within the WordPress admin area. It should be possible to add it to the password protected post form too.