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

Unified Diff: third_party/polymer/v1_0/components-chromium/paper-ripple/paper-ripple.html

Issue 2691393005: MD WebUI: create a Chrome-only <paper-ripple> that's resistant to JS jank (via web animations API) (Closed)
Patch Set: fix some quirks Created 3 years, 10 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: third_party/polymer/v1_0/components-chromium/paper-ripple/paper-ripple.html
diff --git a/third_party/polymer/v1_0/components-chromium/paper-ripple/paper-ripple.html b/third_party/polymer/v1_0/components-chromium/paper-ripple/paper-ripple.html
index 1a8e66bc499cf53b4a4a24ffe7c1c71c86efc460..37b1b04343f1bc3d14b2120f4384a8d05aa73c10 100644
--- a/third_party/polymer/v1_0/components-chromium/paper-ripple/paper-ripple.html
+++ b/third_party/polymer/v1_0/components-chromium/paper-ripple/paper-ripple.html
@@ -77,73 +77,32 @@ Apply `circle` class to make the rippling effect within a circle.
<template>
<style>
:host {
- display: block;
- position: absolute;
border-radius: inherit;
- overflow: hidden;
- top: 0;
- left: 0;
- right: 0;
bottom: 0;
-
- /* See PolymerElements/paper-behaviors/issues/34. On non-Chrome browsers,
- * creating a node (with a position:absolute) in the middle of an event
- * handler "interrupts" that event handler (which happens when the
- * ripple is created on demand) */
+ display: block;
+ left: 0;
+ overflow: hidden;
pointer-events: none;
- }
-
- :host([animating]) {
- /* This resolves a rendering issue in Chrome (as of 40) where the
- ripple is not properly clipped by its parent (which may have
- rounded corners). See: http://jsbin.com/temexa/4
-
- Note: We only apply this style conditionally. Otherwise, the browser
- will create a new compositing layer for every ripple element on the
- page, and that would be bad. */
- -webkit-transform: translate(0, 0);
+ position: absolute;
+ right: 0;
+ top: 0;
+ /* For rounded corners: http://jsbin.com/temexa/4. */
transform: translate3d(0, 0, 0);
}
- #background,
- #waves,
- .wave-container,
- .wave {
+ .ripple {
+ background-color: currentcolor;
+ opacity: .25;
pointer-events: none;
position: absolute;
- top: 0;
- left: 0;
- width: 100%;
- height: 100%;
- }
-
- #background,
- .wave {
- opacity: 0;
- }
-
- #waves,
- .wave {
- overflow: hidden;
+ will-change: height, transform, width;
}
- .wave-container,
- .wave {
+ .ripple,
+ :host(.circle) {
border-radius: 50%;
}
-
- :host(.circle) #background,
- :host(.circle) #waves {
- border-radius: 50%;
- }
-
- :host(.circle) .wave-container {
- overflow: hidden;
- }
</style>
-
- <div id="background"></div>
- <div id="waves"></div>
</template>
</dom-module>
-<script src="paper-ripple-extracted.js"></script></body></html>
+<script src="paper-ripple-extracted.js"></script></body></html>

Powered by Google App Engine
This is Rietveld 408576698