Skip to content

Commit d265357

Browse files
committed
fix: data keys not available by default when loading text widget, without changing tabs
1 parent 69dac7d commit d265357

File tree

4 files changed

+18
-8
lines changed

4 files changed

+18
-8
lines changed

libs/shared/src/lib/components/ui/reference-data-grid/reference-data-grid.component.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,6 @@ export class ReferenceDataGridComponent implements OnInit {
106106
});
107107
const sortField = this.sortField;
108108
const sortOrder = this.sortOrder;
109-
console.log(sortOrder);
110109
if (sortField) {
111110
this.gridData = {
112111
data: (sortOrder === 'asc'
@@ -158,7 +157,6 @@ export class ReferenceDataGridComponent implements OnInit {
158157
* @param sort Sort event.
159158
*/
160159
public onSortChange(sort: SortDescriptor[]): void {
161-
console.log('sorting');
162160
this.sort = sort;
163161
this.skip = 0;
164162
this.onPageChange({ skip: this.skip, take: this.pageSize });

libs/shared/src/lib/components/widgets/editor-settings/editor-settings.component.html

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,8 @@
1818
<input formControlName="title" type="string" />
1919
</div>
2020
</div>
21-
<editor [init]="editor" formControlName="text"></editor>
21+
<editor *ngIf="!loading" [init]="editor" formControlName="text"></editor>
22+
<ui-spinner *ngIf="loading"></ui-spinner>
2223
</form>
2324
</ng-template>
2425
</ui-tab>

libs/shared/src/lib/components/widgets/editor-settings/editor-settings.component.ts

Lines changed: 14 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -54,6 +54,8 @@ export class EditorSettingsComponent
5454
public referenceData: ReferenceData | null = null;
5555
/** Current layout */
5656
public layout: Layout | null = null;
57+
/** Loading indicator */
58+
public loading = true;
5759

5860
/**
5961
* Modal content for the settings of the editor widgets.
@@ -134,6 +136,10 @@ export class EditorSettingsComponent
134136
this.referenceData = null;
135137
}
136138
});
139+
if (!resourceID && !referenceDataID) {
140+
this.updateFields();
141+
this.loading = false;
142+
}
137143
}
138144

139145
/**
@@ -154,7 +160,6 @@ export class EditorSettingsComponent
154160
this.widget.settings.showDataSourceLink =
155161
this.widgetFormGroup.value.showDataSourceLink;
156162
});
157-
this.updateFields();
158163
}
159164

160165
/**
@@ -164,6 +169,7 @@ export class EditorSettingsComponent
164169
* @param layout selected layout id ( optional )
165170
*/
166171
private getResource(resource: string, layout?: string | null) {
172+
this.loading = true;
167173
this.apollo
168174
.query<ResourceQueryResponse>({
169175
query: GET_RESOURCE,
@@ -172,16 +178,17 @@ export class EditorSettingsComponent
172178
layout: layout ? [layout] : undefined,
173179
},
174180
})
175-
.subscribe((res) => {
176-
if (res.data) {
177-
this.resource = res.data.resource;
181+
.subscribe(({ data }) => {
182+
if (data) {
183+
this.resource = data.resource;
178184
if (layout) {
179-
this.layout = res.data?.resource.layouts?.edges[0]?.node || null;
185+
this.layout = data.resource.layouts?.edges[0]?.node || null;
180186
} else {
181187
this.layout = null;
182188
}
183189
this.updateFields();
184190
}
191+
this.loading = false;
185192
});
186193
}
187194

@@ -191,6 +198,7 @@ export class EditorSettingsComponent
191198
* @param referenceData reference data id
192199
*/
193200
private getReferenceData(referenceData: string) {
201+
this.loading = true;
194202
this.apollo
195203
.query<ReferenceDataQueryResponse>({
196204
query: GET_REFERENCE_DATA,
@@ -203,6 +211,7 @@ export class EditorSettingsComponent
203211
this.referenceData = data.referenceData;
204212
this.updateFields();
205213
}
214+
this.loading = false;
206215
});
207216
}
208217

libs/shared/src/lib/components/widgets/editor-settings/editor-settings.module.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ import {
1616
RadioModule,
1717
SelectMenuModule,
1818
SelectOptionModule,
19+
SpinnerModule,
1920
TabsModule,
2021
ToggleModule,
2122
TooltipModule,
@@ -56,6 +57,7 @@ import {
5657
ToggleModule,
5758
ResourceSelectComponent,
5859
ReferenceDataSelectComponent,
60+
SpinnerModule,
5961
],
6062
exports: [EditorSettingsComponent],
6163
providers: [

0 commit comments

Comments
 (0)