From 34357433e7f9d9c5c03a0e7dc424d699a7066739 Mon Sep 17 00:00:00 2001
From: Pete Miller <miller.pete@gmail.com>
Date: Mon, 29 Jun 2020 23:33:25 -0700
Subject: [PATCH] Settings WebUI: fix rows dividers for brave items

---
 .../brave_appearance_page/toolbar.html        | 21 ++++++++++++-------
 .../brave_default_extensions_page.html        |  7 +++++++
 .../brave_new_tab_page.html                   |  6 ++++++
 .../brave_personalization_options.html        |  2 ++
 .../default_brave_shields_page.html           |  2 ++
 .../social_blocking_page.html                 |  4 ++++
 6 files changed, 35 insertions(+), 7 deletions(-)

diff --git a/browser/resources/settings/brave_appearance_page/toolbar.html b/browser/resources/settings/brave_appearance_page/toolbar.html
index 374753799775..2879da7ec2ee 100644
--- a/browser/resources/settings/brave_appearance_page/toolbar.html
+++ b/browser/resources/settings/brave_appearance_page/toolbar.html
@@ -8,29 +8,36 @@
   <template>
     <style include="settings-shared md-select iron-flex">
     </style>
-    <settings-toggle-button pref="{{prefs.brave.location_bar_is_wide}}"
-                            label="$i18n{appearanceSettingsLocationBarIsWide}">
+    <settings-toggle-button
+      class="cr-row"
+      pref="{{prefs.brave.location_bar_is_wide}}"
+      label="$i18n{appearanceSettingsLocationBarIsWide}">
     </settings-toggle-button>
     <settings-toggle-button
+      class="cr-row"
       pref="{{prefs.brave.autocomplete_enabled}}"
       label="$i18n{appearanceSettingsShowAutocompleteInAddressBar}">
     </settings-toggle-button>
     <settings-toggle-button
+      class="cr-row"
       pref="{{prefs.brave.top_site_suggestions_enabled}}"
       label="$i18n{appearanceSettingsUseTopSiteSuggestions}">
     </settings-toggle-button>
     <settings-toggle-button
+      class="cr-row"
       pref="{{prefs.brave.brave_suggested_site_suggestions_enabled}}"
       label="$i18n{appearanceSettingsUseBraveSuggestedSiteSuggestions}">
     </settings-toggle-button>
-    <settings-toggle-button pref="{{prefs.brave.hide_brave_rewards_button}}"
-                            label="$i18n{appearanceSettingsHideBraveRewardsButtonLabel}"
-                            sub-label="$i18n{appearanceSettingsHideBraveRewardsButtonDesc}">
+    <settings-toggle-button
+      pref="{{prefs.brave.hide_brave_rewards_button}}"
+      class="cr-row"
+      label="$i18n{appearanceSettingsHideBraveRewardsButtonLabel}"
+      sub-label="$i18n{appearanceSettingsHideBraveRewardsButtonDesc}">
     </settings-toggle-button>
     <settings-toggle-button
       pref="{{prefs.brave.always_show_bookmark_bar_on_ntp}}"
-      label="$i18n{appearanceSettingsAlwaysShowBookmarkBarOnNTP}"
-    >
+      class="cr-row"
+      label="$i18n{appearanceSettingsAlwaysShowBookmarkBarOnNTP}">
     </settings-toggle-button>
   </template>
   <script src="toolbar.js"></script>
diff --git a/browser/resources/settings/brave_default_extensions_page/brave_default_extensions_page.html b/browser/resources/settings/brave_default_extensions_page/brave_default_extensions_page.html
index c5ab2b4b1722..9330ad61f15f 100644
--- a/browser/resources/settings/brave_default_extensions_page/brave_default_extensions_page.html
+++ b/browser/resources/settings/brave_default_extensions_page/brave_default_extensions_page.html
@@ -74,22 +74,26 @@
       </settings-dropdown-menu>
     </div>
     <settings-toggle-button id="ipfsCompanionEnabled"
+        class="cr-row"
         pref="{{prefs.brave.wallet.load_on_startup}}"
         label="$i18n{loadCryptoWalletsOnStartupDesc}">
     </settings-toggle-button>
     <settings-toggle-button id="hangoutsEnabled"
+        class="cr-row"
         pref="{{prefs.brave.hangouts_enabled}}"
         label="Hangouts"
         sub-label="$i18n{hangoutsEnabledDesc}"
         on-settings-boolean-control-change="onHangoutsEnabledChange_">
     </settings-toggle-button>
     <settings-toggle-button id="ipfsCompanionEnabled"
+        class="cr-row"
         pref="{{prefs.brave.ipfs_companion_enabled}}"
         label="IPFS Companion"
         sub-label="$i18n{ipfsCompanionEnabledDesc}"
         on-settings-boolean-control-change="onIPFSCompanionEnabledChange_">
     </settings-toggle-button>
     <settings-toggle-button id="mediaRouterEnabled"
+        class="cr-row"
         pref="{{prefs.brave.enable_media_router}}"
         label="Media Router"
         sub-label="$i18n{mediaRouterEnabledDesc}"
@@ -97,6 +101,7 @@
     </settings-toggle-button>
     <if expr="enable_tor">
     <settings-toggle-button id="torEnabled"
+        class="cr-row"
         pref=""
         checked="[[torEnabled_]]"
         label="$i18n{torEnabledLabel}"
@@ -106,12 +111,14 @@
     </settings-toggle-button>
     </if>
     <settings-toggle-button id="webTorrentEnabled"
