Skip to content

Commit 0043454

Browse files
authored
Merge pull request #361 from qbicsoftware/development
Release 0.24.0
2 parents 63bbb9f + e9d1ed2 commit 0043454

26 files changed

+782
-398
lines changed

.gitignore

+3
Original file line numberDiff line numberDiff line change
@@ -39,3 +39,6 @@ error-screenshots/
3939
/frontend/generated/vaadin-featureflags.ts
4040
/vaadinfrontend/vite.config.ts
4141
/vaadinfrontend/src/main/dev-bundle/
42+
43+
# Empty file that is created when changing css?
44+
/vaadinfrontend/frontend/themes/datamanager/theme-editor.css

vaadinfrontend/frontend/themes/datamanager/components/button.css

+6
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,12 @@ vaadin-button.primary .button:hover::before {
3030
opacity: 0.05;
3131
}
3232

33+
vaadin-button.tertiary {
34+
padding: 0 calc(var(--lumo-button-size) / 6);
35+
background-color: transparent !important;
36+
min-width: 0;
37+
}
38+
3339
vaadin-button {
3440
cursor: pointer;
3541
}

vaadinfrontend/frontend/themes/datamanager/components/custom.css

+1
Original file line numberDiff line numberDiff line change
@@ -9,3 +9,4 @@
99
@import "page-area.css";
1010
@import "span.css";
1111
@import "vaadin-custom.css";
12+

vaadinfrontend/frontend/themes/datamanager/components/dialog.css

+72-3
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,66 @@ vaadin-dialog-overlay::part(title) {
1818
margin-inline-start: 0;
1919
}
2020

