Skip to content

Commit 0742742

Browse files
committed
Simplify Loading page
1 parent 3bd294a commit 0742742

File tree

4 files changed

+36
-102
lines changed

4 files changed

+36
-102
lines changed

services/static-webserver/client/source/class/osparc/data/model/IframeHandler.js

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -107,12 +107,6 @@ qx.Class.define("osparc.data.model.IframeHandler", {
107107
const loadingPage = new osparc.ui.message.Loading().set({
108108
header: this.__getLoadingPageHeader()
109109
});
110-
if (
111-
osparc.product.Utils.isProduct("s4llite") ||
112-
this.getStudy().getUi().getMode() === "standalone"
113-
) {
114-
loadingPage.setShowToolbar(false);
115-
}
116110

117111
const node = this.getNode();
118112
const thumbnail = node.getMetaData()["thumbnail"];

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

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1191,7 +1191,6 @@ qx.Class.define("osparc.desktop.WorkbenchView", {
11911191
this.nodeSelected(dynamicNode.getNodeId());
11921192
qx.event.Timer.once(() => {
11931193
this.__openIframeTab(dynamicNode);
1194-
dynamicNode.getLoadingPage().maximizeIFrame(true);
11951194
dynamicNode.getIFrame().maximizeIFrame(true);
11961195
}, this, 10);
11971196
return;

services/static-webserver/client/source/class/osparc/ui/message/Loading.js

Lines changed: 35 additions & 94 deletions
Original file line numberDiff line numberDiff line change
@@ -33,12 +33,15 @@ qx.Class.define("osparc.ui.message.Loading", {
3333

3434
construct: function() {
3535
this.base(arguments);
36-
this._setLayout(new qx.ui.layout.VBox());
3736

38-
this.set({
39-
alignX: "center",
40-
alignY: "middle",
41-
});
37+
const layout = new qx.ui.layout.Grid(20, 20);
38+
layout.setRowFlex(this.self().GRID_POS.SPACER_TOP, 1);
39+
layout.setRowFlex(this.self().GRID_POS.SPACER_BOTTOM, 1);
40+
layout.setColumnFlex(0, 1);
41+
layout.setColumnFlex(2, 1);
42+
layout.setColumnMaxWidth(1, 400);
43+
layout.setColumnAlign(1, "center", "middle");
44+
this._setLayout(layout);
4245

4346
this.__buildLayout();
4447
},
@@ -62,28 +65,6 @@ qx.Class.define("osparc.ui.message.Loading", {
6265
nullable: true,
6366
apply: "__applyMessages"
6467
},
65-
66-
toolbarHeight: {
67-
check: "Integer",
68-
init: 25
69-
},
70-
71-
/**
72-
* Show Restart-Maximize Toolbar
73-
*/
74-
showToolbar: {
75-
check: "Boolean",
76-
init: true,
77-
event: "changeShowToolbar",
78-
},
79-
},
80-
81-
// from osparc.widget.PersistentIframe
82-
events: {
83-
/** Fired if the iframe is restored from a minimized or maximized state */
84-
"restore" : "qx.event.type.Event",
85-
/** Fired if the iframe is maximized */
86-
"maximize" : "qx.event.type.Event"
8768
},
8869

8970
statics: {
@@ -93,10 +74,12 @@ qx.Class.define("osparc.ui.message.Loading", {
9374
STATUS_ICON_SIZE: 20,
9475

9576
GRID_POS: {
77+
SPACER_TOP: 0,
9678
LOGO: 1,
9779
WAITING: 2,
9880
MESSAGES: 3,
99-
EXTRA_WIDGETS: 4
81+
EXTRA_WIDGETS: 4,
82+
SPACER_BOTTOM: 5,
10083
}
10184
},
10285

@@ -108,44 +91,18 @@ qx.Class.define("osparc.ui.message.Loading", {
10891
__maxButton: null,
10992

11093
__buildLayout: function() {
111-
this.__createMaximizeButton();
112-
this.__createMainLayout();
113-
},
114-
115-
__createMaximizeButton: function() {
116-
const maximize = false;
117-
const maxButton = this.__maxButton = osparc.widget.PersistentIframe.createToolbarButton(maximize).set({
118-
label: osparc.widget.PersistentIframe.getZoomLabel(maximize),
119-
icon: osparc.widget.PersistentIframe.getZoomIcon(maximize),
120-
});
121-
osparc.utils.Utils.setIdToWidget(maxButton, osparc.widget.PersistentIframe.getMaximizeWidgetId(maximize));
122-
maxButton.addListener("execute", () => this.maximizeIFrame(!this.hasState("maximized")), this);
123-
124-
const buttonsContainer = new qx.ui.container.Composite(new qx.ui.layout.HBox(10).set({
125-
alignX: "right",
126-
alignY: "middle"
127-
}));
128-
this.bind("showToolbar", buttonsContainer, "visibility", {
129-
converter: showToolbar => showToolbar ? "visible" : "excluded"
130-
});
131-
buttonsContainer.add(maxButton);
132-
this._add(buttonsContainer);
133-
},
134-
135-
__createMainLayout: function() {
136-
const layout = new qx.ui.layout.Grid(20, 20);
137-
layout.setColumnFlex(0, 1);
138-
layout.setColumnAlign(0, "center", "middle");
139-
const mainLayout = new qx.ui.container.Composite(layout).set({
140-
maxWidth: 400,
141-
alignX: "center",
94+
this._add(new qx.ui.core.Spacer(), {
95+
column: 0,
96+
row: 0
14297
});
14398
this._add(new qx.ui.core.Spacer(), {
144-
flex: 1
99+
column: 2,
100+
row: 0
145101
});
146-
this._add(mainLayout);
102+
147103
this._add(new qx.ui.core.Spacer(), {
148-
flex: 1
104+
column: 1,
105+
row: this.self().GRID_POS.SPACER_TOP
149106
});
150107

151108
const productLogoPath = osparc.product.Utils.getLogoPath();
@@ -165,8 +122,8 @@ qx.Class.define("osparc.ui.message.Loading", {
165122
height: logoHeight
166123
});
167124
}
168-
mainLayout.add(thumbnail, {
169-
column: 0,
125+
this._add(thumbnail, {
126+
column: 1,
170127
row: this.self().GRID_POS.LOGO
171128
});
172129

@@ -178,37 +135,39 @@ qx.Class.define("osparc.ui.message.Loading", {
178135
gap: 15,
179136
allowGrowX: false
180137
});
138+
const icon = waitingHeader.getChildControl("icon");
139+
osparc.service.StatusUI.updateCircleAnimation(icon);
181140
const label = waitingHeader.getChildControl("label");
182141
label.set({
183142
rich: true,
184-
wrap: true
143+
wrap: true,
144+
alignX: "center",
185145
});
186-
const icon = waitingHeader.getChildControl("icon");
187-
osparc.service.StatusUI.updateCircleAnimation(icon);
188-
mainLayout.add(waitingHeader, {
189-
column: 0,
146+
this._add(waitingHeader, {
147+
column: 1,
190148
row: this.self().GRID_POS.WAITING
191149
});
192150

193151
const messages = this.__messagesContainer = new qx.ui.container.Composite(new qx.ui.layout.VBox(10).set({
194152
alignX: "center"
195153
}));
196-
mainLayout.add(messages, {
197-
column: 0,
154+
this._add(messages, {
155+
column: 1,
198156
row: this.self().GRID_POS.MESSAGES
199157
});
200158

201159
const extraWidgets = this.__extraWidgets = new qx.ui.container.Composite(new qx.ui.layout.VBox(10).set({
202160
alignX: "center"
203161
}));
204-
mainLayout.add(extraWidgets, {
205-
column: 0,
162+
this._add(extraWidgets, {
163+
column: 1,
206164
row: this.self().GRID_POS.EXTRA_WIDGETS
207165
});
208-
},
209166

210-
__applyShowToolbar: function(show) {
211-
this.setToolbarHeight(show ? 25 : 0);
167+
this._add(new qx.ui.core.Spacer(), {
168+
column: 1,
169+
row: this.self().GRID_POS.SPACER_BOTTOM
170+
});
212171
},
213172

214173
__applyLogo: function(newLogo) {
@@ -276,23 +235,5 @@ qx.Class.define("osparc.ui.message.Loading", {
276235
addExtraWidget: function(widget) {
277236
this.__extraWidgets.add(widget);
278237
},
279-
280-
// from osparc.widget.PersistentIframe
281-
maximizeIFrame: function(maximize) {
282-
if (maximize) {
283-
this.fireEvent("maximize");
284-
this.addState("maximized");
285-
} else {
286-
this.fireEvent("restore");
287-
this.removeState("maximized");
288-
}
289-
const maxButton = this.__maxButton;
290-
maxButton.set({
291-
label: osparc.widget.PersistentIframe.getZoomLabel(maximize),
292-
icon: osparc.widget.PersistentIframe.getZoomIcon(maximize)
293-
});
294-
osparc.utils.Utils.setIdToWidget(maxButton, osparc.widget.PersistentIframe.getMaximizeWidgetId(maximize));
295-
qx.event.message.Bus.getInstance().dispatchByName("maximizeIframe", this.hasState("maximized"));
296-
}
297238
}
298239
});

services/static-webserver/client/source/class/osparc/widget/PersistentIframe.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -234,7 +234,7 @@ qx.Class.define("osparc.widget.PersistentIframe", {
234234
right: (iframeParentPos.right - iframeParentPos.left - divPos.right)
235235
});
236236

237-
this.__buttonContainer.setVisibility(this.isShowToolbar() ? "visible" : "excluded");
237+
this.__buttonsContainer.setVisibility(this.isShowToolbar() ? "visible" : "excluded");
238238
}, 0);
239239
},
240240

0 commit comments

Comments
 (0)