Chromium Code Reviews Chromium Code Reviews
Issue 896133005:
Plugin Power Saver: Implement UI by Chrome designers. (Closed)
Base URL: https://chromium.googlesource.com/chromium/src.git@master| Index: chrome/renderer/resources/plugin_poster.html |
| diff --git a/chrome/renderer/resources/plugin_poster.html b/chrome/renderer/resources/plugin_poster.html |
| index 790f4d98e51748b7aa895ccc8c6278322af53426..22c65b29e759b4dfcef2e73f13f23785ec55427c 100644 |
| --- a/chrome/renderer/resources/plugin_poster.html |
| +++ b/chrome/renderer/resources/plugin_poster.html |
| @@ -12,28 +12,41 @@ function notifyDidFinishLoading() { |
| <link rel="stylesheet" href="plugin_placeholders.css"></link> |
| <style> |
| #outer { |
| + border: none; |
| cursor: pointer; |
| + background-position: center; |
| + background-repeat: no-repeat; |
| + background-size: contain; |
| +} |
| + |
| +#shielding { |
| + background-color: rgba(0, 0, 0, 0.5); |
| + height: 100%; |
| + position: absolute; |
| + width: 100%; |
| } |
| #plugin_icon { |
| - opacity: 0.2; |
| + opacity: 0.8; |
| } |
| #outer:hover #plugin_icon { |
| - opacity: 1.0; |
| + opacity: 0.95; |
| } |
| #inner { |
| - margin-top: -23px; |
| + margin-top: -25px; |
| } |
| </style> |
| </head> |
| -<body i18n-values=".style.background-image:background" onload="notifyDidFinishLoading();"> |
| - <div i18n-values="title:name" id="outer" onclick="plugin.load()"> |
| - <div id="inner"> |
| - <div><img id="plugin_icon" src="play.png" /></div> |
| - <p id="debug"> </p> |
| +<body onload="notifyDidFinishLoading();"> |
| + <div i18n-values="title:name;.style.background-image:background" id="outer" onclick="plugin.load()"> |
|
Bernhard Bauer
2015/02/10 09:08:16
Can you break this line to make it fit into 80 col
tommycli
2015/02/10 17:56:11
Done.
|
| + <div id="shielding"> |
| + <div id="inner"> |
| + <div><img id="plugin_icon" src="plugin_power_saver_play.png" /></div> |
| + <p id="debug"> </p> |
|
Bernhard Bauer
2015/02/10 09:08:16
While you're making changes, can you remove this l
tommycli
2015/02/10 17:56:11
Done.
|
| + </div> |
| </div> |
| </div> |
| </body> |
