Page MenuHomePhabricator

Page issues clashing with breadcrumbs
Closed, ResolvedPublic

Event Timeline

Jdlrobson raised the priority of this task from to Needs Triage.
Jdlrobson updated the task description. (Show Details)
Jdlrobson added a project: MobileFrontend.
Jdlrobson moved this task to Backlog on the MobileFrontend board.
Jdlrobson added a subscriber: Jdlrobson.
Jdlrobson set Security to None.
Jdlrobson triaged this task as Medium priority.Oct 8 2015, 5:15 PM

@Jdlrobson Is there another page you can point to with this issue? That link doesn't seem to cause the problem anymore since Joaquin's last update to the page.

@Jdlrobson Is there another page you can point to with this issue? That link doesn't seem to cause the problem anymore since Joaquin's last update to the page.

Nevermind. Realized I can repro this with the old revision.

Need a bit of design clarification. Do we want "Page issues" to show up after the other stuff that can be injected in the lead section (e.g. breadcrumbs, revision number, revision table, etc.), or before? IMO it should be after, but I'm not sure given the text size. It should be noted that keeping it before everything is an easier fix as well.

I think right now all we need to do is make this take up a new line. Changing order is likely to be tricky and can be done later and is possibly not worth spending too much time on given not all pages use breadcrumbs.

Change 250751 had a related patch set uploaded (by Jhobs):
Make page issues link take up an entire line

https://gerrit.wikimedia.org/r/250751

Change 250751 merged by jenkins-bot:
Make page issues link take up an entire line

https://gerrit.wikimedia.org/r/250751

phuedx added a subscriber: phuedx.

Let's remember to take a look at this *after* it's deployed, eh?