WordPress.org

Make WordPress Core

Opened 11 months ago

Last modified 7 months ago

#37484 new enhancement

Make form in request_filesystem_credentials() in line with WP admin form styling / HTML structure / security

Reported by: dashaluna Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: Upgrade/Install Keywords:
Focuses: ui, accessibility, administration Cc:

Description (last modified by SergeyBiryukov)

Currently, the Connection Information form output by request_filesystem_credentials() looks very sad and doesn't reflect the admin form HTML structure and style.

This is how it looks: Screen Shot 2016-07-27 at 13.41.52.png

It would be great if it can be standardised to look like WP admin settings UI, i.e. Screen Shot 2016-07-27 at 13.41.23.png

While at it, it would be great to secure the form code with escape family functions (esc_attr, esc_html and so on).

Attachments (2)

Screen Shot 2016-07-27 at 13.41.23.png (217.9 KB) - added by dashaluna 11 months ago.
Connection Information admin form
Screen Shot 2016-07-27 at 13.41.52.png (380.7 KB) - added by dashaluna 11 months ago.
General Settings admin form

Download all attachments as: .zip

Change History (6)

@dashaluna
11 months ago

Connection Information admin form

@dashaluna
11 months ago

General Settings admin form

#1 @swissspidy
11 months ago

  • Component changed from Formatting to Upgrade/Install

Related / possible duplicate of: #36973

#2 @dashaluna
11 months ago

Oops, sorry I got confused how to add images in the description and now I can't edit it :( Added images as attachments.

#3 @SergeyBiryukov
11 months ago

  • Description modified (diff)

#4 @joedolson
7 months ago

On the accessibility front, these fields should probably be changed to stop using implicit labels, but I don't see any other a11y issues from reviewing the code.

Note: See TracTickets for help on using tickets.