Skip to content

Commit 1a57879

Browse files
bmeurerDevtools-frontend LUCI CQ
authored andcommitted
[sources] Fix incorrect styling of expandable containers.
https://crrev.com/c/6189464 moved the `viewContainerStyles` onto the wrong widget (`ContainerWidget` vs. `ExpandableContainerWidget`). Fixed: 391829280 Change-Id: Ibc1df84556fbc20837694d1f8b2ee3cac00bb89f Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6194055 Commit-Queue: Benedikt Meurer <[email protected]> Reviewed-by: Kateryna Prokopenko <[email protected]> Auto-Submit: Benedikt Meurer <[email protected]> Commit-Queue: Kateryna Prokopenko <[email protected]>
1 parent 614261d commit 1a57879

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

front_end/ui/legacy/ViewManager.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -403,8 +403,6 @@ export class ContainerWidget extends VBox {
403403
}
404404

405405
override wasShown(): void {
406-
super.wasShown();
407-
this.registerCSSFiles([viewContainersStyles]);
408406
void this.materialize().then(() => {
409407
const widget = widgetForView.get(this.view);
410408
if (widget) {
@@ -454,6 +452,8 @@ class ExpandableContainerWidget extends VBox {
454452
}
455453

456454
override wasShown(): void {
455+
super.wasShown();
456+
this.registerCSSFiles([viewContainersStyles]);
457457
if (this.widget && this.materializePromise) {
458458
void this.materializePromise.then(() => {
459459
if (this.titleElement.classList.contains('expanded') && this.widget) {

0 commit comments

Comments
 (0)