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

[stable7] Add info NoteCard #4228

Closed
wants to merge 1 commit into from

Conversation

backportbot-nextcloud[bot]
Copy link

Backport of #4063

Signed-off-by: Julian-Samuel Gebühr <julian-samuel@gebuehr.net>
Co-Authored-By: Simon L. <szaimen@e.mail.de>
Co-Authored-By: Raimund Schlüßler <raimund.schluessler+github@mailbox.org>
@backportbot-nextcloud backportbot-nextcloud bot added the enhancement New feature or request label Jun 16, 2023
@backportbot-nextcloud backportbot-nextcloud bot added this to the 7.11.7 milestone Jun 16, 2023
@max-nextcloud
Copy link
Contributor

This would probably allow us to make use of the note card in text in order to fix the color of info boxes to not depend on the theme color (nextcloud/text#4161). All versions of text that we'd want to backport the fix to use at least Nextcloud vue 7.

@szaimen
Copy link
Contributor

szaimen commented Jun 16, 2023

@max-nextcloud you know this needs 27? nextcloud/server#38205

@max-nextcloud
Copy link
Contributor

@szaimen ahh... good point. That's a pitty.

How should we deal with that... only have it in 8.0 and then only use 8.0 in apps that only ship for 27+ ?
Well i guess it's just for this component.

@szaimen
Copy link
Contributor

szaimen commented Jun 16, 2023

I think vue 8.0 is planned to release for nextcliud 28 and higher only so if we dont backport this change here we should be safe?

@max-nextcloud
Copy link
Contributor

@szaimen ok. Makes sense.

@raimund-schluessler raimund-schluessler deleted the backport/4063/stable7 branch October 18, 2023 07:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants