From 30203b3676091950a6c6eda1d8b278f747810f12 Mon Sep 17 00:00:00 2001
From: Simon Hong <shong@brave.com>
Date: Mon, 29 Jun 2020 09:52:29 +0900
Subject: [PATCH] Move Look & feel section to Shields prefs page

Look & feel section is moved to the top to include choosing Shields default view.
---
 .../default_brave_shields_page.html           | 29 +++++++++++--------
 1 file changed, 17 insertions(+), 12 deletions(-)

diff --git a/browser/resources/settings/default_brave_shields_page/default_brave_shields_page.html b/browser/resources/settings/default_brave_shields_page/default_brave_shields_page.html
index afdf8c628ee2..266be8451fa4 100644
--- a/browser/resources/settings/default_brave_shields_page/default_brave_shields_page.html
+++ b/browser/resources/settings/default_brave_shields_page/default_brave_shields_page.html
@@ -34,10 +34,13 @@
       .shields-view-col:last-of-type {
         margin: 16px 0 24px 16px;
       }
-       .shields-view-label {
-         margin: 0 20px;
-         cursor: pointer;
-       }
+      .shields-view-label {
+        margin: 0 20px;
+        cursor: pointer;
+      }
+      .default-view-padding {
+        padding-top: 20px;
+      }
     </style>
     <div class="settings-box first">
       <div class="flex">
@@ -49,8 +52,16 @@
       </div>
     </div>
     <div class="settings-box">
+      <div class="label shields-primary-title">$i18n{shieldsLookFeelTitle}</div>
+    </div>
+    <settings-toggle-button
+        class="continuation"
+        pref="{{prefs.brave.shields.stats_badge_visible}}"
+        label="$i18n{showStatsBlockedBadgeLabel}">
+    </settings-toggle-button>
+    <div class="settings-box continuation default-view-padding">
       <div class="flex">
-        <div class="label shields-primary-title">$i18n{defaultView}</div>
+        <div class="label">$i18n{defaultView}</div>
         <settings-radio-group
           pref="{{prefs.brave.shields.advanced_view_enabled}}">
           <controlled-radio-button class="shields-view-col" name="false"
@@ -136,13 +147,7 @@
         </template>
       </select>
     </div>
-    <div class="settings-box">
-      <div class="label shields-primary-title">$i18n{shieldsLookFeelTitle}</div>
-    </div>
-    <settings-toggle-button
-        class="continuation"
-        pref="{{prefs.brave.shields.stats_badge_visible}}"
-        label="$i18n{showStatsBlockedBadgeLabel}">
+
   </template>
   <script src="default_brave_shields_page.js"></script>
 </dom-module>