Skip to content

WIP#1096

Draft
MCFK wants to merge 1 commit intomainfrom
refactor/ftable-checkbox-not-editable
Draft

WIP#1096
MCFK wants to merge 1 commit intomainfrom
refactor/ftable-checkbox-not-editable

Conversation

@MCFK
Copy link
Collaborator

@MCFK MCFK commented Mar 11, 2026

Ej för granskning (just nu) endast för diskussion.

@ext
Copy link
Contributor

ext commented Mar 11, 2026

Det är så här jag hade förväntat mig att det fungerar när den inte är redigerbar. Att den bara visar dess state.

</td>
<td v-else ref="target" tabindex="-1" class="table-ng__cell table-ng__cell--checkbox">
<input :checked="Boolean(column.checked(row))" type="checkbox" :aria-label />
<f-icon v-if="checked" :aria-label name="success"></f-icon>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Även detta bara för diskussion, jag tycker fortfarande att vi borde ha en mappning mellan komponent och ikon. Ungefär som semantiska färger.

En komponent nyttjar token checkbox-enabled som sen är mappad mot ikonen success.

Annars blir det så svårt att ändra ikoner, i praktiken kan inte jag använda en success ikon som det ut som 🎉 för det passar inte för en checkbox.

@github-actions
Copy link

CHANGELOG

No commits contributing to a release was found, no changelog entries will be added for this Pull Request.

If this PR should be included in a release amend the commit message(s) to use:

  • fix: for a patch version or
  • feat: for minor version.

Read more about the release process (swedish).

@github-actions
Copy link

Förhandsgranskning 🐛 🔍

Dokumentation och exampel applikationer finns att förhandsgranska på:

Sida URL
Dokumentation https://Forsakringskassan.github.io/designsystem/pr-preview/pr-1096/
Vue Sandbox https://Forsakringskassan.github.io/designsystem/pr-preview/pr-1096/vue-sandbox
Applikationsmall https://Forsakringskassan.github.io/designsystem/pr-preview/pr-1096/examples/page-layout

Senast uppdaterad 2026-03-11 14:54 UTC i gh-pages.

@github-actions
Copy link

Artifact sizes

Artifact sizes in this build (unchanged artifacts collapsed below).

Artifact Files Size Change
@fkui/vue-labs (esm) 1 file(s) 248.8KiB → 248.8KiB (-39B) -0.02%
7 unchanged artifacts
Artifact Files Size
@fkui/date (esm) 1 file(s) 32.2KiB
@fkui/design 2 file(s) 189.8KiB
@fkui/logic (esm) 1 file(s) 162.3KiB
@fkui/vue (esm) 1 file(s) 1.1MiB
@fkui/vue (FBadge bundle) 1 file(s) 550.5KiB
@fkui/vue/cypress 1 file(s) 56.0KiB
@fkui/vue/selectors 1 file(s) 3.3KiB

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants