From 5bb430ef4357ecf75eb5fb8e23066365bc10107e Mon Sep 17 00:00:00 2001
From: Will Rossiter <will@fullscreen.io>
Date: Thu, 1 Dec 2016 09:31:15 +1300
Subject: [PATCH] Remove references to ss-ui

---
 client/src/legacy/CMSMain.AddForm.js          |  9 +++----
 client/src/legacy/CMSMain.EditForm.js         | 24 +++++++++++--------
 code/Controllers/CMSMain.php                  |  3 ++-
 code/Controllers/CMSPageAddController.php     |  4 ++--
 code/Model/SiteTree.php                       | 12 +++++-----
 .../Includes/CMSMain_ViewControls.ss          |  2 +-
 .../CMSPagesController_ContentToolActions.ss  |  4 ++--
 7 files changed, 32 insertions(+), 26 deletions(-)

diff --git a/client/src/legacy/CMSMain.AddForm.js b/client/src/legacy/CMSMain.AddForm.js
index 6f7c6466..f3e48020 100644
--- a/client/src/legacy/CMSMain.AddForm.js
+++ b/client/src/legacy/CMSMain.AddForm.js
@@ -137,10 +137,11 @@ $.entwine('ss', function($){
       selectedEl.siblings().setSelected(false);
 
       // Disable the "Create" button if none of the pagetypes are available
-      var buttonState = this.find('#Form_AddForm_PageType div.radio:not(.disabled)').length
-        ? 'enable'
-        : 'disable';
-      this.find('button[name=action_doAdd]').button(buttonState);
+      if(this.find('#Form_AddForm_PageType div.radio:not(.disabled)').length) {
+        this.find('button[name=action_doAdd]').removeAttr('disabled');
+      } else {
+        this.find('button[name=action_doAdd]').addAttr('disabled');
+      }
 
       this.find('.message-restricted')[allAllowed ? 'hide' : 'show']();
     }
diff --git a/client/src/legacy/CMSMain.EditForm.js b/client/src/legacy/CMSMain.EditForm.js
index dc9c1c1e..586e323b 100644
--- a/client/src/legacy/CMSMain.EditForm.js
+++ b/client/src/legacy/CMSMain.EditForm.js
@@ -379,15 +379,15 @@ $.entwine('ss', function($){
 	 */
 	$('.cms-edit-form.changed').entwine({
 		onmatch: function(e) {
-			this.find('button[name=action_save]').button('option', 'showingAlternate', true);
-			this.find('button[name=action_publish]').button('option', 'showingAlternate', true);
+			this.find('button[name=action_save]').attr('data-showingAlternate', true);
+			this.find('button[name=action_publish]').attr('data-showingAlternate', true);
 			this._super(e);
 		},
 		onunmatch: function(e) {
 			var saveButton = this.find('button[name=action_save]');
-			if(saveButton.data('button')) saveButton.button('option', 'showingAlternate', false);
+			if(saveButton.data('button')) saveButton.attr('data-showingAlternate', false);
 			var publishButton = this.find('button[name=action_publish]');
-			if(publishButton.data('button')) publishButton.button('option', 'showingAlternate', false);
+			if(publishButton.data('button')) publishButton.attr('data-showingAlternate', false);
 			this._super(e);
 		}
 	});
@@ -397,11 +397,13 @@ $.entwine('ss', function($){
 		 * Bind to ssui.button event to trigger stylistic changes.
 		 */
 		onbuttonafterrefreshalternate: function() {
-			if (this.button('option', 'showingAlternate')) {
-				this.addClass('ss-ui-action-constructive');
+			if (this.data('showingAlternate')) {
+				this.addClass('btn-primary');
+        this.removeClass('btn-secondary');
 			}
 			else {
-				this.removeClass('ss-ui-action-constructive');
+				this.removeClass('btn-primary');
+        this.addClass('btn-secondary');
 			}
 		}
 	});
@@ -411,11 +413,13 @@ $.entwine('ss', function($){
 		 * Bind to ssui.button event to trigger stylistic changes.
 		 */
 		onbuttonafterrefreshalternate: function() {
-			if (this.button('option', 'showingAlternate')) {
-				this.addClass('ss-ui-action-constructive');
+			if (this.data('showingAlternate')) {
+				this.addClass('btn-primary');
+        this.removeClass('btn-secondary');
 			}
 			else {
-				this.removeClass('ss-ui-action-constructive');
+				this.removeClass('btn-primary');
+        this.addClass('btn-secondary');
 			}
 		}
 	});
diff --git a/code/Controllers/CMSMain.php b/code/Controllers/CMSMain.php
index e19dbac4..586d0125 100644
--- a/code/Controllers/CMSMain.php
+++ b/code/Controllers/CMSMain.php
@@ -450,8 +450,9 @@ class CMSMain extends LeftAndMain implements CurrentPageIdentifier, PermissionPr
 		// Create the Search and Reset action
 		$actions = new FieldList(
 			FormAction::create('doSearch',  _t('CMSMain_left_ss.APPLY_FILTER', 'Search'))
-				->addExtraClass('ss-ui-action-constructive'),
+				->addExtraClass('btn btn-primary'),
 			ResetFormAction::create('clear', _t('CMSMain_left_ss.CLEAR_FILTER', 'Clear'))
+				->addExtraClass('btn btn-secondary')
 		);
 
 		// Use <button> to allow full jQuery UI styling on the all of the Actions
diff --git a/code/Controllers/CMSPageAddController.php b/code/Controllers/CMSPageAddController.php
index 60113417..f3d67d4c 100644
--- a/code/Controllers/CMSPageAddController.php
+++ b/code/Controllers/CMSPageAddController.php
@@ -132,10 +132,10 @@ class CMSPageAddController extends CMSPageEditController {
 
 		$actions = new FieldList(
 			FormAction::create("doAdd", _t('CMSMain.Create',"Create"))
-				->addExtraClass('ss-ui-action-constructive')->setAttribute('data-icon', 'accept')
+				->addExtraClass('btn-primary font-icon-plus-circled')
 				->setUseButtonTag(true),
 			FormAction::create("doCancel", _t('CMSMain.Cancel',"Cancel"))
-				->addExtraClass('ss-ui-action-destructive ss-ui-action-cancel')
+				->addExtraClass('btn-secondary')
 				->setUseButtonTag(true)
 		);
 
diff --git a/code/Model/SiteTree.php b/code/Model/SiteTree.php
index dcc0db56..fecbc380 100755
--- a/code/Model/SiteTree.php
+++ b/code/Model/SiteTree.php
@@ -2322,7 +2322,7 @@ class SiteTree extends DataObject implements PermissionProvider,i18nEntityProvid
 				: _t('CMSMain.ARCHIVE', 'Archive');
 			$moreOptions->push(
 				FormAction::create('archive', $title)
-					->addExtraClass('delete ss-ui-action-destructive')
+					->addExtraClass('delete btn btn-secondary')
 					->setDescription(_t(
 						'SiteTree.BUTTONDELETEDESC',
 						'Remove from draft/live and send to archive'
@@ -2334,8 +2334,8 @@ class SiteTree extends DataObject implements PermissionProvider,i18nEntityProvid
 		if ($canEdit && $isOnDraft) {
 			$majorActions->push(
 				FormAction::create('save', _t('SiteTree.BUTTONSAVED', 'Saved'))
-					->setAttribute('data-icon', 'accept')
-					->setAttribute('data-icon-alternate', 'addpage')
+					->addExtraClass('btn-primary font-icon-check-mark')
+					->setUseButtonTag(true)
 					->setAttribute('data-text-alternate', _t('CMSMain.SAVEDRAFT','Save draft'))
 			);
 		}
@@ -2344,14 +2344,14 @@ class SiteTree extends DataObject implements PermissionProvider,i18nEntityProvid
 			// "publish", as with "save", it supports an alternate state to show when action is needed.
 			$majorActions->push(
 				$publish = FormAction::create('publish', _t('SiteTree.BUTTONPUBLISHED', 'Published'))
-					->setAttribute('data-icon', 'accept')
-					->setAttribute('data-icon-alternate', 'disk')
+					->addExtraClass('font-icon-check-mark-2')
+					->setUseButtonTag(true)
 					->setAttribute('data-text-alternate', _t('SiteTree.BUTTONSAVEPUBLISH', 'Save & publish'))
 			);
 
 			// Set up the initial state of the button to reflect the state of the underlying SiteTree object.
 			if($stagesDiffer) {
-				$publish->addExtraClass('ss-ui-alternate');
+				$publish->addExtraClass('btn-primary');
 			}
 		}
 
diff --git a/templates/SilverStripe/CMS/Controllers/Includes/CMSMain_ViewControls.ss b/templates/SilverStripe/CMS/Controllers/Includes/CMSMain_ViewControls.ss
index d74a07e0..2b87b3e2 100644
--- a/templates/SilverStripe/CMS/Controllers/Includes/CMSMain_ViewControls.ss
+++ b/templates/SilverStripe/CMS/Controllers/Includes/CMSMain_ViewControls.ss
@@ -1,5 +1,5 @@
 <div class="view-controls">
-	<button id="filters-button" class="icon-button font-icon-search no-text" title="<% _t('CMSPagesController_Tools_ss.FILTER', 'Filter') %>"></button>
+	<button id="filters-button" class="btn btn-secondary icon-button font-icon-search no-text" title="<% _t('CMSPagesController_Tools_ss.FILTER', 'Filter') %>"></button>
 	<div class="icon-button-group">
 		<%-- Change to data-pjax-target="Content-PageList" to enable in-edit listview --%>
 		<a class="cms-panel-link icon-button font-icon-tree page-view-link <% if $ViewState == 'treeview' %>active<% end_if %>"
diff --git a/templates/SilverStripe/CMS/Controllers/Includes/CMSPagesController_ContentToolActions.ss b/templates/SilverStripe/CMS/Controllers/Includes/CMSPagesController_ContentToolActions.ss
index 01436c6e..69ee9c4a 100644
--- a/templates/SilverStripe/CMS/Controllers/Includes/CMSPagesController_ContentToolActions.ss
+++ b/templates/SilverStripe/CMS/Controllers/Includes/CMSPagesController_ContentToolActions.ss
@@ -1,9 +1,9 @@
 <div class="toolbar toolbar--content cms-content-toolbar">
 	<div class="cms-actions-buttons-row">
-		<a class="ss-ui-button cms-content-addpage-button tool-button font-icon-plus" href="$LinkPageAdd" data-url-addpage="{$LinkPageAdd('', 'ParentID=%s')}"><% _t('CMSMain.AddNewButton', 'Add new') %></a>
+		<a class="btn btn-primary cms-content-addpage-button tool-button font-icon-plus" href="$LinkPageAdd" data-url-addpage="{$LinkPageAdd('', 'ParentID=%s')}"><% _t('CMSMain.AddNewButton', 'Add new') %></a>
 
 		<% if $View == 'Tree' %>
-		<button type="button" class="cms-content-batchactions-button tool-button font-icon-check-mark-2" data-toolid="batch-actions">
+		<button type="button" class="cms-content-batchactions-button btn btn-secondary tool-button font-icon-check-mark-2" data-toolid="batch-actions">
 			<% _t("CMSPagesController_ContentToolbar_ss.MULTISELECT","Batch actions") %>
 		</button>
 		<% end_if %>