From 41e50383671843823a9c41d11b02a8cd9165e0b0 Mon Sep 17 00:00:00 2001
From: Anton Tananaev <anton.tananaev@gmail.com>
Date: Fri, 4 Nov 2016 08:03:57 +1300
Subject: Move some buttons to map

---
 web/app/view/Devices.js           | 52 ++++++++++++++++++++++-----------------
 web/app/view/DevicesController.js |  4 ++-
 web/app/view/Main.js              |  1 -
 web/app/view/Map.js               | 45 +++++++++++++++++++++++++++++++++
 web/app/view/MapController.js     | 23 -----------------
 5 files changed, 77 insertions(+), 48 deletions(-)

diff --git a/web/app/view/Devices.js b/web/app/view/Devices.js
index 55741cd..3cfd554 100644
--- a/web/app/view/Devices.js
+++ b/web/app/view/Devices.js
@@ -36,12 +36,39 @@ Ext.define('Traccar.view.Devices', {
         this.callParent();
     },
 
-    title: Strings.deviceTitle,
     selType: 'rowmodel',
 
     tbar: {
-        xtype: 'editToolbar',
         items: [{
+            xtype: 'tbtext',
+            html: Strings.deviceTitle,
+            baseCls: 'x-panel-header-title-default'
+        }, {
+            xtype: 'tbfill'
+        }, {
+            xtype: 'button',
+            handler: 'onAddClick',
+            reference: 'toolbarAddButton',
+            glyph: 'xf067@FontAwesome',
+            tooltip: Strings.sharedAdd,
+            tooltipType: 'title'
+        }, {
+            xtype: 'button',
+            disabled: true,
+            handler: 'onEditClick',
+            reference: 'toolbarEditButton',
+            glyph: 'xf040@FontAwesome',
+            tooltip: Strings.sharedEdit,
+            tooltipType: 'title'
+        }, {
+            xtype: 'button',
+            disabled: true,
+            handler: 'onRemoveClick',
+            reference: 'toolbarRemoveButton',
+            glyph: 'xf00d@FontAwesome',
+            tooltip: Strings.sharedRemove,
+            tooltipType: 'title'
+        }, {
             xtype: 'button',
             disabled: true,
             handler: 'onGeofencesClick',
@@ -56,25 +83,6 @@ Ext.define('Traccar.view.Devices', {
             glyph: 'xf093@FontAwesome',
             tooltip: Strings.deviceCommand,
             tooltipType: 'title'
-        }, {
-            xtype: 'tbfill'
-        }, {
-            id: 'muteButton',
-            glyph: 'xf1f7@FontAwesome',
-            tooltip: Strings.sharedMute,
-            tooltipType: 'title',
-            pressed : true,
-            enableToggle: true,
-            listeners: {
-                toggle: function (button, pressed) {
-                    if (pressed) {
-                        button.setGlyph('xf1f7@FontAwesome');
-                    } else {
-                        button.setGlyph('xf0a2@FontAwesome');
-                    }
-                },
-                scope: this
-            }
         }, {
             id: 'deviceFollowButton',
             glyph: 'xf05b@FontAwesome',
@@ -82,8 +90,6 @@ Ext.define('Traccar.view.Devices', {
             tooltipType: 'title',
             enableToggle: true,
             toggleHandler: 'onFollowClick'
-        }, {
-            xtype: 'settingsMenu'
         }]
     },
 
diff --git a/web/app/view/DevicesController.js b/web/app/view/DevicesController.js
index 731cbed..aa8e788 100644
--- a/web/app/view/DevicesController.js
+++ b/web/app/view/DevicesController.js
@@ -124,7 +124,9 @@ Ext.define('Traccar.view.DevicesController', {
         var device;
         if (pressed) {
             device = this.getView().getSelectionModel().getSelection()[0];
-            this.fireEvent('selectdevice', device, true);
+            if (device) {
+                this.fireEvent('selectdevice', device, true);
+            }
         }
     },
 
diff --git a/web/app/view/Main.js b/web/app/view/Main.js
index be9f00d..29db829 100644
--- a/web/app/view/Main.js
+++ b/web/app/view/Main.js
@@ -62,7 +62,6 @@ Ext.define('Traccar.view.Main', {
     }, {
         region: 'center',
         xtype: 'mapView',
-        header: true,
         collapsible: false
     }]
 });
diff --git a/web/app/view/Map.js b/web/app/view/Map.js
index ca5e39d..c5b6210 100644
--- a/web/app/view/Map.js
+++ b/web/app/view/Map.js
@@ -26,6 +26,51 @@ Ext.define('Traccar.view.Map', {
     controller: 'map',
 
     title: Strings.mapTitle,
+    tbar: {
+        items: [{
+            xtype: 'tbtext',
+            html: Strings.mapTitle,
+            baseCls: 'x-panel-header-title-default'
+        }, {
+            xtype: 'tbfill'
+        }, {
+            xtype: 'button',
+            tooltipType: 'title',
+            handler: 'showGeofences',
+            reference: 'showGeofencesButton',
+            glyph: 'xf21d@FontAwesome',
+            enableToggle: true,
+            pressed: true,
+            tooltip: Strings.sharedGeofences
+        }, {
+            xtype: 'button',
+            tooltipType: 'title',
+            handler: 'showLiveRoutes',
+            reference: 'showLiveRoutes',
+            glyph: 'xf1b0@FontAwesome',
+            enableToggle: true,
+            tooltip: Strings.mapLiveRoutes
+        }, {
+            id: 'muteButton',
+            glyph: 'xf1f7@FontAwesome',
+            tooltip: Strings.sharedMute,
+            tooltipType: 'title',
+            pressed : true,
+            enableToggle: true,
+            listeners: {
+                toggle: function (button, pressed) {
+                    if (pressed) {
+                        button.setGlyph('xf1f7@FontAwesome');
+                    } else {
+                        button.setGlyph('xf0a2@FontAwesome');
+                    }
+                },
+                scope: this
+            }
+        }, {
+            xtype: 'settingsMenu'
+        }]
+    },
 
     getLatestSource: function () {
         return this.latestSource;
diff --git a/web/app/view/MapController.js b/web/app/view/MapController.js
index 23cbc9e..323f29d 100644
--- a/web/app/view/MapController.js
+++ b/web/app/view/MapController.js
@@ -67,29 +67,6 @@ Ext.define('Traccar.view.MapController', {
         this.reportMarkers = {};
         this.liveRoutes = {};
         this.liveRouteLength = Traccar.app.getAttributePreference('web.liveRouteLength', 10);
-        this.getView().header = {
-            xtype: 'header',
-            title: Strings.mapTitle,
-            defaults: {
-                xtype: 'button',
-                tooltipType: 'title',
-                margin: Traccar.Style.headerButtonsMargin
-            },
-            items: [{
-                handler: 'showGeofences',
-                reference: 'showGeofencesButton',
-                glyph: 'xf21d@FontAwesome',
-                enableToggle: true,
-                pressed: true,
-                tooltip: Strings.sharedGeofences
-            }, {
-                handler: 'showLiveRoutes',
-                reference: 'showLiveRoutes',
-                glyph: 'xf1b0@FontAwesome',
-                enableToggle: true,
-                tooltip: Strings.mapLiveRoutes
-            }]
-        };
     },
 
     getDeviceColor: function (device) {
-- 
cgit v1.2.3