ckaczor/azuredatastudio
1
0
Fork 0
mirror of https://github.com/ckaczor/azuredatastudio.git synced 2026-01-23 17:23:02 -05:00
Code Issues Packages Projects Releases Wiki Activity

fix chart's checkbox option type (#15014)

Browse Source 
* fix chart's checkbox option type

* overwrite the useragent margin for checkbox
This commit is contained in:
Alan Ren
2021-04-07 10:19:00 -07:00
committed by GitHub
parent addef2d584
commit 7fc991b678
3 changed files with 15 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
src/sql/workbench/contrib/charts/browser/chartView.ts
View File

@@ -317,11 +317,13 @@ export class ChartView extends Disposable implements IPanelView {
}
private createOption(option: IChartOption, container: HTMLElement) {
const label = DOM.$('div.option-label');
label.innerText = option.label;
const optionContainer = DOM.$('div.option-container');
const optionInput = DOM.$('div.option-input');
optionContainer.appendChild(label);
if (option.type !== ControlType.checkbox) {
const label = DOM.$('div.option-label');
label.innerText = option.label;
optionContainer.appendChild(label);
}
optionContainer.appendChild(optionInput);
let setFunc: ((val: string) => void) | ((val: number) => void) | ((val: boolean) => void);
let entry = option.configEntry as keyof IInsightOptions;
@@ -329,7 +331,7 @@ export class ChartView extends Disposable implements IPanelView {
switch (option.type) {
case ControlType.checkbox:
let checkbox = new Checkbox(optionInput, {
label: '',
label: option.label,
ariaLabel: option.label,
checked: value,
onChange: () => {

4
src/sql/workbench/contrib/charts/browser/media/chartView.css
View File

@@ -31,3 +31,7 @@
min-width: 250px;
padding-right: 10px;
}
.option-container {
margin-bottom: 5px;
}