Page MenuHomePhabricator

Diff not centered
Closed, ResolvedPublic


@KHammerstein pointed out that if you visit
the diff and the fixed info panel at the bottom are not center aligned.

They should be.

Screen Shot 2015-06-24 at 11.52.24 AM.png (655×1 px, 53 KB)

Blocked on:
Need a mock to show how it should look.
Note Baha's comments:

  • Should the user info be centered
  • I think the left and right paddings should be removed now that the content is aligned in the center:


diff-width-19.png (805×1 px, 174 KB)

Event Timeline

Jdlrobson raised the priority of this task from to Needs Triage.
Jdlrobson updated the task description. (Show Details)
Jdlrobson added subscribers: Moushira, bmansurov, phuedx and 5 others.

Change 227906 had a related patch set uploaded (by Jdlrobson):
Diff should be centered in beta

The patch has some questions/comments.

For the benefit of non-devs - the question is should the user info bar be fix positioned as well? We are assuming yes.

phuedx changed the task status from Open to Stalled.Aug 5 2015, 10:17 AM

This is blocked on @KHammerstein/Design.

Note patch has been rewritten to center the user info.

Jdlrobson updated the task description. (Show Details)

I just went with your suggestion @bmansurov I'm keen to get this off my radar. @KHammerstein can always file a follow up bug.

This is the design as currently implemented in tablet mode:

Change 227906 merged by jenkins-bot:
Diff should be centered in beta

Jhernandez added a subscriber: Jhernandez.

Can't see it on beta labas, I don't know why. Have a look at the mock kaity posted please.

diff-width-19.png (805×1 px, 174 KB)

It's basically the same but the footer expands full width like the header.

Beta labs doesn't seem to be running latest code.

Jhernandez changed the task status from Stalled to Open.Aug 14 2015, 10:26 AM

Noice! Done! 👍👏