Skip to content

Commit 5a855b1

Browse files
jackfranklinDevtools-frontend LUCI CQ
authored andcommitted
RPP: Fix focus outline overlap on "Learn more" link
Swapping from padding to margin means that the focus outline no longer overlaps the text. As a drive-by, I left a comment to explain why we use inline styles here as it tripped me up at first :) Fixed: 375982185 Change-Id: I855e51bb8e69ab3fb5cfd33099841c9154a4f6c4 Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6148228 Auto-Submit: Jack Franklin <[email protected]> Reviewed-by: Andres Olivares <[email protected]> Commit-Queue: Andres Olivares <[email protected]>
1 parent 3e94c0d commit 5a855b1

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

front_end/panels/timeline/TimelinePanel.ts

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1275,7 +1275,10 @@ export class TimelinePanel extends UI.Panel.Panel implements Client, TimelineMod
12751275

12761276
const localLink = UI.XLink.XLink.create(
12771277
'https://developer.chrome.com/docs/devtools/performance/extension', i18nString(UIStrings.learnMore));
1278-
localLink.style.paddingLeft = '5px';
1278+
// Has to be done in JS because the element is inserted into the
1279+
// checkbox's shadow DOM so any styling into timelinePanel.css would
1280+
// not apply.
1281+
localLink.style.marginLeft = '5px';
12791282
thirdPartyCheckbox.element.shadowRoot?.appendChild(localLink);
12801283
thirdPartyToolbar.appendToolbarItem(thirdPartyCheckbox);
12811284

0 commit comments

Comments
 (0)