Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

custom vl tabs component met content slot #181

Open
roefem opened this issue Nov 28, 2023 · 3 comments
Open

custom vl tabs component met content slot #181

roefem opened this issue Nov 28, 2023 · 3 comments

Comments

@roefem
Copy link
Contributor

roefem commented Nov 28, 2023

Custom component ter vervanging van vl-tabs, maar met een slot waarin content gerenderd wordt.
Gelijkaardig aan de OeContainer, voordeel is dat vue-router child routes eenvoudiger gebruikt kunnen worden om tabs te renderen

@wouter-adriaens
Copy link
Contributor

wouter-adriaens commented Dec 1, 2023

Volgens mij kunnen we met de laatste versie van de vl-tabs dit probleem oplossen.

Voorbeeld met routing en met "@govflanders/vl-ui-design-system-vue3": "^1.0.2-alpha.0":

<vl-tabs
  :key="activeDossier?.id"
  :active-tab="activeTab"
  :hash-change="false"
  class="vl-u-flex vl-u-flex-direction-column oe-flex-1"
  @tab-change="onTabChange"
>
  <router-view v-slot="{ Component, route }">
    <vl-tab v-for="tab in tabs" :id="tab.id" :key="tab.id" :label="tab.label" :class="tab.class">
      <keep-alive>
        <component :is="Component" v-if="route.name === tab.id" />
      </keep-alive>
    </vl-tab>
  </router-view>
</vl-tabs>

Route te updaten in onTabChange

Te bekijken of dit aan de verwachtingen voldoet en of dit ticket alsnog relevant is.

@cedrikv cedrikv modified the milestones: Sprint 221, Sprint 222 Dec 4, 2023
@cedrikv
Copy link
Contributor

cedrikv commented Dec 11, 2023

wat doen we hier dan mee @roefem ?

@cedrikv cedrikv modified the milestones: Sprint 222, Sprint 223 Dec 18, 2023
@claeyswo claeyswo modified the milestones: Sprint 223, Sprint 224 Jan 3, 2024
@roefem
Copy link
Contributor Author

roefem commented Jan 9, 2024

wat doen we hier dan mee @roefem ?

Een issue aanmaken in Plannen om dit uit te proberen?

@cedrikv cedrikv removed this from the Sprint 224 milestone Jan 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants