WordPress.org

Make WordPress Core

Opened 21 months ago

Last modified 21 months ago

#37848 new enhancement

Form fields misaligned on export.php

Reported by: adhun Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: Export Keywords: has-patch needs-refresh has-screenshots
Focuses: ui, administration Cc:

Description

On the export page, all the form fields are misaligned. Refer attached screenshots for a better view.

Attachments (3)

Before-patch.png (56.1 KB) - added by adhun 21 months ago.
Current export screen
After patch.png (63.3 KB) - added by adhun 21 months ago.
After Patch
patch.diff (8.2 KB) - added by adhun 21 months ago.
Patch

Download all attachments as: .zip

Change History (8)

@adhun
21 months ago

Current export screen

@adhun
21 months ago

After Patch

@adhun
21 months ago

Patch

#1 @swissspidy
21 months ago

  • Component changed from Administration to Export
  • Focuses template removed
  • Keywords has-patch needs-refresh has-screenshots added
  • Version trunk deleted

Hey there,

Thanks for opening this ticket and contributing a patch.

As you've added the accessibility focus, I had a quick look at the patch in that regard. The patch removes the <label>s in a few places, in some other places the labels are missing the for attribute. This needs to be fixed.

There's a great handbook entry for accessible HTML: https://make.wordpress.org/accessibility/handbook/quick-start-guide/#forms

#2 @afercia
21 months ago

I'd recommend to avoid to use a layout table just for the sake of a slightly better visual alignment. There are also a couple of long standing tickets related to accessibility improvements for the settings pages and forms in the admin, where the proposed goal is to remove the layout tables, see #16413 and #18801. Unfortunately there hasn't been so much movement on those tickets, but in the meanwhile it would be better to don't introduce new layout tables.

This ticket was mentioned in Slack in #accessibility by afercia. View the logs.


21 months ago

#4 @afercia
21 months ago

  • Focuses accessibility removed

Going to remove the accessibility focus to get this ticket out from the a11y report but will keep following. Thinking any enhancement should be done together with other similar admin screens, see #16413 and #18801 (though the export screen is not exactly a "Settings page").

#5 @Cheffheid
21 months ago

  • Type changed from defect (bug) to enhancement
Note: See TracTickets for help on using tickets.