WordPress.org

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#33796 closed defect (bug) (fixed)

Install wordpress step 2 page 'Username' and 'Password' label's are not aligned to text

Reported by: umesh.nevase Owned by: SergeyBiryukov
Milestone: 4.4 Priority: normal
Severity: normal Version: 4.3
Component: Upgrade/Install Keywords: has-patch needs-testing
Focuses: ui, administration Cc:

Description

When we install wordpress, after filling all the details wordpress get installed and we redirected to step 2 page for login with sucess message 'WordPress has been installed. Were you expecting more steps? Sorry to disappoint.'

on this host/wp-admin/install.php?step=2 page the labels 'Username' and 'Password' are not aligned with the text in next td 'admin' and 'Your chosen password.' I've attached screenshot which clearly explains the issue.

Attachments (2)

install-step2-edited.png (25.3 KB) - added by umesh.nevase 4 years ago.
I draw line, which shows the label and text are not aligned.
33796.patch (1.2 KB) - added by umesh.nevase 4 years ago.
I've added patch file, which fixes the issue.

Download all attachments as: .zip

Change History (8)

@umesh.nevase
4 years ago

I draw line, which shows the label and text are not aligned.

@umesh.nevase
4 years ago

I've added patch file, which fixes the issue.

#1 @umesh.nevase
4 years ago

  • Keywords has-patch needs-testing added

#2 @SergeyBiryukov
4 years ago

  • Component changed from Administration to Upgrade/Install
  • Focuses accessibility rtl removed
  • Milestone changed from Awaiting Review to 4.4

#3 @SergeyBiryukov
4 years ago

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

In 34043:

Better align table headers with table cells on success screen after installing.

Props umesh.nevase.
Fixes #33796.

#4 follow-up: @umesh.nevase
4 years ago

Thanks @SergeyBiryukov,

It should be also added in install-rtl.css

#5 in reply to: ↑ 4 @SergeyBiryukov
4 years ago

Replying to umesh.nevase:

It should be also added in install-rtl.css

RTL and minified files are updated by our build tools: https://build.trac.wordpress.org/changeset/34011.

#6 @ryan
4 years ago

Here's this screen in the context of flow with Macnchrome:

https://make.wordpress.org/flow/tag/upgrade-install/#jp-carousel-2695

And iPhone 6+:

https://make.wordpress.org/flow/tag/upgrade-install/#jp-carousel-2714

This ticket could use a fresh screenshot showing this screen on a phone, but I'll refrain from tagging needs-screenshots for now.

Note: See TracTickets for help on using tickets.