Make WordPress Core

Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#33796 closed defect (bug) (fixed)

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

Reported by: umeshnevase's profile umesh.nevase Owned by: sergeybiryukov's profile 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 10 years ago.
I draw line, which shows the label and text are not aligned.
33796.patch (1.2 KB) - added by umesh.nevase 10 years ago.
I've added patch file, which fixes the issue.

Download all attachments as: .zip

Change History (8)

@umesh.nevase
10 years ago

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

@umesh.nevase
10 years ago

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

#1 @umesh.nevase
10 years ago

  • Keywords has-patch needs-testing added

#2 @SergeyBiryukov
10 years ago

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

#3 @SergeyBiryukov
10 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
10 years ago

Thanks @SergeyBiryukov,

It should be also added in install-rtl.css

#5 in reply to: ↑ 4 @SergeyBiryukov
10 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
10 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.