Skip to content

Commit

Permalink
Update checkbox to use icon font check #86708
Browse files Browse the repository at this point in the history
  • Loading branch information
Miguel Solorio committed Dec 11, 2019
1 parent a4880b1 commit cc70266
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 13 deletions.
3 changes: 0 additions & 3 deletions src/vs/base/browser/ui/checkbox/check-dark.svg

This file was deleted.

3 changes: 0 additions & 3 deletions src/vs/base/browser/ui/checkbox/check-light.svg

This file was deleted.

9 changes: 3 additions & 6 deletions src/vs/base/browser/ui/checkbox/checkbox.css
Original file line number Diff line number Diff line change
Expand Up @@ -44,10 +44,7 @@
background-size: 16px !important;
}

.monaco-custom-checkbox.monaco-simple-checkbox.checked {
background: url('check-light.svg') center center no-repeat;
}

.monaco-custom-checkbox.monaco-simple-checkbox.checked {
background: url('check-dark.svg') center center no-repeat;
/* hide check when unchecked */
.monaco-custom-checkbox.monaco-simple-checkbox.unchecked:not(.checked)::before {
visibility: hidden;;
}
2 changes: 1 addition & 1 deletion src/vs/base/browser/ui/checkbox/checkbox.ts
Original file line number Diff line number Diff line change
Expand Up @@ -192,7 +192,7 @@ export class SimpleCheckbox extends Widget {
constructor(private title: string, private isChecked: boolean) {
super();

this.checkbox = new Checkbox({ title: this.title, isChecked: this.isChecked, actionClassName: 'monaco-simple-checkbox' });
this.checkbox = new Checkbox({ title: this.title, isChecked: this.isChecked, actionClassName: 'monaco-simple-checkbox codicon-check' });

this.domNode = this.checkbox.domNode;

Expand Down

1 comment on commit cc70266

@miguelsolorio
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FYI @sbatten

Please sign in to comment.