Skip to content

Commit df6c8a4

Browse files
IbrahimMNadainada
andauthored
refactor setTitle in MediaPicker.Controller.js (#11428)
* refactor setTitle in MediaPicker.Controller.js * adding vm.activeTab Co-authored-by: inada <[email protected]>
1 parent be6e087 commit df6c8a4

File tree

1 file changed

+40
-37
lines changed

1 file changed

+40
-37
lines changed

src/Umbraco.Web.UI.Client/src/views/common/infiniteeditors/mediapicker/mediapicker.controller.js

Lines changed: 40 additions & 37 deletions
Original file line numberDiff line numberDiff line change
@@ -103,40 +103,43 @@ angular.module("umbraco")
103103
$scope.target = dialogOptions.currentTarget;
104104
}
105105

106-
function setTitle() {
107-
if (!$scope.model.title) {
108-
localizationService.localizeMany(["defaultdialogs_selectMedia", "mediaPicker_tabClipboard"])
109-
.then(function (data) {
110-
$scope.model.title = data[0];
111-
112-
113-
vm.navigation = [{
114-
"alias": "empty",
115-
"name": data[0],
116-
"icon": "icon-umb-media",
117-
"active": true,
118-
"view": ""
119-
}];
120-
121-
if(vm.clipboardItems) {
122-
vm.navigation.push({
123-
"alias": "clipboard",
124-
"name": data[1],
125-
"icon": "icon-paste-in",
126-
"view": "",
127-
"disabled": vm.clipboardItems.length === 0
128-
});
129-
}
130-
131-
vm.activeTab = vm.navigation[0];
132-
});
133-
134-
}
135-
}
136-
137-
function onInit() {
138-
139-
setTitle();
106+
function setTitle(data) {
107+
if (!$scope.model.title)
108+
$scope.model.title = data[0];
109+
}
110+
111+
function setNavigation(data) {
112+
if (!vm.navigation.length) {
113+
vm.navigation = [{
114+
"alias": "empty",
115+
"name": data[0],
116+
"icon": "icon-umb-media",
117+
"active": true,
118+
"view": ""
119+
}];
120+
if (vm.clipboardItems) {
121+
vm.navigation.push({
122+
"alias": "clipboard",
123+
"name": data[1],
124+
"icon": "icon-paste-in",
125+
"view": "",
126+
"disabled": vm.clipboardItems.length === 0
127+
});
128+
}
129+
vm.activeTab = vm.navigation[0];
130+
}
131+
}
132+
133+
134+
function onInit() {
135+
136+
137+
localizationService.localizeMany(["defaultdialogs_selectMedia", "mediaPicker_tabClipboard"])
138+
.then(function (localizationResult) {
139+
setTitle(localizationResult);
140+
setNavigation(localizationResult);
141+
});
142+
140143

141144
userService.getCurrentUser().then(function (userData) {
142145
userStartNodes = userData.startMediaIds;
@@ -425,9 +428,9 @@ angular.module("umbraco")
425428
};
426429

427430
function onNavigationChanged(tab) {
428-
vm.activeTab.active = false;
429-
vm.activeTab = tab;
430-
vm.activeTab.active = true;
431+
vm.activeTab.active = false;
432+
vm.activeTab = tab;
433+
vm.activeTab.active = true;
431434
};
432435

433436
function clickClearClipboard() {

0 commit comments

Comments
 (0)