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

Issue 2761583002: Make FadingEdgeScrollView more generic (Closed)

Created:
3 years, 9 months ago by gone
Modified:
3 years, 9 months ago
CC:
chromium-reviews, mahmadi+paymentswatch_chromium.org, rouslan+payments_chromium.org, sebsg+paymentswatch_chromium.org, gogerald+paymentswatch_chromium.org, agrieve+watch_chromium.org
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Make FadingEdgeScrollView more generic * Move this class from out of the payments directory into the widget directory. * Make the dialog edges consistent with Material Design and use lines that fade in/out instead of shadows that fade in/out. Screenshots: https://drive.google.com/corp/drive/folders/0B7c8ZkXVwskDRkpvdDhubHVnRmc BUG=701859 Review-Url: https://codereview.chromium.org/2761583002 Cr-Commit-Position: refs/heads/master@{#458198} Committed: https://chromium.googlesource.com/chromium/src/+/68d7e575cb12398943fb86f2776919702ff074d0

Patch Set 1 #

Total comments: 8

Patch Set 2 : Comments, added hard edge for payments #

Total comments: 2

Patch Set 3 : Make FadingEdgeScrollView more generic #

Messages

Total messages: 21 (12 generated)
gone
+Ted for FadingEdge class logic changes +Rouslan for edits to Payments stuff
3 years, 9 months ago (2017-03-17 23:37:10 UTC) #2
Ted C
lgtm https://codereview.chromium.org/2761583002/diff/1/chrome/android/java/src/org/chromium/chrome/browser/widget/FadingEdgeScrollView.java File chrome/android/java/src/org/chromium/chrome/browser/widget/FadingEdgeScrollView.java (right): https://codereview.chromium.org/2761583002/diff/1/chrome/android/java/src/org/chromium/chrome/browser/widget/FadingEdgeScrollView.java#newcode1 chrome/android/java/src/org/chromium/chrome/browser/widget/FadingEdgeScrollView.java:1: // Copyright 2016 The Chromium Authors. All rights ...
3 years, 9 months ago (2017-03-17 23:52:44 UTC) #3
please use gerrit instead
lgtm
3 years, 9 months ago (2017-03-18 00:01:41 UTC) #4
gone
The weirdo edge at the top was bugging me, so I added support to forcefully ...
3 years, 9 months ago (2017-03-18 00:16:51 UTC) #5
gone
Got a second lgtm from Bruno for the hard edge behavior, btw.
3 years, 9 months ago (2017-03-18 00:30:35 UTC) #6
Ted C
still lgtm https://codereview.chromium.org/2761583002/diff/20001/chrome/android/java/src/org/chromium/chrome/browser/widget/FadingEdgeScrollView.java File chrome/android/java/src/org/chromium/chrome/browser/widget/FadingEdgeScrollView.java (right): https://codereview.chromium.org/2761583002/diff/20001/chrome/android/java/src/org/chromium/chrome/browser/widget/FadingEdgeScrollView.java#newcode21 chrome/android/java/src/org/chromium/chrome/browser/widget/FadingEdgeScrollView.java:21: public static final int DRAW_NO_EDGE = 0; ...
3 years, 9 months ago (2017-03-20 19:10:31 UTC) #9
gone
https://codereview.chromium.org/2761583002/diff/20001/chrome/android/java/src/org/chromium/chrome/browser/widget/FadingEdgeScrollView.java File chrome/android/java/src/org/chromium/chrome/browser/widget/FadingEdgeScrollView.java (right): https://codereview.chromium.org/2761583002/diff/20001/chrome/android/java/src/org/chromium/chrome/browser/widget/FadingEdgeScrollView.java#newcode21 chrome/android/java/src/org/chromium/chrome/browser/widget/FadingEdgeScrollView.java:21: public static final int DRAW_NO_EDGE = 0; On 2017/03/20 ...
3 years, 9 months ago (2017-03-20 20:06:20 UTC) #15
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2761583002/40001
3 years, 9 months ago (2017-03-20 20:07:11 UTC) #18
commit-bot: I haz the power
3 years, 9 months ago (2017-03-20 22:17:34 UTC) #21
Message was sent while issue was closed.
Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/68d7e575cb12398943fb86f27769...

Powered by Google App Engine
This is Rietveld 408576698