#43066 closed enhancement (fixed)
Improve JS Documentation: password-strength-meter.js
| Reported by: | jjcomack | Owned by: | atimmer |
|---|---|---|---|
| Priority: | normal | Milestone: | 5.1 |
| Component: | Login and Registration | Version: | |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: | javascript, docs |
Description
In our ongoing effort to improve JavaScript documentation, this patch adds descriptions and JSDoc formatting to the following file:
password-strength-meter.js
Please give props to:
Attachments (3)
Change History (11)
#2
in reply to: ↑ 1
@
9 years ago
- Milestone Awaiting Review → 5.0
- Owner set to
- Status new → assigned
Replying to swissspidy:
Just noticed this on the last line of the patch. We spell it backward compatibility, not backwards compatibility.
I've added needs-refresh per the above @jjcomack, could you refresh the patch with this change please.
#3
follow-up:
↓ 4
@
9 years ago
Attached a newer version. Alas, I forgot to rename the file prior to uploading, so you need to look at the document-password-strength.diff file
#4
in reply to: ↑ 3
@
9 years ago
Replying to jjcomack:
Attached a newer version. Alas, I forgot to rename the file prior to uploading, so you need to look at the
document-password-strength.difffile
There's a few unrelated changes in the document-password-strength.diff patch @jjcomack, it looks like your repo needs updating from master
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Just noticed this on the last line of the patch. We spell it backward compatibility, not backwards compatibility.
See https://make.wordpress.org/core/handbook/glossary/