search

nextcloud-libraries / nextcloud-vue

🍱 Vue.js components for Nextcloud app development ✌ https://npmjs.org/@nextcloud/vue
https://nextcloud-vue-components.netlify.app/
Other
217 stars 89 forks source link

[NcAppContent] list is rebuild every time when details were shown with no-split or mobile view #6204

Open wofferl opened 2 weeks ago

wofferl commented 2 weeks ago

https://github.com/nextcloud-libraries/nextcloud-vue/blob/b7eeee8966b33e8f963b30d665b48ada2c2c6d48/src/components/NcAppContent/NcAppContent.vue#L63-L73

Is this change in 8.10.0 (6f74155) intentional, to remove the list from dom every time the details are shown?

susnux commented 2 weeks ago

cc @GretaD

GretaD commented 2 weeks ago

Hello, this is "intended" by implementation. But we understand concerns raised in this ticket. I'm unsure how to proceed. I'll loop @ShGKme so we can discuss this soon if we can approach this differently.

wofferl commented 1 week ago

Wouldn't v-show for the list be the right approach here or are there technical reasons for removing the list?

                                <slot v-show="!showDetails" name="list" />

                                <slot v-if="showDetails" />