Make WordPress Core

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#51970 closed defect (bug) (fixed)

App Passwords: Ensure description paragraphs aren't hidden on the Authorize Application screen

Reported by: timothyblynjacobs's profile TimothyBlynJacobs Owned by: timothyblynjacobs's profile TimothyBlynJacobs
Milestone: 5.6 Priority: normal
Severity: normal Version: 5.6
Component: Application Passwords Keywords: has-patch
Focuses: css, administration, rest-api Cc:

Description

On mobile devices, there is a css rule that seems to hide all paragraphs in the form. This causes the information about where the user is redirected to after approving the app to be hidden which poses a security issue.

@media screen and (max-width: 782px)
list-tables.css:1726.form-wrap > p {
display: none;
}

Attachments (2)

Before.png (200.3 KB) - added by TimothyBlynJacobs 4 years ago.
After.png (216.3 KB) - added by TimothyBlynJacobs 4 years ago.

Download all attachments as: .zip

Change History (5)

This ticket was mentioned in PR #799 on WordPress/wordpress-develop by TimothyBJacobs.


4 years ago
#1

  • Keywords has-patch added

#2 @TimothyBlynJacobs
4 years ago

  • Owner set to TimothyBlynJacobs
  • Resolution set to fixed
  • Status changed from new to closed

In 49772:

App Passwords: Ensure redirection description is shown on mobile devices.

Props #core.
Fixes #51970.

#3 @helen
4 years ago

In 49773:

App Passwords: Ensure redirection description is shown on mobile devices.

Props TimothyBlynJacobs.
Fixes #51970.
Merges [49772] to the 5.6 branch.

Note: See TracTickets for help on using tickets.