WordPress.org
Get WordPress

Make WordPress Core

Opened 5 years ago

Closed 5 years ago

#50622 closed defect (bug) (fixed)

Sitemap: add container div

Reported by: ramiy's profile ramiy Owned by: swissspidy's profile swissspidy
Milestone: 5.5 Priority: normal
Severity: normal Version:
Component: Sitemaps Keywords: has-patch commit
Focuses: Cc:

Description

I think we should add a container div to the sitemap. This was we can center the entire content area (margin: 0 auto) using a single CSS selector (#sitemap).

See the attached patch.

Attachments (2)

50622.patch​ (879 bytes) - added by ramiy 5 years ago.
50622.diff​ (4.5 KB) - added by swissspidy 5 years ago.

Download all attachments as: .zip

Change History (6)

@ramiy
5 years ago

#1 @ramiy
5 years ago

Try the patch with this css:

#sitemap {
  max-width: 980px;
  margin: 0 auto;
}

#sitemap__table {
  width: 100%;
}

#2 @SergeyBiryukov
5 years ago

  • Milestone changed from Awaiting Review to 5.5

@swissspidy
5 years ago

#3 @swissspidy
5 years ago

  • Keywords has-patch commit added

#4 @swissspidy
5 years ago

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

In 48471:

Sitemaps: Add wrapping <div> around sitemap in stylesheet.

This makes it easier to style the sitemap and for example center the entire content area.

Props ramiy.
Fixes #50622.

Note: See TracTickets for help on using tickets.