Skip to content

Commit b718a71

Browse files
fix: resolve NcColorPicker focus-trap error and improve UX
- Remove palette-only prop to fix focus-trap error - Use @submit event instead of @update:value for better compatibility - Add fill-color to palette icon to show current selected color - Change button type to tertiary for proper styling Signed-off-by: Vitor Mattos <[email protected]>
1 parent 6b44239 commit b718a71

1 file changed

Lines changed: 3 additions & 4 deletions

File tree

src/Components/Draw/Editor.vue

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,11 +8,10 @@
88
<NcColorPicker ref="colorPicker"
99
v-model="color"
1010
:palette="customPalette"
11-
:palette-only="true"
12-
@input="updateColor">
13-
<NcButton>
11+
@submit="updateColor">
12+
<NcButton type="tertiary">
1413
<template #icon>
15-
<PaletteIcon :size="20" />
14+
<PaletteIcon :size="20" :fill-color="color" />
1615
</template>
1716
{{ t('libresign', 'Change color') }}
1817
</NcButton>

0 commit comments

Comments
 (0)