Page MenuHomePhabricator

Set the last edited bar max-width similar to T96317
Closed, ResolvedPublic1 Estimated Story Points

Description

The last modified bar should not take up the full screen it should look like this:

footer-lastedited-10.png (1×2 px, 184 KB)

Other tasks in the series: T96317, T96332, T96327, and T98385.

Event Timeline

There are a very large number of changes, so older changes are hidden. Show Older Changes

Change 210841 had a related patch set uploaded (by Bmansurov):
Center-align the last-modified bar on tablets

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

@KHammerstein: should the grey background of the last edited bar span the full width of the screen in stable, i.e. when it's immediately under the header? I should've pinged you about this earlier.

@phuedx lets leave it the same in stable for now

Change 210841 merged by jenkins-bot:
Alpha, Beta: Center-align the last-modified bar on tablets

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

@KHammerstein: It should be ready for sign off now.

@phuedx Is this live on beta labs? I'm not seeing a change.

@bmansurov
I'm seeing this:

Screenshot 2015-06-10 13.52.58.png (1×2 px, 414 KB)

It still looks full-width

@KHammerstein, the background is, but the text is center-aligned. Should we make the background center-aligned too? That would look weird, no?

This is what I was trying – and failing, I think – to ask about in T98925#1301061.

@bmansurov It looks a little weird, but not as weird as being full-width when everything else on the page has a constrained width.
Lets make the max width the same as the other tasks. Sorry for the mix-up!

We will update the last modified bar in the future.

@KHammerstein, could you please upload a mock? I'm not sure what the padding should be, should the center-aligned text start at the left where the background starts or should there be a little horizontal padding? Thanks

What does this all mean? (this card is very badly written and unclear)

@KHammerstein: I think the amount of discussion around this warrants a mock. Also remember that the last edited bar is in different positions in stable and beta/alpha.

@bmansurov @phuedx

footer-lastedited-10.png (1×2 px, 184 KB)

Good point Sam, I think when we move the max-width to stable that should include moving the last edited bar to the bottom. Its more work but I think will make sense. Should I make a task?

@KHammerstein, thanks. Yes, please create a new task for moving the bar down.

T103141 Last edited bar at moved from top to bottom of article to stable

Change 220627 had a related patch set uploaded (by Bmansurov):
Alpha, beta: Limit the last modified bar background width

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

Change 220627 merged by jenkins-bot:
Alpha, beta: Limit the last modified bar background width

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

Change 222604 had a related patch set uploaded (by Phuedx):
Fix minor regression in tablet pre-content gutter

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

Change 222604 merged by jenkins-bot:
Fix minor regression in tablet pre-content gutter

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