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

Unified Diff: ui/views/controls/scrollbar/overlay_scroll_bar.cc

Issue 2496643002: Implement Sebastien's overlay scrollbars for native UI (Views). (Closed)
Patch Set: clarify comment Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ui/views/controls/scrollbar/overlay_scroll_bar.cc
diff --git a/ui/views/controls/scrollbar/overlay_scroll_bar.cc b/ui/views/controls/scrollbar/overlay_scroll_bar.cc
index 4a5e27e1b632763a6cf88f304595148f5ce541c1..7e3dd3fe599d984310f3cbd1adfe4a3cb8cc2c06 100644
--- a/ui/views/controls/scrollbar/overlay_scroll_bar.cc
+++ b/ui/views/controls/scrollbar/overlay_scroll_bar.cc
@@ -7,85 +7,113 @@
#include "base/macros.h"
#include "third_party/skia/include/core/SkColor.h"
#include "third_party/skia/include/core/SkXfermode.h"
+#include "ui/compositor/scoped_layer_animation_settings.h"
#include "ui/gfx/canvas.h"
#include "ui/views/background.h"
#include "ui/views/border.h"
-#include "ui/views/controls/scrollbar/base_scroll_bar_thumb.h"
namespace views {
namespace {
-const int kScrollbarWidth = 10;
-const int kThumbInsetInside = 3;
-const int kThumbInsetFromEdge = 1;
-const int kThumbCornerRadius = 2;
-const int kThumbMinimumSize = kScrollbarWidth;
-const int kThumbHoverAlpha = 128;
-const int kThumbDefaultAlpha = 64;
-
-class OverlayScrollBarThumb : public BaseScrollBarThumb,
- public gfx::AnimationDelegate {
- public:
- explicit OverlayScrollBarThumb(BaseScrollBar* scroll_bar);
- ~OverlayScrollBarThumb() override;
-
- protected:
- // View overrides:
- gfx::Size GetPreferredSize() const override;
- void OnPaint(gfx::Canvas* canvas) override;
-
- // gfx::AnimationDelegate overrides:
- void AnimationProgressed(const gfx::Animation* animation) override;
-
- private:
- double animation_opacity_;
- DISALLOW_COPY_AND_ASSIGN(OverlayScrollBarThumb);
-};
-
-OverlayScrollBarThumb::OverlayScrollBarThumb(BaseScrollBar* scroll_bar)
+// Total thickness of the thumb (matches visuals when hovered).
+const int kThumbThickness = 11;
+// When unhovered, the thumb is slimmer by this amount of dip.
+const int kThumbUnhoveredDifference = 4;
+const int kThumbStroke = 1;
+const SkAlpha kThumbHoverAlpha = 0x80; // 0.5
+const SkAlpha kThumbDefaultAlpha = 0x4D; // 0.3
+
+} // namespace
+
+OverlayScrollBar::Thumb::Thumb(OverlayScrollBar* scroll_bar)
: BaseScrollBarThumb(scroll_bar),
- animation_opacity_(0.0) {
+ scroll_bar_(scroll_bar),
+ hover_animation_(this) {
// This is necessary, otherwise the thumb will be rendered below the views if
// those views paint to their own layers.
SetPaintToLayer(true);
layer()->SetFillsBoundsOpaquely(false);
+ layer()->SetOpacity(0);
}
-OverlayScrollBarThumb::~OverlayScrollBarThumb() {
-}
+OverlayScrollBar::Thumb::~Thumb() {}
-gfx::Size OverlayScrollBarThumb::GetPreferredSize() const {
- return gfx::Size(kThumbMinimumSize, kThumbMinimumSize);
+gfx::Size OverlayScrollBar::Thumb::GetPreferredSize() const {
+ return gfx::Size(kThumbThickness, kThumbThickness);
}
-void OverlayScrollBarThumb::OnPaint(gfx::Canvas* canvas) {
- gfx::Rect local_bounds(GetLocalBounds());
- SkPaint paint;
- int alpha = kThumbDefaultAlpha * animation_opacity_;
- if (GetState() == CustomButton::STATE_HOVERED) {
- alpha = kThumbHoverAlpha * animation_opacity_;
- } else if(GetState() == CustomButton::STATE_PRESSED) {
- // If we are in pressed state, no need to worry about animation,
- // just display the deeper color.
- alpha = kThumbHoverAlpha;
+void OverlayScrollBar::Thumb::OnPaint(gfx::Canvas* canvas) {
+ SkAlpha alpha = kThumbHoverAlpha;
+ // If we are in pressed state, no need to worry about animation,
+ // just display the deeper color.
+ if (GetState() != CustomButton::STATE_PRESSED) {
+ alpha = hover_animation_.CurrentValueBetween(kThumbDefaultAlpha,
+ kThumbHoverAlpha);
}
- paint.setStyle(SkPaint::kFill_Style);
- paint.setColor(SkColorSetARGB(alpha, 0, 0, 0));
- canvas->DrawRoundRect(local_bounds, kThumbCornerRadius, paint);
+ SkPaint fill_paint;
+ fill_paint.setAntiAlias(true);
+ fill_paint.setStyle(SkPaint::kFill_Style);
+ fill_paint.setColor(SkColorSetA(SK_ColorBLACK, alpha));
+ gfx::RectF fill_bounds(GetLocalBounds());
+ const double slimmer_by =
+ hover_animation_.CurrentValueBetween(kThumbUnhoveredDifference, 0);
+ fill_bounds.Inset(gfx::InsetsF(IsHorizontal() ? slimmer_by : 0,
+ IsHorizontal() ? 0 : slimmer_by, 0, 0));
+ fill_bounds.Inset(gfx::InsetsF(kThumbStroke, kThumbStroke,
+ IsHorizontal() ? 0 : kThumbStroke,
+ IsHorizontal() ? kThumbStroke : 0));
+ canvas->DrawRect(fill_bounds, fill_paint);
+
+ SkPaint stroke_paint;
+ stroke_paint.setAntiAlias(true);
+ stroke_paint.setStyle(SkPaint::kFill_Style);
+ stroke_paint.setColor(SkColorSetA(SK_ColorWHITE, alpha));
+ stroke_paint.setStrokeWidth(kThumbStroke);
+ gfx::RectF stroke_bounds(fill_bounds);
+ stroke_bounds.Inset(gfx::InsetsF(-0.5f));
+ // The stroke doesn't apply to the far edge of the thumb.
+ SkPath path;
+ path.moveTo(gfx::PointFToSkPoint(stroke_bounds.top_right()));
+ path.lineTo(gfx::PointFToSkPoint(stroke_bounds.origin()));
+ path.lineTo(gfx::PointFToSkPoint(stroke_bounds.bottom_left()));
+ if (IsHorizontal()) {
+ path.moveTo(gfx::PointFToSkPoint(stroke_bounds.bottom_right()));
+ path.close();
+ } else {
+ path.lineTo(gfx::PointFToSkPoint(stroke_bounds.bottom_right()));
+ }
+ canvas->DrawPath(path, stroke_paint);
+}
+
+void OverlayScrollBar::Thumb::OnBoundsChanged(
+ const gfx::Rect& previous_bounds) {
+ scroll_bar_->ShowThumb();
+ // Don't start the hide countdown if the thumb is still hovered or pressed.
+ if (GetState() == CustomButton::STATE_NORMAL)
+ scroll_bar_->StartThumbHideCountdown();
+}
+
+void OverlayScrollBar::Thumb::SetState(CustomButton::ButtonState state) {
+ if (state != GetState()) {
+ if (state == CustomButton::STATE_NORMAL) {
+ hover_animation_.Hide();
+ scroll_bar_->StartThumbHideCountdown();
+ } else {
+ hover_animation_.Show();
sadrul 2016/11/12 03:01:10 Since the thumb has a layer, can you just animate
Evan Stade 2016/11/14 16:56:07 I've changed all animations to be applied to layer
+ }
+ }
+ BaseScrollBarThumb::SetState(state);
}
-void OverlayScrollBarThumb::AnimationProgressed(
+void OverlayScrollBar::Thumb::AnimationProgressed(
const gfx::Animation* animation) {
- animation_opacity_ = animation->GetCurrentValue();
SchedulePaint();
}
-} // namespace
-
OverlayScrollBar::OverlayScrollBar(bool horizontal)
- : BaseScrollBar(horizontal, new OverlayScrollBarThumb(this)),
- animation_(static_cast<OverlayScrollBarThumb*>(GetThumb())) {
+ : BaseScrollBar(horizontal, new Thumb(this)),
+ thumb_hide_timer_(false, false) {
set_notify_enter_exit_on_child(true);
}
@@ -93,18 +121,7 @@ OverlayScrollBar::~OverlayScrollBar() {
}
gfx::Rect OverlayScrollBar::GetTrackBounds() const {
- gfx::Rect local_bounds(GetLocalBounds());
- if (IsHorizontal()) {
- local_bounds.Inset(kThumbInsetFromEdge, kThumbInsetInside,
- kThumbInsetFromEdge, kThumbInsetFromEdge);
- } else {
- local_bounds.Inset(kThumbInsetInside, kThumbInsetFromEdge,
- kThumbInsetFromEdge, kThumbInsetFromEdge);
- }
- gfx::Size track_size = local_bounds.size();
- track_size.SetToMax(GetThumb()->size());
- local_bounds.set_size(track_size);
- return local_bounds;
+ return GetLocalBounds();
}
int OverlayScrollBar::GetLayoutSize() const {
@@ -112,31 +129,34 @@ int OverlayScrollBar::GetLayoutSize() const {
}
int OverlayScrollBar::GetContentOverlapSize() const {
- return kScrollbarWidth;
+ return kThumbThickness;
}
-void OverlayScrollBar::OnMouseEnteredScrollView(const ui::MouseEvent& event) {
- animation_.Show();
+void OverlayScrollBar::OnMouseEntered(const ui::MouseEvent& event) {
+ ShowThumb();
}
-void OverlayScrollBar::OnMouseExitedScrollView(const ui::MouseEvent& event) {
- animation_.Hide();
+void OverlayScrollBar::OnMouseExited(const ui::MouseEvent& event) {
+ StartThumbHideCountdown();
}
-void OverlayScrollBar::OnGestureEvent(ui::GestureEvent* event) {
- switch (event->type()) {
- case ui::ET_GESTURE_SCROLL_BEGIN:
- animation_.Show();
- break;
- case ui::ET_GESTURE_SCROLL_END:
- case ui::ET_SCROLL_FLING_START:
- case ui::ET_GESTURE_END:
- animation_.Hide();
- break;
- default:
- break;
- }
- BaseScrollBar::OnGestureEvent(event);
+void OverlayScrollBar::ShowThumb() {
+ GetThumb()->layer()->SetOpacity(1.0f);
+ thumb_hide_timer_.Stop();
+}
+
+void OverlayScrollBar::HideThumb() {
+ ui::ScopedLayerAnimationSettings settings(GetThumb()->layer()->GetAnimator());
+ settings.SetTransitionDuration(base::TimeDelta::FromMilliseconds(200));
+ GetThumb()->layer()->SetOpacity(0.0f);
+}
+
+void OverlayScrollBar::StartThumbHideCountdown() {
+ if (IsMouseHovered())
+ return;
+ thumb_hide_timer_.Start(
+ FROM_HERE, base::TimeDelta::FromSeconds(1),
+ base::Bind(&OverlayScrollBar::HideThumb, base::Unretained(this)));
}
void OverlayScrollBar::Layout() {

Powered by Google App Engine
This is Rietveld 408576698