Skip to content

Conversation

@emoral435
Copy link
Contributor

Reverts #43604

We still need to get nextcloud-libraries/nextcloud-vue#5216 merged first before we get the previous PR merged 🥹

Without having the newest NcVue library version, it will introduce a regression as such:

firefox_0G03fcmqBJ

Once the new library gets merged, will add the changes back!

@emoral435 emoral435 self-assigned this Feb 16, 2024
@emoral435 emoral435 requested a review from skjnldsv as a code owner February 16, 2024 18:08
@emoral435 emoral435 requested review from Pytal and susnux February 16, 2024 18:08
@skjnldsv
Copy link
Member

We can also just fast forward to a new NC/Vue release. This is 29 as alpha state, that happens :)

@emoral435
Copy link
Contributor Author

Ah, okay! Truthfully, I was a bit nervous about making the breaking visual change on the developer instance, but the buttons still work, and having to submit the same PR again would be a lot. Closing PR unless otherwise told to open again.

@emoral435 emoral435 closed this Feb 19, 2024
@emoral435 emoral435 deleted the revert-43604-enh/breadcrumbs/43569/apply-breadcrumbs-logic-from-personal-files-to-all-files-view branch February 19, 2024 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants