[next] refactor: consistent export style for isSlotPopulated

src/components/NcActions/NcActions.vue

@@ -921,14 +921,13 @@
 import { useElementBounding, useWindowSize } from '@vueuse/core'
 import { Fragment, computed, h, mergeProps, ref, toRef, warn } from 'vue'
 import { useTrapStackControl } from '../../composables/useTrapStackControl.ts'
+import { createElementId } from '../../utils/createElementId.ts'
+import { isSlotPopulated } from '../../utils/isSlotPopulated.ts'
 import { t } from '../../l10n.js'
 
+import IconDotsHorizontal from 'vue-material-design-icons/DotsHorizontal.vue'
 import NcButton from '../NcButton/index.ts'
 import NcPopover from '../NcPopover/index.js'
-import { createElementId } from '../../utils/createElementId.ts'
-import isSlotPopulated from '../../utils/isSlotPopulated.ts'
-
-import IconDotsHorizontal from 'vue-material-design-icons/DotsHorizontal.vue'
 
 const focusableSelector = '.focusable'
 
@@ -1396,7 +1395,7 @@
 		},
 
 		/**
 		 * @return {NodeListOf<HTMLElement>}
 		 */
 		getFocusableMenuItemElements() {
 			return this.$refs.menu.querySelectorAll(focusableSelector)

src/components/NcAppSidebar/NcAppSidebar.vue

@@ -714,10 +714,10 @@
 import { vOnClickOutside as ClickOutside } from '@vueuse/components'
 import { createFocusTrap } from 'focus-trap'
 import { useIsSmallMobile } from '../../composables/useIsMobile/index.js'
-import { getTrapStack } from '../../utils/focusTrap.ts'
 import { createElementId } from '../../utils/createElementId.ts'
+import { getTrapStack } from '../../utils/focusTrap.ts'
+import { isSlotPopulated } from '../../utils/isSlotPopulated.ts'
 import { t } from '../../l10n.js'
-import isSlotPopulated from '../../utils/isSlotPopulated.ts'
 
 import IconArrowRight from 'vue-material-design-icons/ArrowRight.vue'
 import IconClose from 'vue-material-design-icons/Close.vue'
@@ -1099,7 +1099,7 @@
 			 * @type {Event}
 			 */
 			// eslint-disable-next-line vue/require-explicit-emits
 			this.$emit('figure-click', e)
 		},
 
 		/**
@@ -1190,7 +1190,7 @@
 			 *
 			 * @type {Event}
 			 */
 			this.$emit('submit-name', event)
 		},
 		onDismissEditing() {
 			// Disable editing
@@ -1200,7 +1200,7 @@
 			 *
 			 * @type {Event}
 			 */
 			this.$emit('dismiss-editing')
 		},
 		onUpdateActive(activeTab) {
 			/**

src/components/NcBreadcrumbs/NcBreadcrumbs.vue

@@ -120,19 +120,18 @@ export default {
 </docs>
 
 <script>
+import { subscribe, unsubscribe } from '@nextcloud/event-bus'
+import { cloneVNode, h, Fragment } from 'vue'
+import debounce from 'debounce'
+
+import { isSlotPopulated } from '../../utils/isSlotPopulated.ts'
+
+import IconFolder from 'vue-material-design-icons/Folder.vue'
 import NcActions from '../NcActions/index.js'
 import NcActionButton from '../NcActionButton/index.js'
 import NcActionRouter from '../NcActionRouter/index.js'
 import NcActionLink from '../NcActionLink/index.js'
 import NcBreadcrumb from '../NcBreadcrumb/index.js'
-import isSlotPopulated from '../../utils/isSlotPopulated.ts'
-
-import { subscribe, unsubscribe } from '@nextcloud/event-bus'
-
-import IconFolder from 'vue-material-design-icons/Folder.vue'
-
-import debounce from 'debounce'
-import { cloneVNode, h, Fragment } from 'vue'
 
 const crumbClass = 'vue-crumb'
 

src/components/NcButton/NcButton.vue

@@ -422,8 +422,8 @@ td.row-size {
 import type { PropType } from 'vue'
 
 import { defineComponent, h, resolveComponent } from 'vue'
-import { ButtonAlignment, ButtonNativeType, ButtonType } from './types'
-import isSlotPopulated from '../../utils/isSlotPopulated'
+import { ButtonAlignment, ButtonNativeType, ButtonType } from './types.ts'
+import { isSlotPopulated } from '../../utils/isSlotPopulated.ts'
 
 export default defineComponent({
 	name: 'NcButton',

src/utils/isSlotPopulated.ts

@@ -11,7 +11,7 @@ import { Fragment, Comment, Text } from 'vue'
  *
  * @param vnodes The array of vnodes to check
  */
-const isSlotPopulated = function(vnodes?: VNode[] | VNodeNormalizedChildren) {
+export function isSlotPopulated(vnodes?: VNode[] | VNodeNormalizedChildren) {
 	return Array.isArray(vnodes) && vnodes.some(node => {
 		if (node === null) return false
 		if (typeof node === 'object') {
@@ -23,5 +23,3 @@ const isSlotPopulated = function(vnodes?: VNode[] | VNodeNormalizedChildren) {
 		return true
 	})
 }
-
-export default isSlotPopulated

tests/unit/utils/isSlotPopulated.spec.ts

@@ -7,7 +7,7 @@ import { mount } from '@vue/test-utils'
 import { describe, expect, it } from 'vitest'
 import { defineComponent } from 'vue'
 
-import isSlotPopulated from '../../../src/utils/isSlotPopulated'
+import { isSlotPopulated } from '../../../src/utils/isSlotPopulated.ts'
 
 const IsSlotPopulatedTest = defineComponent({
 	name: 'IsSlotPopulatedTest',