Skip to content

Clarify TOTP setup instructions#763

Merged
kasparsd merged 4 commits intoWordPress:masterfrom
masteradhoc:patch-5
Feb 13, 2026
Merged

Clarify TOTP setup instructions#763
kasparsd merged 4 commits intoWordPress:masterfrom
masteradhoc:patch-5

Conversation

@masteradhoc
Copy link
Contributor

Fixes #486

What?

The PR intends to improve user guidance on how to setup the TOTP.

Why?

The current setup text is quite basic and doesnt help unexperienced users enough to understand how the setup has to be done.

How?

After many contributors feedback in #486 this text is a draft to discuss.

Testing Instructions

  1. go to wp-admin/users.php
  2. choose your user and edit it
  3. see the TOTP setup text
  4. apply the PR
  5. check the new TOTP setup text

Screenshots or screencast

Current State
image

New Proposal
image

Changelog Entry

Changed - Clarify TOTP setup instructions

@github-actions
Copy link

github-actions bot commented Jan 31, 2026

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: masteradhoc <masteradhoc@git.wordpress.org>
Co-authored-by: kasparsd <kasparsd@git.wordpress.org>
Co-authored-by: jeffpaul <jeffpaul@git.wordpress.org>
Co-authored-by: pbiron <pbiron@git.wordpress.org>
Co-authored-by: r-a-y <r-a-y@git.wordpress.org>
Co-authored-by: dd32 <dd32@git.wordpress.org>
Co-authored-by: iandunn <iandunn@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@jeffpaul
Copy link
Member

jeffpaul commented Feb 9, 2026

I'd like to get @iandunn or @kasparsd input here as they've done some good work analyzing the prior states to continually improve this.

Copy link
Collaborator

@kasparsd kasparsd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Had one optional suggestion with regards to the QR placement.

The copy looks good. We can always iterate on it.

</li>
<li>
<?php esc_html_e( 'Scan this QR code using the app you installed:', 'two-factor' ); ?>
<?php /* QR image intentionally omitted. */ ?>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we actually place it here for semantics?

I believe list items <li> now support paragraph tags per HTML spec so we could add both the QR code and the follow-up paragraph (If scanning isn’t possible...) as list items here.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kasparsd can you give it another try? reworked and committed the changes.

@masteradhoc masteradhoc requested a review from kasparsd February 10, 2026 20:34
@kasparsd kasparsd merged commit 73b376c into WordPress:master Feb 13, 2026
4 checks passed
@masteradhoc masteradhoc deleted the patch-5 branch February 13, 2026 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clarify TOTP setup instructions

3 participants