Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(250)

Issue 9827045: TRIVIAL: Revert notifications panel CSS problem.

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 11 months ago by bac
Modified:
10 years, 11 months ago
Reviewers:
mp+166130
Visibility:
Public.

Description

TRIVIAL: Revert notifications panel CSS problem. A fix to avoid a problem with a white bar happening with narrow browsers windows caused the notification panel to not fully display. This revision reverts the problem, fixing the notifications panel but causing the original rendering issue to be present again. From Ben: When you shrink the width of the viewport a white bar cuts the screen in half w/o something like that fix. This happens alot with the chrome dev tools open on the right side. https://code.launchpad.net/~bac/juju-gui/rev_navbar_css/+merge/166130 (do not edit description out of merge proposal)

Patch Set 1 #

Patch Set 2 : TRIVIAL: Revert notifications panel CSS problem. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -1 line) Patch
A [revision details] View 1 chunk +2 lines, -0 lines 0 comments Download
M lib/views/stylesheet.less View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 2
bac
Please take a look.
10 years, 11 months ago (2013-05-28 20:00:08 UTC) #1
bac
10 years, 11 months ago (2013-05-28 20:06:28 UTC) #2
*** Submitted:

TRIVIAL: Revert notifications panel CSS problem.

A fix to avoid a problem with a white bar happening with narrow browsers
windows caused the notification panel to not fully display.  This revision
reverts the problem, fixing the notifications panel but causing the original
rendering issue to be present again.

From Ben: When you shrink the width of the viewport a white bar cuts the
screen in half w/o something like that fix. This happens alot with the chrome
dev tools open on the right side.

R=
CC=
https://codereview.appspot.com/9827045
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b