Opened 6 years ago
Closed 6 years ago
#50658 closed enhancement (fixed)
Sitemap: center the container
| Reported by: | ramiy | Owned by: | SergeyBiryukov |
|---|---|---|---|
| Priority: | lowest | Milestone: | 5.6 |
| Component: | Sitemaps | Version: | 5.5 |
| Severity: | normal | Keywords: | has-patch has-screenshots commit |
| Cc: | Focuses: |
Description
Following #50622, now that we have a container div, let's center the sitemap content.
See the attached screenshots and the patch.
Attachments (4)
Change History (16)
This ticket was mentioned in Slack in #core-sitemaps by swissspidy. View the logs.
6 years ago
This ticket was mentioned in Slack in #core by meaganhanes. View the logs.
6 years ago
#6
in reply to: ↑ 4
@
6 years ago
Replying to pacifika:
Could you expand on the rationale for this change please?
Because not only bots use xml-sitemaps, humans are checking them too. This is why we have a sitemaps stylesheet in the first place, to style the page for regular people. And like all content on the web consumed by humans, it should be visually centered. Or am I wrong?
This ticket was mentioned in Slack in #core-sitemaps by ramiy. View the logs.
6 years ago
This ticket was mentioned in Slack in #core by laura. View the logs.
6 years ago
#9
@
6 years ago
5.6 Beta 1 is less than 2 weeks away. @swissspidy is this ticket on your radar to land in 5.6? If yes, what's the next step to move it forward?
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Could you expand on the rationale for this change please?