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

Extract buttons from pages so the animation does not affect them #993

Merged
merged 2 commits into from
Nov 11, 2023

Conversation

marcoambrosini
Copy link
Member

Signed-off-by Marco marcoambrosini@icloud.com

Screen.Recording.2023-11-09.at.09.45.12.mov

@marcoambrosini marcoambrosini self-assigned this Nov 9, 2023
@marcoambrosini
Copy link
Member Author

/compile amend /

@marcoambrosini
Copy link
Member Author

/compile amend /

Copy link
Collaborator

@szaimen szaimen left a comment

Choose a reason for hiding this comment

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

LGTM but didnt test and conflicts

@szaimen szaimen added this to the Nextcloud 28 milestone Nov 9, 2023
src/App.vue Outdated
@@ -62,6 +62,15 @@
<Page2 v-else-if="page === 2" @next="goToNextPage" />
<Page3 v-else-if="page === 3" @close="close" />
Copy link
Collaborator

Choose a reason for hiding this comment

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

Peobably you dont need to catch the emits anymore?

Signed-off-by Marco <marcoambrosini@icloud.com>
@marcoambrosini
Copy link
Member Author

/compile amend /

@szaimen
Copy link
Collaborator

szaimen commented Nov 10, 2023

/compile amend /

@szaimen
Copy link
Collaborator

szaimen commented Nov 10, 2023

/compile

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@szaimen szaimen merged commit 0a6fcc7 into master Nov 11, 2023
18 checks passed
@szaimen szaimen deleted the enh/fix-buttons-transition branch November 11, 2023 10:06
szaimen added a commit that referenced this pull request Nov 15, 2023
…hem #993"

Signed-off-by: Simon L <szaimen@e.mail.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants