[cleanup] Remove .checkbox CSS class and unused small parameter.

Bug: 388445687
Change-Id: I5389d6fed8f3127c99612d1c073fc179b570de2f
Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6164905
Auto-Submit: Benedikt Meurer <bmeurer@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Reviewed-by: Danil Somsikov <dsv@chromium.org>
This commit is contained in:
Benedikt Meurer
2025-01-12 02:15:21 -08:00
committed by Devtools-frontend LUCI CQ
parent c7b23e12d4
commit 4b78c22ab7
3 changed files with 3 additions and 5 deletions
+1 -1
View File
@@ -33,7 +33,7 @@ describeWithMockConnection('ConsoleView', () => {
it('adds a title to every checkbox label in the settings view', async () => {
const consoleSettingsCheckboxes =
consoleView.element.querySelector('devtools-toolbar')!.querySelectorAll('.toolbar-item.checkbox');
consoleView.element.querySelector('devtools-toolbar')!.querySelectorAll('dt-checkbox');
if (!consoleSettingsCheckboxes) {
assert.fail('No checkbox found in console settings');
return;
+1 -3
View File
@@ -1215,11 +1215,9 @@ export class ToolbarCheckbox extends ToolbarItem<void> {
constructor(
text: Common.UIString.LocalizedString, tooltip?: Common.UIString.LocalizedString,
listener?: ((arg0: MouseEvent) => void), jslogContext?: string, small?: boolean) {
listener?: ((arg0: MouseEvent) => void), jslogContext?: string) {
super(CheckboxLabel.create(text));
this.element.classList.add('checkbox');
this.inputElement = (this.element as CheckboxLabel).checkboxElement;
this.inputElement.classList.toggle('small', small);
if (tooltip) {
// install on the checkbox
Tooltip.install(this.inputElement, tooltip);
+1 -1
View File
@@ -1009,7 +1009,7 @@ devtools-toolbar {
/* Checkbox */
& > .checkbox {
& > dt-checkbox {
padding: 0 5px 0 0;
}