From 5199e8e901619bb4045f1a157c3af8af3772cd0e Mon Sep 17 00:00:00 2001
From: Dave Woestenborghs <dave.woestenborghs@gmail.com>
Date: Thu, 25 Oct 2018 12:23:34 +0200
Subject: [PATCH] #3433 replace checkbox with for setting culture variants with
 toggle box

---
 .../views/permissions/permissions.controller.js    | 14 ++++++++++++--
 .../views/permissions/permissions.html             | 10 +++++++---
 2 files changed, 19 insertions(+), 5 deletions(-)

diff --git a/src/Umbraco.Web.UI.Client/src/views/documenttypes/views/permissions/permissions.controller.js b/src/Umbraco.Web.UI.Client/src/views/documenttypes/views/permissions/permissions.controller.js
index 028380ff81..4a7a870618 100644
--- a/src/Umbraco.Web.UI.Client/src/views/documenttypes/views/permissions/permissions.controller.js
+++ b/src/Umbraco.Web.UI.Client/src/views/documenttypes/views/permissions/permissions.controller.js
@@ -23,7 +23,8 @@
 
         vm.addChild = addChild;
         vm.removeChild = removeChild;
-        vm.toggle = toggle;
+        vm.toggleAllowAsRoot = toggleAllowAsRoot;
+        vm.toggleAllowCultureVariants = toggleAllowCultureVariants;
 
         /* ---------- INIT ---------- */
 
@@ -86,7 +87,7 @@
         /**
          * Toggle the $scope.model.allowAsRoot value to either true or false
          */
-        function toggle(){
+        function toggleAllowAsRoot(){
             if($scope.model.allowAsRoot){
                 $scope.model.allowAsRoot = false;
                 return;
@@ -95,6 +96,15 @@
             $scope.model.allowAsRoot = true;
         }
 
+        function toggleAllowCultureVariants() {
+            if ($scope.model.allowCultureVariant) {
+                $scope.model.allowCultureVariant = false;
+                return;
+            }
+
+            $scope.model.allowCultureVariant = true;
+        }
+
     }
 
     angular.module("umbraco").controller("Umbraco.Editors.DocumentType.PermissionsController", PermissionsController);
diff --git a/src/Umbraco.Web.UI.Client/src/views/documenttypes/views/permissions/permissions.html b/src/Umbraco.Web.UI.Client/src/views/documenttypes/views/permissions/permissions.html
index 2ecd1c518c..956b0d32ed 100644
--- a/src/Umbraco.Web.UI.Client/src/views/documenttypes/views/permissions/permissions.html
+++ b/src/Umbraco.Web.UI.Client/src/views/documenttypes/views/permissions/permissions.html
@@ -13,7 +13,7 @@
                     <umb-toggle
                         data-element="permissions-allow-as-root"
                         checked="model.allowAsRoot"
-                        on-click="vm.toggle()"
+                        on-click="vm.toggleAllowAsRoot()"
                         hotkey="alt+shift+r">
                     </umb-toggle>
                 </div>
@@ -48,8 +48,12 @@
                     <small>Define the rules for how this content type's properties can be varied</small>
                 </div>
                 <div class="sub-view-column-right">
-                    <label class="checkbox no-indent">
-                        <input type="checkbox" ng-model="model.allowCultureVariant" />
+                    <umb-toggle data-element="permissions-allow-culture-variant"
+                                checked="model.allowCultureVariant"
+                                on-click="vm.toggleAllowCultureVariants()"
+                                hotkey="alt+shift+v">
+                    </umb-toggle>
+                    <label class="checkbox no-indent">                       
                         Allow varying by Culture
                     </label>
                 </div>