mirror of
https://github.com/grafana/grafana.git
synced 2025-02-25 18:55:37 -06:00
VariableEditorListRow: Adjust icon styles (#70014)
This commit is contained in:
parent
cda10fae52
commit
367faf8102
@ -79,14 +79,9 @@ export function VariableEditorListRow({
|
||||
</td>
|
||||
|
||||
<td role="gridcell" className={styles.column}>
|
||||
<div className={styles.icons}>
|
||||
<VariableCheckIndicator passed={passed} />
|
||||
</td>
|
||||
|
||||
<td role="gridcell" className={styles.column}>
|
||||
<VariableUsagesButton id={variable.id} isAdhoc={isAdHoc(variable)} usages={usagesNetwork} />
|
||||
</td>
|
||||
|
||||
<td role="gridcell" className={styles.column}>
|
||||
<IconButton
|
||||
onClick={(event) => {
|
||||
event.preventDefault();
|
||||
@ -97,9 +92,6 @@ export function VariableEditorListRow({
|
||||
tooltip="Duplicate variable"
|
||||
aria-label={selectors.pages.Dashboard.Settings.Variables.List.tableRowDuplicateButtons(variable.name)}
|
||||
/>
|
||||
</td>
|
||||
|
||||
<td role="gridcell" className={styles.column}>
|
||||
<IconButton
|
||||
onClick={(event) => {
|
||||
event.preventDefault();
|
||||
@ -110,11 +102,10 @@ export function VariableEditorListRow({
|
||||
tooltip="Remove variable"
|
||||
aria-label={selectors.pages.Dashboard.Settings.Variables.List.tableRowRemoveButtons(variable.name)}
|
||||
/>
|
||||
</td>
|
||||
<td role="gridcell" className={styles.column}>
|
||||
<div {...provided.dragHandleProps} className={styles.dragHandle}>
|
||||
<Icon name="draggabledots" size="lg" />
|
||||
</div>
|
||||
</div>
|
||||
</td>
|
||||
</tr>
|
||||
)}
|
||||
@ -166,6 +157,7 @@ function getStyles(theme: GrafanaTheme2) {
|
||||
return {
|
||||
dragHandle: css`
|
||||
cursor: grab;
|
||||
margin-left: ${theme.spacing(1)};
|
||||
`,
|
||||
column: css`
|
||||
width: 1%;
|
||||
@ -185,9 +177,16 @@ function getStyles(theme: GrafanaTheme2) {
|
||||
`,
|
||||
iconPassed: css`
|
||||
color: ${theme.v1.palette.greenBase};
|
||||
margin-right: ${theme.spacing(2)};
|
||||
`,
|
||||
iconFailed: css`
|
||||
color: ${theme.v1.palette.orange};
|
||||
margin-right: ${theme.spacing(2)};
|
||||
`,
|
||||
icons: css`
|
||||
display: flex;
|
||||
gap: ${theme.spacing(2)};
|
||||
align-items: center;
|
||||
`,
|
||||
};
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user