Skip to content

Commit f9be0d3

Browse files
committed
getIconFromResource
1 parent 2b22766 commit f9be0d3

File tree

3 files changed

+15
-10
lines changed

3 files changed

+15
-10
lines changed

services/static-webserver/client/source/class/osparc/dashboard/NewPlusMenu.js

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -209,17 +209,9 @@ qx.Class.define("osparc.dashboard.NewPlusMenu", {
209209
__addIcon: function(menuButton, resourceInfo, resourceMetadata) {
210210
let source = null;
211211
if (resourceInfo && resourceInfo["icon"]) {
212-
// first the one set in the ui_config
213212
source = resourceInfo["icon"];
214-
} else if (resourceMetadata && resourceMetadata["icon"]) {
215-
// second the icon from the resource
216-
source = resourceMetadata["icon"];
217-
} else if (resourceMetadata && resourceMetadata["thumbnail"]) {
218-
// third the thumbnail from the resource
219-
source = resourceMetadata["thumbnail"];
220213
} else {
221-
// finally product icon
222-
source = osparc.dashboard.CardBase.PRODUCT_ICON;
214+
source = osparc.utils.Utils.getIconFromResource(resourceMetadata);
223215
}
224216

225217
if (source) {

services/static-webserver/client/source/class/osparc/desktop/credits/CreditsServiceListItem.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,8 @@ qx.Class.define("osparc.desktop.credits.CreditsServiceListItem", {
3333
const name = this.getChildControl("title");
3434
const serviceMetadata = osparc.service.Utils.getLatest(serviceKey);
3535
if (serviceMetadata) {
36-
icon.setSource(serviceMetadata["thumbnail"] ? serviceMetadata["thumbnail"] : osparc.dashboard.CardBase.PRODUCT_THUMBNAIL);
36+
const source = osparc.utils.Utils.getIconFromResource(serviceMetadata);
37+
icon.setSource(source);
3738
name.setValue(serviceMetadata["name"]);
3839
} else {
3940
icon.setSource(osparc.dashboard.CardBase.PRODUCT_THUMBNAIL);

services/static-webserver/client/source/class/osparc/utils/Utils.js

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -111,6 +111,18 @@ qx.Class.define("osparc.utils.Utils", {
111111
return newName;
112112
},
113113

114+
getIconFromResource: function(resourceMetadata) {
115+
if (resourceMetadata) {
116+
if (resourceMetadata["icon"]) {
117+
return resourceMetadata["icon"];
118+
}
119+
if (resourceMetadata["thumbnail"]) {
120+
return resourceMetadata["thumbnail"];
121+
}
122+
}
123+
return osparc.dashboard.CardBase.PRODUCT_ICON;
124+
},
125+
114126
isEmail: function(value) {
115127
const reg = /^([A-Za-z0-9_\-.+])+@([A-Za-z0-9_\-.])+\.([A-Za-z]{2,})$/;
116128
return reg.test(value);

0 commit comments

Comments
 (0)