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

Unified Diff: chrome/browser/resources/security_warnings/interstitial_v2.js

Issue 865943002: Update SSL/malware/net error interstitial design (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Switch button layout switching from JS to CSS flexbox Created 5 years, 11 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/resources/security_warnings/interstitial_v2.js
diff --git a/chrome/browser/resources/security_warnings/interstitial_v2.js b/chrome/browser/resources/security_warnings/interstitial_v2.js
index e9a846ad6b0a91bc70831b0d04f5d960e0e1cd14..bccbf418e37cf1bc9031c6a2080eaa84576b7300 100644
--- a/chrome/browser/resources/security_warnings/interstitial_v2.js
+++ b/chrome/browser/resources/security_warnings/interstitial_v2.js
@@ -139,6 +139,14 @@ function setupEvents() {
} else {
$('details-button').addEventListener('click', function(event) {
var hiddenDetails = $('details').classList.toggle('hidden');
+
+ if (mobileNav) {
+ // Details appear over the main content on small screens.
+ $('main-content').classList.toggle('hidden', !hiddenDetails);
+ } else {
+ $('main-content').classList.toggle('hidden', false);
arv (Not doing code reviews) 2015/01/23 22:13:13 remove('hidden')
edwardjung 2015/01/26 10:55:38 Done.
+ }
+
$('details-button').innerText = hiddenDetails ?
loadTimeData.getString('openDetails') :
loadTimeData.getString('closeDetails');

Powered by Google App Engine
This is Rietveld 408576698