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

Issue 13145043: Fixes bug #1213264: IE10 inspector scrollbar (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 9 months ago by huwshimi
Modified:
11 years, 9 months ago
Reviewers:
mp+181177, jeff.pihach, rharding
Visibility:
Public.

Description

Fixes bug #1213264: IE10 inspector scrollbar Fixed bug #1213264: Overview Inspector has horizontal scroll bar in IE10 Also did a drive-by to fix a breakage in Firefox and some general spacing issues for the constraints confirmation. https://code.launchpad.net/~huwshimi/juju-gui/inspector-overview-scrollbar-ie10-1213264/+merge/181177 (do not edit description out of merge proposal)

Patch Set 1 #

Patch Set 2 : IE10 inspector scrollbar fix #

Patch Set 3 : Fixes bug #1213264: IE10 inspector scrollbar #

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

Messages

Total messages: 5
huwshimi
Please take a look.
11 years, 9 months ago (2013-08-21 00:17:33 UTC) #1
huwshimi
Please take a look.
11 years, 9 months ago (2013-08-21 03:24:08 UTC) #2
jeff.pihach
LGTM Thanks for this fix! IE QA OK
11 years, 9 months ago (2013-08-21 05:19:55 UTC) #3
huwshimi
*** Submitted: Fixes bug #1213264: IE10 inspector scrollbar Fixed bug #1213264: Overview Inspector has horizontal ...
11 years, 9 months ago (2013-08-21 06:16:49 UTC) #4
rharding
11 years, 9 months ago (2013-08-21 11:36:15 UTC) #5
LGTM going with hatch's QA.
Sign in to reply to this message.

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