[RPP]: Make one row Interactions and Animations tracks non expandable [devtools/devtools-frontend : main]

0 views
Skip to first unread message

Paul Irish (Gerrit)

unread,
Feb 9, 2026, 12:34:45 PMFeb 9
to Jack Franklin, devtools-rev...@chromium.org
Attention needed from Jack Franklin

Paul Irish voted

Auto-Submit+1
Commit-Queue+1
Open in Gerrit

Related details

Attention is currently required from:
  • Jack Franklin
Submit Requirements:
  • requirement satisfiedCode-Owners
  • requirement is not satisfiedCode-Review
  • requirement is not satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: devtools/devtools-frontend
Gerrit-Branch: main
Gerrit-Change-Id: I69bc2fb8a45e8a7cdeed88ab263721a337507b38
Gerrit-Change-Number: 7558793
Gerrit-PatchSet: 1
Gerrit-Owner: Paul Irish <paul...@chromium.org>
Gerrit-Reviewer: Jack Franklin <jacktf...@chromium.org>
Gerrit-Reviewer: Paul Irish <paul...@chromium.org>
Gerrit-Attention: Jack Franklin <jacktf...@chromium.org>
Gerrit-Comment-Date: Mon, 09 Feb 2026 17:34:43 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
unsatisfied_requirement
open
diffy

Jack Franklin (Gerrit)

unread,
Feb 10, 2026, 4:14:13 AMFeb 10
to Paul Irish, Devtools-frontend LUCI CQ, devtools-rev...@chromium.org
Attention needed from Paul Irish

Jack Franklin voted and added 1 comment

Votes added by Jack Franklin

Code-Review+1

1 comment

Patchset-level comments
File-level comment, Patchset 2 (Latest):
Jack Franklin . resolved

love it % test updates, thanks!

Open in Gerrit

Related details

Attention is currently required from:
  • Paul Irish
Submit Requirements:
  • requirement satisfiedCode-Owners
  • requirement satisfiedCode-Review
  • requirement satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: devtools/devtools-frontend
Gerrit-Branch: main
Gerrit-Change-Id: I69bc2fb8a45e8a7cdeed88ab263721a337507b38
Gerrit-Change-Number: 7558793
Gerrit-PatchSet: 2
Gerrit-Owner: Paul Irish <paul...@chromium.org>
Gerrit-Reviewer: Jack Franklin <jacktf...@chromium.org>
Gerrit-Reviewer: Paul Irish <paul...@chromium.org>
Gerrit-CC: Devtools-frontend LUCI CQ <devtools-fro...@luci-project-accounts.iam.gserviceaccount.com>
Gerrit-Attention: Paul Irish <paul...@chromium.org>
Gerrit-Comment-Date: Tue, 10 Feb 2026 09:14:08 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Jack Franklin (Gerrit)

unread,
Feb 10, 2026, 12:07:25 PMFeb 10
to Paul Irish, Devtools-frontend LUCI CQ, devtools-rev...@chromium.org
Attention needed from Paul Irish

Jack Franklin voted and added 1 comment

Votes added by Jack Franklin

Code-Review+1

1 comment

Patchset-level comments
File-level comment, Patchset 4 (Latest):
Jack Franklin . resolved

lgtm

Open in Gerrit

Related details

Attention is currently required from:
  • Paul Irish
Submit Requirements:
  • requirement satisfiedCode-Owners
  • requirement satisfiedCode-Review
  • requirement satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: devtools/devtools-frontend
Gerrit-Branch: main
Gerrit-Change-Id: I69bc2fb8a45e8a7cdeed88ab263721a337507b38
Gerrit-Change-Number: 7558793
Gerrit-PatchSet: 4
Gerrit-Owner: Paul Irish <paul...@chromium.org>
Gerrit-Reviewer: Jack Franklin <jacktf...@chromium.org>
Gerrit-Reviewer: Paul Irish <paul...@chromium.org>
Gerrit-CC: Devtools-frontend LUCI CQ <devtools-fro...@luci-project-accounts.iam.gserviceaccount.com>
Gerrit-Attention: Paul Irish <paul...@chromium.org>
Gerrit-Comment-Date: Tue, 10 Feb 2026 17:07:21 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Paul Irish (Gerrit)

