Make WordPress Core

Opened 10 years ago

Closed 10 years ago

#29336 closed defect (bug) (fixed)

Error messages page does not have the viewport meta tag

Reported by: rellect's profile rellect Owned by: rachelbaker's profile rachelbaker
Milestone: 4.3 Priority: normal
Severity: normal Version: 3.8
Component: Comments Keywords: has-patch
Focuses: ui Cc:

Description

Try to send an empty comment, you will get an error that you need to fill the fields. This error page does not have the viewport meta tag, which means this page don't have a responsive style in mobile.

This is tested in 4.0 beta.

Attachments (1)

29336.diff (626 bytes) - added by leogopal 10 years ago.
Adds the viewport meta tag for mobile assistance.

Download all attachments as: .zip

Change History (8)

@leogopal
10 years ago

Adds the viewport meta tag for mobile assistance.

#1 @leogopal
10 years ago

  • Keywords has-patch added

#2 @rachelbaker
10 years ago

  • Owner set to rachelbaker
  • Status changed from new to reviewing

#3 follow-up: @johnbillion
10 years ago

  • Milestone changed from Awaiting Review to 4.3

How strange. The viewport meta tag got missed in r26299 despite it being in the patch on #25956.

@iammattthomas Was there a reason for this or did it just get missed?

#4 @johnbillion
10 years ago

  • Version changed from 3.9.2 to 3.8

#5 in reply to: ↑ 3 @iammattthomas
10 years ago

Replying to johnbillion:

How strange. The viewport meta tag got missed in r26299 despite it being in the patch on #25956.

@iammattthomas Was there a reason for this or did it just get missed?

Hrm, hard to say, given how long ago it was—but given my commit message in r26299 it looks like it was just an oversight on my part:

Update error screens with the new visual design:

  • Bring in new styles for backgrounds/text/link/borders/buttons
  • Switch to device-width for the viewport and tweak margins etc so the layout works nicely on all device sizes
Last edited 10 years ago by iammattthomas (previous) (diff)

#6 @rachelbaker
10 years ago

  • Status changed from reviewing to accepted

Let's fix in 4.3, patch looks good.

Last edited 10 years ago by rachelbaker (previous) (diff)

#7 @johnbillion
10 years ago

  • Resolution set to fixed
  • Status changed from accepted to closed

In 32501:

Add a viewport meta tag to wp_die().

Props leogopal

Fixes #29336

Note: See TracTickets for help on using tickets.