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

Allow Element and Content deprecation notices #1988

Merged
merged 4 commits into from
Dec 29, 2020

Conversation

tvdeyen
Copy link
Member

@tvdeyen tvdeyen commented Dec 23, 2020

What is this pull request for?

Elements and Contents can be marked as deprecated.

They will be visually displayed with a warning. The deprecation notice can be localized.

This also changes the position of the content hint icon.

Screenshots

Screenshot from 2020-12-28 14-14-04

Checklist

  • I have followed Pull Request guidelines
  • I have added a detailed description into each commit message
  • I have added tests to cover this change

@tvdeyen tvdeyen self-assigned this Dec 23, 2020
@tvdeyen tvdeyen added this to the 5.2 milestone Dec 23, 2020
Elements can be marked as deprecated.

They will be visually displayed with a warning. The deprecation notice can
be localized.
Contents can be marked as deprecated.

They will be visually displayed with a warning.
The deprecation notice can be localized.
This way it does not conflict with warnings
that might be displayed as well and it aligns
nicely with the essence boolean
It now uses the content_editor decorator that has two new
methods to display warnings.
@tvdeyen tvdeyen marked this pull request as ready for review December 28, 2020 13:59
@tvdeyen tvdeyen changed the title (Feat) Element deprecations Allow Element and Content deprecation notices Dec 28, 2020
@tvdeyen tvdeyen removed their assignment Dec 28, 2020
@tvdeyen tvdeyen requested a review from mamhoff December 28, 2020 14:00
@pelargir
Copy link
Contributor

This works quite well. Nice work! 👍

Copy link
Contributor

@mamhoff mamhoff left a comment

Choose a reason for hiding this comment

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

Nice work!

@tvdeyen tvdeyen merged commit 1e57aa6 into AlchemyCMS:main Dec 29, 2020
@tvdeyen tvdeyen deleted the element-deprecations branch December 29, 2020 21:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants