WordPress.org

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#13832 closed defect (bug) (invalid)

wp-signup.php produces invalid html with default themes

Reported by: nkuttler Owned by:
Milestone: Priority: normal
Severity: normal Version: 2.9.2
Component: Multisite Keywords: invalid html
Focuses: Cc:

Description

When you visit /wp-signup.php the created output uses the current theme and emebds a <div> with id="content".

As <div id="content"> is already used by twentyten, kubrick, and probably the majority of themes out there this will produce invalid markup in many installs.

If #content is already styled the page layout will easily break.

Simply renaming the id to mucontent would fix this, although it's not clear to me why the div needs an id.

Attachments (1)

diff.patch (363 bytes) - added by nkuttler 4 years ago.
Trivial patch

Download all attachments as: .zip

Change History (3)

nkuttler4 years ago

Trivial patch

comment:1 nkuttler4 years ago

  • Resolution set to invalid
  • Status changed from new to closed

The initial description is incorrect, none of the current or former default themes produce invalid markup.

As a theme developer I was surprised to see that themes apparently have to use a <div id="content"> for content, and that they can't put that <div> into header.php. I'll look into updating the codex.

comment:2 westi4 years ago

  • Milestone 3.1 deleted
Note: See TracTickets for help on using tickets.