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

Unified Diff: chrome/browser/resources/settings/settings_page_css.html

Issue 2041083002: MD Settings: Workaround for section expand/collapse animation jank. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix scroll position Created 4 years, 6 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
« no previous file with comments | « chrome/browser/resources/settings/settings_page/main_page_behavior.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/resources/settings/settings_page_css.html
diff --git a/chrome/browser/resources/settings/settings_page_css.html b/chrome/browser/resources/settings/settings_page_css.html
index 1b6d6b73d0f2c413b0a7e91d3d058dc0e98fbb87..d6f108936df5f4328096c33bd16384ddfb7aef8f 100644
--- a/chrome/browser/resources/settings/settings_page_css.html
+++ b/chrome/browser/resources/settings/settings_page_css.html
@@ -25,11 +25,22 @@
* with the top item in the side nav menu. Both things are desired
* by Alan (bettes@). */
margin-bottom: 3px;
- padding-top: var(--settings-page-padding-top);
+ /* TODO(michaelpg): This makes more sense as padding, but uses margin as
+ * a workaround for crbug.com/617287. Because margins collapse, add 3px
+ * to retain mostly the same spacing. */
+ margin-top: calc(var(--settings-page-padding-top) + 3px);
+ }
+
+ :host > div > :first-of-type {
+ /* TODO(michaelpg): This is offset because of the previous addition of
+ * 3px. */
dschuyler 2016/06/08 01:04:41 Let's include the bug number at each point of cha
michaelpg 2016/06/09 18:31:30 Done.
+ margin-top: var(--settings-page-padding-top);
}
.expanded.frozen {
- margin-top: -var(--settings-page-padding-top);
+ /* TODO(michaelpg): This is zeroed out because of the previous
+ * replacement of margin with padding, above. */
dschuyler 2016/06/08 01:04:41 * replacement of margin with padding, above. crbug
michaelpg 2016/06/09 18:31:30 Done.
+ margin-top: 0; /* -var(--settings-page-padding-top); */
}
</style>
</template>
« no previous file with comments | « chrome/browser/resources/settings/settings_page/main_page_behavior.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698