Frontend: Fix frontend styleguide emotion example (#70371)

fix wrong example
This commit is contained in:
Sven Grossmann
2023-06-20 12:31:38 +02:00
committed by GitHub
parent ae1da1a38d
commit 39a18ca6ba

View File

@@ -39,8 +39,9 @@ can have your getStyles function return an object with many class names and use
Let's say you need to style a component that has a different background depending on the `isActive` property : Let's say you need to style a component that has a different background depending on the `isActive` property :
```tsx ```tsx
import React from 'react';
import { css, cx } from '@emotion/css'; import { css, cx } from '@emotion/css';
import React from 'react';
import { GrafanaTheme2 } from '@grafana/data'; import { GrafanaTheme2 } from '@grafana/data';
import { useStyles2 } from '@grafana/ui'; import { useStyles2 } from '@grafana/ui';
@@ -49,8 +50,7 @@ interface ComponentAProps {
} }
const ComponentA = ({ isActive }: ComponentAProps) => { const ComponentA = ({ isActive }: ComponentAProps) => {
const theme = useTheme(); const styles = useStyles2(getStyles);
const styles = useStyles2(theme);
return ( return (
<div className={cx(styles.wrapper, isActive && styles.active)}> <div className={cx(styles.wrapper, isActive && styles.active)}>
@@ -64,15 +64,15 @@ const ComponentA = ({ isActive }: ComponentAProps) => {
const getStyles = (theme: GrafanaTheme2) => { const getStyles = (theme: GrafanaTheme2) => {
return { return {
wrapper: css({ wrapper: css({
background: theme.colors.background.secondary; background: theme.colors.background.secondary,
}), }),
active: css({ active: css({
background: theme.colors.primary.main, background: theme.colors.primary.main,
text: theme.colors.primary.contrastText, text: theme.colors.primary.contrastText,
}, }),
icon: css({ icon: css({
fontSize: theme.typography.bodySmall.fontSize; fontSize: theme.typography.bodySmall.fontSize,
}) }),
}; };
}; };
``` ```