21+
/* set the width of the notification */
22+
.notification-dialog::part(overlay) {
23+
width: 36.75rem;
24+
}
25+
26+
.notification-dialog .content {
27+
width: 100%;
28+
}
29+
30+
/* notification dialog header */
31+
.notification-dialog > [slot="header"] {
32+
display: flex;
33+
align-items: center;
34+
justify-items: flex-start;
35+
gap: var(--lumo-space-s);
36+
}
37+
38+
.notification-dialog .title {
39+
font-family: var(--lumo-font-family);
40+
line-height: var(--lumo-line-height-m);
41+
font-size: var(--lumo-font-size-xl);
42+
margin: 0;
43+
}
44+
45+
/* icon in header */
46+
.notification-dialog > [slot="header"] vaadin-icon {
47+
width: 1.83rem;
48+
height: 1.83rem;
49+
margin: 0.46rem;
50+
margin-inline-start: calc(var(--lumo-space-l) - var(--lumo-space-m));
51+
}
52+
53+
.notification-dialog .error-icon {
54+
fill: var(--lumo-error-color);
55+
}
56+
57+
.notification-dialog .warning-icon {
58+
fill: var(--lumo-warning-color);
59+
}
60+
61+
.notification-dialog .info-icon {
62+
fill: var(--lumo-primary-color);
63+
}
64+
65+
/* experimental groups exist dialog on variable edit */
66+
.existing-groups-prevent-variable-edit .content .experimental-group-count {
67+
font-weight: bold;
68+
}
69+
70+
/* samples exist on variable edit */
71+
.existing-samples-prevent-variable-edit .content .sample-count {
72+
font-weight: bold;
73+
}
74+
75+
/* samples exist on group edit */
76+
.existing-samples-prevent-group-edit .content .sample-count {
77+
font-weight: bold;
78+
}
79+
80+
2181
.add-user-to-project-dialog::part(overlay) {
2282
height: 100%;
2383
min-width: 66vw;
@@ -27,8 +87,14 @@ vaadin-dialog-overlay::part(title) {
2787
height: 100%
2888
}
2989

90+
.batch-registration-dialog vaadin-tabsheet {
91+
height: 100%;
92+
width: 100%;
93+
}
94+
3095
.batch-registration-dialog::part(overlay) {
3196
min-width: 66vw;
97+
height: 100%;
3298
}
3399

34100
.batch-registration-dialog .stepper vaadin-tabs {
@@ -54,7 +120,6 @@ vaadin-dialog-overlay::part(title) {
54120

55121
.batch-registration-dialog .batch-content .data-type-information {
56122
display: flex;
57-
height: inherit;
58123
flex-direction: column;
59124
}
60125

@@ -86,10 +151,14 @@ vaadin-dialog-overlay::part(title) {
86151
gap: var(--lumo-space-s);
87152
}
88153

154+
.batch-registration-dialog .sample-spreadsheet-container {
155+
width: 100%;
156+
height: 100%;
157+
}
158+
89159
.batch-registration-dialog .sample-spreadsheet {
90-
/*Todo Find solution on how to let spreadsheet set its size*/
91160
width: 100%;
92-
height: 60vh;
161+
height: 100%;
93162
}
94163

95164
.create-project-dialog .content {

vaadinfrontend/frontend/themes/datamanager/components/div.css

-1
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@
77
.disclaimer {
88
display: flex;
99
justify-content: center;
10-
padding: var(--lumo-space-s);
1110
width: 100%;
1211
height: 100%;
1312
}

vaadinfrontend/frontend/themes/datamanager/components/main.css

+7-3
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,11 @@
55

66
.main.experiment {
77
grid-template-columns: minmax(min-content, 80%) minmax(min-content, 20%);
8-
grid-template-rows: minmax(130px, 5%) minmax(max-content, 90%);
8+
grid-template-rows: minmax(130px, auto) 1fr;
99
grid-template-areas:
1010
"navbar support"
1111
"content support";
12+
height: 100%;
1213
}
1314

1415
.main.experiment .content {
@@ -21,14 +22,16 @@
2122

2223
.main.project {
2324
grid-template-columns: minmax(min-content, 80%) minmax(min-content, 20%);
24-
grid-template-rows: minmax(130px, 5%) minmax(min-content, 90%);
25+
grid-template-rows: minmax(130px, auto) 1fr;
2526
grid-template-areas:
2627
"navbar support"
2728
"content support";
29+
height: 100%;
2830
}
2931

3032
.main.project .content {
3133
grid-area: content;
34+
height: 100%;
3235
}
3336

3437
.main.project .support {
@@ -37,10 +40,11 @@
3740

3841
.main.sample {
3942
grid-template-columns: minmax(max-content, 100%) minmax(min-content, 20%);
40-
grid-template-rows: minmax(130px, 5%) minmax(min-content, 90%);
43+
grid-template-rows: minmax(130px, auto) 1fr;
4144
grid-template-areas:
4245
"navbar support"
4346
"content content";
47+
height: 100%;
4448
}
4549

4650
.main.sample .content {

vaadinfrontend/frontend/themes/datamanager/components/page-area.css

+78-2
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@
2525
flex-direction: column;
2626
display: flex;
2727
gap: var(--lumo-space-m);
28+
height: 100%;
2829
}
2930

3031
.page-area.attachments-area .delete-button {
@@ -42,24 +43,78 @@
4243
gap: var(--lumo-space-xs);
4344
}
4445

46+
.page-area.experiment-details-component {
47+
height: 100%;
48+
display: flex;
49+
flex-direction: column;
50+
}
51+
52+
.page-area.experiment-details-component .sample-source-display {
53+
flex-direction: row;
54+
display: flex;
55+
}
56+
57+
.page-area.experiment-details-component .sample-source-display .icon-with-list {
58+
flex-direction: row;
59+
flex-grow: 0.2;
60+
display: flex;
61+
}
62+
63+
.page-area.experiment-details-component .sample-source-display .icon-with-list .taglist {
64+
flex-direction: column;
65+
display: flex;
66+
line-height: 1.3em;
67+
}
68+
69+
.page-area.experiment-details-component .sample-source-display .icon-with-list .taglist .title {
70+
font-size: small;
71+
}
72+
73+
.page-area.experiment-details-component .sample-source-display .icon-with-list vaadin-icon {
74+
color: var(--lumo-primary-color);
75+
margin-right: 1em;
76+
margin-top: 0.5em;
77+
}
78+
4579
.page-area.experiment-details-component .header {
4680
display: flex;
4781
justify-content: space-between;
4882
align-items: baseline;
4983
}
5084

51-
.page-area.experiment-details-component .content {
85+
.page-area.experiment-details-component .details-content {
5286
padding: var(--lumo-space-m);
5387
display: flex;
5488
flex-direction: column;
89+
flex-grow: 1;
5590
gap: var(--lumo-space-m);
5691
font-size: var(--lumo-font-size-s);
92+
height: 100%;
93+
}
94+
95+
.page-area.experiment-details-component vaadin-tabsheet {
96+
height: 100%;
97+
width: 100%;
98+
}
99+
100+
.page-area.experiment-details-component .details-content .experimental-groups-container {
101+
height: 100%;
102+
width: 100%;
103+
}
104+
105+
.page-area.experiment-details-component .details-content .experimental-variables-container {
106+
height: 100%;
107+
width: 100%;
57108
}
58109

59110
.page-area.experiment-details-component .sample-registration-possible {
60111
margin-bottom: var(--lumo-space-m);
61112
}
62113

114+
.page-area.experiment-list-component {
115+
height: 100%;
116+
}
117+
63118
.page-area.project-access-component {
64119
width: 100%;
65120
height: 100%;
@@ -85,6 +140,10 @@
85140
align-items: baseline;
86141
}
87142

143+
.page-area.project-details-component {
144+
height: 100%;
145+
}
146+
88147
.page-area.project-details-component .details-content {
89148
padding: var(--lumo-space-m);
90149
display: flex;
@@ -111,10 +170,17 @@
111170
color: var(--lumo-secondary-text-color);
112171
}
113172

114-
.page-area.sample-details-component .content {
173+
.page-area.sample-details-component {
174+
height: 100%;
175+
display: flex;
176+
flex-direction: column;
177+
}
178+
179+
.page-area.sample-details-component .sample-details-content {
115180
display: flex;
116181
flex-direction: column;
117182
margin-bottom: var(--lumo-space-m);
183+
height: 100%;
118184
}
119185

120186
.page-area.sample-details-component .button-and-search-bar {
@@ -135,3 +201,13 @@
135201
display: inline-flex;
136202
align-items: end;
137203
}
204+
205+
.page-area.sample-details-component .sample-grid {
206+
height: 100%;
207+
width: 100%;
208+
}
209+
210+
.page-area.sample-details-component .sample-tab-content {
211+
height: 100%;
212+
width: 100%;
213+
}

vaadinfrontend/src/main/java/life/qbic/datamanager/views/general/DialogWindow.java

+1-22
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ public class DialogWindow extends Dialog {
1919
protected DialogWindow() {
2020
this.addClassName("dialog-window");
2121
confirmButton.addClassName("primary");
22+
cancelButton.setThemeName("tertiary");
2223
setCloseOnOutsideClick(false);
2324
setCloseOnEsc(false);
2425
}
@@ -38,26 +39,4 @@ public void setConfirmButtonLabel(String confirmLabel){
3839
public void setCancelButtonLabel(String cancelLabel){
3940
cancelButton.setText(cancelLabel);
4041
}
41-
42-
protected static class Container<T> {
43-
44-
private T value;
45-
46-
/**
47-
* Returns the value stored in this container
48-
* @return
49-
*/
50-
public T value() {
51-
return this.value;
52-
}
53-
54-
/**
55-
* Sets the value stored in this container
56-
* @return
57-
*/
58-
public void setValue(T newValue) {
59-
this.value = newValue;
60-
}
61-
62-
}
6342
}

0 commit comments

Comments
 (0)