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

Unified Diff: chrome/browser/ui/views/download/download_item_view_md.cc

Issue 1915983009: Fixed material design ink drop location for the MD download shelf buttons. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 months 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: chrome/browser/ui/views/download/download_item_view_md.cc
diff --git a/chrome/browser/ui/views/download/download_item_view_md.cc b/chrome/browser/ui/views/download/download_item_view_md.cc
index d48f87cad5f3b6dc7e556380a6d8fec99c16b027..72685f1d16bcfc6309de5c9fca18318528cc585b 100644
--- a/chrome/browser/ui/views/download/download_item_view_md.cc
+++ b/chrome/browser/ui/views/download/download_item_view_md.cc
@@ -476,7 +476,7 @@ void DownloadItemViewMd::AddInkDropLayer(ui::Layer* ink_drop_layer) {
std::unique_ptr<views::InkDropAnimation>
DownloadItemViewMd::CreateInkDropAnimation() const {
return base::WrapUnique(new views::FloodFillInkDropAnimation(
- size(), ink_drop_delegate_.last_ink_drop_location(),
+ gfx::Rect(size()), ink_drop_delegate_.last_ink_drop_location(),
color_utils::DeriveDefaultIconColor(GetTextColor())));
}
@@ -835,6 +835,10 @@ void DownloadItemViewMd::HandlePressEvent(const ui::LocatedEvent& event,
if (complete_animation_.get() && complete_animation_->is_animating())
complete_animation_->End();
+ // Don't show the ripple for right clicks.
+ if (!active_event)
+ return;
+
ink_drop_delegate_.set_last_ink_drop_location(event.location());
ink_drop_delegate_.OnAction(views::InkDropState::ACTION_PENDING);
}

Powered by Google App Engine
This is Rietveld 408576698