unread,
Feb 10, 2026, 1:50:03 PMFeb 10
to Jack Franklin, Devtools-frontend LUCI CQ, devtools-rev...@chromium.org

Paul Irish voted

Auto-Submit+1
Commit-Queue+2
Open in Gerrit

Related details

Attention set is empty
Submit Requirements:
  • requirement satisfiedCode-Owners
  • requirement satisfiedCode-Review
  • requirement satisfiedReview-Enforcement
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: comment
Gerrit-Project: devtools/devtools-frontend
Gerrit-Branch: main
Gerrit-Change-Id: I69bc2fb8a45e8a7cdeed88ab263721a337507b38
Gerrit-Change-Number: 7558793
Gerrit-PatchSet: 4
Gerrit-Owner: Paul Irish <paul...@chromium.org>
Gerrit-Reviewer: Jack Franklin <jacktf...@chromium.org>
Gerrit-Reviewer: Paul Irish <paul...@chromium.org>
Gerrit-CC: Devtools-frontend LUCI CQ <devtools-fro...@luci-project-accounts.iam.gserviceaccount.com>
Gerrit-Comment-Date: Tue, 10 Feb 2026 18:50:00 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
satisfied_requirement
open
diffy

Devtools-frontend LUCI CQ (Gerrit)

unread,
Feb 11, 2026, 2:11:02 AMFeb 11
to Paul Irish, Jack Franklin, devtools-rev...@chromium.org

Devtools-frontend LUCI CQ submitted the change

Change information

Commit message:
[RPP]: Make one row Interactions and Animations tracks non expandable

This CL updates the Interactions and Animations tracks to use
GroupCollapsibleState.IF_MULTI_ROW. This ensures that the
expand/collapse arrow is hidden if the track content fits within a
single row, reducing visual noise and aligning behavior with the
Timings and Custom tracks.
Bug: 481076667, 443216198
Fixed: 481076667
Change-Id: I69bc2fb8a45e8a7cdeed88ab263721a337507b38
Reviewed-by: Jack Franklin <jacktf...@chromium.org>
Auto-Submit: Paul Irish <paul...@chromium.org>
Commit-Queue: Paul Irish <paul...@chromium.org>
Files:
  • M front_end/panels/timeline/AnimationsTrackAppender.ts
  • M front_end/panels/timeline/InteractionsTrackAppender.ts
  • M test/goldens/linux/timeline/animations_track.png
  • M test/goldens/linux/timeline/flamechart_view_network_collapsed.png
  • M test/goldens/linux/timeline/flamechart_view_network_expanded.png
  • M test/goldens/linux/timeline/interactions_track.png
  • M test/goldens/linux/timeline/interactions_track_candystripe.png
  • M test/goldens/linux/timeline/timeline_with_main_thread_selection.png
  • M test/goldens/linux/timeline/timeline_with_network_selection.png
Change size: XS
Delta: 9 files changed, 5 insertions(+), 2 deletions(-)
Branch: refs/heads/main
Submit Requirements:
  • requirement satisfiedCode-Review: +1 by Jack Franklin
Open in Gerrit
Inspect html for hidden footers to help with email filtering. To unsubscribe visit settings. DiffyGerrit
Gerrit-MessageType: merged
Gerrit-Project: devtools/devtools-frontend
Gerrit-Branch: main
Gerrit-Change-Id: I69bc2fb8a45e8a7cdeed88ab263721a337507b38
Gerrit-Change-Number: 7558793
Gerrit-PatchSet: 5
Gerrit-Owner: Paul Irish <paul...@chromium.org>
Gerrit-Reviewer: Devtools-frontend LUCI CQ <devtools-fro...@luci-project-accounts.iam.gserviceaccount.com>
open
diffy
satisfied_requirement
Reply all
Reply to author
Forward
0 new messages