+        class="cr-row"
         pref="{{prefs.brave.webtorrent_enabled}}"
         label="WebTorrent"
         sub-label="$i18n{webTorrentEnabledDesc}"
         on-settings-boolean-control-change="onWebTorrentEnabledChange_">
     </settings-toggle-button>
     <settings-toggle-button
+        class="cr-row"
         pref="{{prefs.brave.ask_widevine_install}}"
         label="Widevine"
         sub-label="$i18n{appearanceSettingsAskWidevineInstallDesc}">
diff --git a/browser/resources/settings/brave_new_tab_page/brave_new_tab_page.html b/browser/resources/settings/brave_new_tab_page/brave_new_tab_page.html
index 7a177c6d3045..4c96214fdf34 100644
--- a/browser/resources/settings/brave_new_tab_page/brave_new_tab_page.html
+++ b/browser/resources/settings/brave_new_tab_page/brave_new_tab_page.html
@@ -47,30 +47,36 @@
     </div>
     <div class="flex">
       <settings-toggle-button
+          class="cr-row"
           pref="{{prefs.brave.new_tab_page.show_stats}}"
           label="$i18n{braveNewTabStats}">
       </settings-toggle-button>
       <settings-toggle-button
+          class="cr-row"
           pref="{{prefs.brave.new_tab_page.show_rewards}}"
           label="$i18n{braveNewTabBraveRewards}">
       </settings-toggle-button>
       <template is="dom-if" if="[[isBraveTogetherSupported_]]">
         <settings-toggle-button
+            class="cr-row"
             pref="{{prefs.brave.new_tab_page.show_together}}"
             label="$i18n{braveNewTabTogether}">
         </settings-toggle-button>
       </template>
       <template is="dom-if" if="[[isBinanceSupported_]]">
         <settings-toggle-button
+            class="cr-row"
             pref="{{prefs.brave.new_tab_page.show_binance}}"
             label="$i18n{braveNewTabBinance}">
         </settings-toggle-button>
       </template>
       <settings-toggle-button
+          class="cr-row"
           pref="{{prefs.brave.new_tab_page.show_top_sites}}"
           label="$i18n{braveNewTabTopSites}">
       </settings-toggle-button>
       <settings-toggle-button
+          class="cr-row"
           pref="{{prefs.brave.new_tab_page.show_clock}}"
           label="$i18n{braveNewTabClock}">
       </settings-toggle-button>
diff --git a/browser/resources/settings/brave_privacy_page/brave_personalization_options.html b/browser/resources/settings/brave_privacy_page/brave_personalization_options.html
index c0a56513fc3a..48dc51cc32e5 100644
--- a/browser/resources/settings/brave_privacy_page/brave_personalization_options.html
+++ b/browser/resources/settings/brave_privacy_page/brave_personalization_options.html
@@ -33,6 +33,7 @@
       </select>
     </div>
     <settings-toggle-button id="p3aEnabled"
+        class="cr-row"
         pref=""
         checked="[[p3aEnabled_]]"
         label="$i18n{p3aEnableTitle}"
@@ -40,6 +41,7 @@
         on-settings-boolean-control-change="onP3AEnabledChange_">
     </settings-toggle-button>
     <settings-toggle-button id="pushMessagingEnabled"
+        class="cr-row"
         pref="{{prefs.brave.gcm.channel_status}}"
         label="$i18n{pushMessagingLabel}">
         <template is="dom-if" if="[[shouldShowRestart_(
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..065807b0fb18 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
@@ -103,11 +103,13 @@
       </select>
     </div>
     <settings-toggle-button id="httpsEverywhereControlType"
+        class="cr-row"
         pref="{{prefs.brave.https_everywhere_default}}"
         label="$i18n{httpsEverywhereControlLabel}"
         on-settings-boolean-control-change="onHTTPSEverywhereControlChange_">
     </settings-toggle-button>
     <settings-toggle-button id="noScriptControlType"
+        class="cr-row"
         pref="{{prefs.brave.no_script_default}}"
         label="$i18n{noScriptControlLabel}"
         on-settings-boolean-control-change="onNoScriptControlChange_">
diff --git a/browser/resources/settings/social_blocking_page/social_blocking_page.html b/browser/resources/settings/social_blocking_page/social_blocking_page.html
index bf7343c64a0b..e2f959e9e751 100644
--- a/browser/resources/settings/social_blocking_page/social_blocking_page.html
+++ b/browser/resources/settings/social_blocking_page/social_blocking_page.html
@@ -10,18 +10,22 @@
     <style include="settings-shared md-select iron-flex">
     </style>
     <settings-toggle-button id="googleLoginControlType"
+        class="cr-row"
         pref="{{prefs.brave.google_login_default}}"
         label="$i18n{googleLoginControlLabel}">
     </settings-toggle-button>
     <settings-toggle-button id="fbEmbedControlType"
+        class="cr-row"
         pref="{{prefs.brave.fb_embed_default}}"
         label="$i18n{fbEmbedControlLabel}">
     </settings-toggle-button>
     <settings-toggle-button id="twitterEmbedControlType"
+        class="cr-row"
         pref="{{prefs.brave.twitter_embed_default}}"
         label="$i18n{twitterEmbedControlLabel}">
     </settings-toggle-button>
     <settings-toggle-button id="linkedInEmbedControlType"
+        class="cr-row"
         pref="{{prefs.brave.linkedin_embed_default}}"
         label="$i18n{linkedInEmbedControlLabel}">
     </settings-toggle-button>