Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(245)

Issue 376123002: Files.app: Override overflow specifying to use the original scroll bar. (Closed)

Created:
6 years, 5 months ago by hirono
Modified:
6 years, 5 months ago
Reviewers:
fukino
CC:
chromium-reviews, rginda+watch_chromium.org, mtomasz+watch_chromium.org, yoshiki+watch_chromium.org
Project:
chromium
Visibility:
Public.

Description

Files.app: Override overflow specifying to use the original scroll bar. The TableList specifies the overflow property depending on its height. If it specifies 'hidden', the list cannot be scrolled even if the Files.app specific scroll bar is visible. BUG=391698 TEST=manually Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=282005

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -0 lines) Patch
M ui/file_manager/file_manager/foreground/css/file_manager.css View 1 chunk +6 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
hirono
PTAL the CL? Thank you!
6 years, 5 months ago (2014-07-09 06:43:29 UTC) #1
fukino
On 2014/07/09 06:43:29, hirono wrote: > PTAL the CL? Thank you! LGTM. Thanks!
6 years, 5 months ago (2014-07-09 07:06:51 UTC) #2
hirono
The CQ bit was checked by hirono@chromium.org
6 years, 5 months ago (2014-07-09 09:10:59 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/hirono@chromium.org/376123002/1
6 years, 5 months ago (2014-07-09 09:11:44 UTC) #4
commit-bot: I haz the power
6 years, 5 months ago (2014-07-09 11:17:15 UTC) #5
Message was sent while issue was closed.
Change committed as 282005

Powered by Google App Engine
This is Rietveld 408576698