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

Bump @nextcloud/vue to v7 #1206

Merged
merged 2 commits into from
Sep 6, 2022
Merged

Bump @nextcloud/vue to v7 #1206

merged 2 commits into from
Sep 6, 2022

Conversation

artonge
Copy link
Collaborator

@artonge artonge commented Sep 5, 2022

No description provided.

Signed-off-by: Louis Chemineau <louis@chmn.me>
@artonge artonge added the 2. developing Work in progress label Sep 5, 2022
@artonge artonge added this to the Nextcloud 25 milestone Sep 5, 2022
@artonge artonge self-assigned this Sep 5, 2022
@artonge artonge marked this pull request as draft September 5, 2022 14:18
@artonge artonge marked this pull request as ready for review September 5, 2022 16:13
Signed-off-by: Louis Chemineau <louis@chmn.me>
@skjnldsv skjnldsv force-pushed the bump/nextcloud_vue_7 branch from 4e6785e to 84f83f6 Compare September 6, 2022 08:32
Copy link
Member

@skjnldsv skjnldsv left a comment

Choose a reason for hiding this comment

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

Pushed a fix.
I removed the usage of NcTextField because it have a bug on the focus event it seems (focusing the input triggers the focus event more than one time for god know why. So let's keep it simple for now)

Albums have an horizontal scrollbar and a weird rendering data issue
image

We can fix it in a followup if you want

@artonge artonge merged commit ce56825 into master Sep 6, 2022
@artonge artonge deleted the bump/nextcloud_vue_7 branch September 6, 2022 08:38
<template>
<div v-if="isReady" class="emptycontent">
<!-- eslint-disable-next-line vue/no-v-html (because it's an SVG file) -->
<div v-if="haveIllustration" class="illustration" v-html="illustration" />
Copy link
Member

Choose a reason for hiding this comment

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

@artonge regression, we were not using the vue component.
So you, migrating to it, broke support for illustrations as well as default empty contents :)

<NcEmptyContent v-if="error">
{{ t('photos', 'An error occurred') }}
</NcEmptyContent>

<NcEmptyContent v-if="!loading && !hasTagsWithFiles" key="Ncemptycontent" illustration-name="empty">
{{ t('photos', 'No tags yet') }}
<template #desc>
{{ t('photos', 'Photos with tags will show up here') }}
</template>
</NcEmptyContent>

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Seen, and on it ;) trying to abstract those views to have more consistency between them


<!-- Folder content -->
<div v-else-if="!loading">
<div class="photos-navigation">
<Actions class="photos-navigation__back">
<NcActions class="photos-navigation__back">
<ActionButton @click="$router.push({name: 'tags'})">
Copy link
Member

Choose a reason for hiding this comment

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

Regression. I'm on it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2. developing Work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants