Skip to content

Commit a700f4c

Browse files
Connor ClarkDevtools-frontend LUCI CQ
authored andcommitted
[RPP] Remove subtle background highlight for selected track
This was called out as potentially conflicting with the planned dimming behavior. Bug: 370836271 Change-Id: I7cb897e2232386c25daec0824500aa2ec5cff0e9 Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/5960127 Commit-Queue: Paul Irish <[email protected]> Reviewed-by: Paul Irish <[email protected]>
1 parent b50c1db commit a700f4c

File tree

9 files changed

+3
-20
lines changed

9 files changed

+3
-20
lines changed

front_end/ui/legacy/components/perf_ui/FlameChart.ts

Lines changed: 3 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -2174,16 +2174,6 @@ export class FlameChart extends Common.ObjectWrapper.eventMixin<EventTypes, type
21742174

21752175
const {markerIndices, colorBuckets, titleIndices} = this.getDrawableData(context, timelineData);
21762176

2177-
context.save();
2178-
this.forEachGroupInViewport((offset, index, group, isFirst, groupHeight) => {
2179-
if (this.isGroupFocused(index)) {
2180-
context.fillStyle =
2181-
ThemeSupport.ThemeSupport.instance().getComputedValue('--selected-group-background', this.contentElement);
2182-
context.fillRect(0, offset, canvasWidth, groupHeight - group.style.padding);
2183-
}
2184-
});
2185-
context.restore();
2186-
21872177
const groups = this.rawTimelineData?.groups || [];
21882178
const trackIndex = groups.findIndex(g => g.name.includes('Main'));
21892179
const group = groups.at(trackIndex);
@@ -2678,14 +2668,9 @@ export class FlameChart extends Common.ObjectWrapper.eventMixin<EventTypes, type
26782668
if (this.isGroupCollapsible(index) && !group.expanded || group.style.shareHeaderLine) {
26792669
// In edit mode, we draw an extra rectangle for the save icon.
26802670
const labelBackgroundWidth = this.labelWidthForGroup(context, group);
2681-
if (this.isGroupFocused(index)) {
2682-
context.fillStyle =
2683-
ThemeSupport.ThemeSupport.instance().getComputedValue('--selected-group-background', this.contentElement);
2684-
} else {
2685-
const parsedColor = Common.Color.parse(group.style.backgroundColor);
2686-
if (parsedColor) {
2687-
context.fillStyle = (parsedColor.setAlpha(0.8).asString() as string);
2688-
}
2671+
const parsedColor = Common.Color.parse(group.style.backgroundColor);
2672+
if (parsedColor) {
2673+
context.fillStyle = (parsedColor.setAlpha(0.8).asString() as string);
26892674
}
26902675
context.fillRect(
26912676
iconsWidth + HEADER_LEFT_PADDING, offset + HEADER_LABEL_Y_PADDING, labelBackgroundWidth,

front_end/ui/legacy/components/perf_ui/flameChart.css

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,12 +7,10 @@
77
.flame-chart-main-pane {
88
overflow: hidden;
99

10-
--selected-group-background: hsl(215deg 85% 98%);
1110
--selected-group-border: hsl(216deg 68% 54%);
1211
}
1312

1413
:host-context(.theme-with-dark-background) .flame-chart-main-pane {
15-
--selected-group-background: hsl(215deg 85% 15%);
1614
--selected-group-border: hsl(216deg 68% 46%);
1715
}
1816

93 Bytes
Loading
93 Bytes
Loading
77 Bytes
Loading
-2.62 KB
Loading
-123 Bytes
Loading
331 Bytes
Loading
177 Bytes
Loading

0 commit comments

Comments
 (0)