Skip to content

Commit a03dd3a

Browse files
committed
fix(ui): use useTextDirection from VueUse for RTL detection
Replace raw document.documentElement.dir check with VueUse's useTextDirection composable per review feedback. SSR-safe and consistent with how the rest of the codebase uses VueUse.
1 parent 70a9cdb commit a03dd3a

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

app/components/Chart/SplitSparkline.vue

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ import {
77
type VueUiXyDatasetItem,
88
} from 'vue-data-ui'
99
import { getPalette, lightenColor } from 'vue-data-ui/utils'
10+
import { useTextDirection } from '@vueuse/core'
1011
1112
import('vue-data-ui/style.css')
1213
@@ -66,10 +67,8 @@ const { colors } = useCssVariables(
6667
)
6768
6869
const isDarkMode = computed(() => resolvedMode.value === 'dark')
69-
const isRtl = computed(() => {
70-
if (import.meta.server) return false
71-
return document.documentElement.dir === 'rtl'
72-
})
70+
const dir = useTextDirection()
71+
const isRtl = computed(() => dir.value === 'rtl')
7372
7473
const datasets = computed<VueUiSparklineDatasetItem[][]>(() => {
7574
return (props.dataset ?? []).map(unit => {

0 commit comments

Comments
 (0)