Skip to content

Added state icon to show unpublished changes #18624

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

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

OskarKruger
Copy link
Contributor

@OskarKruger OskarKruger commented Mar 10, 2025

Prerequisites

  • I have added steps to test this contribution in the description below

If there's an existing issue for this PR then this fixes #16324 (comment)

Description

Implement feature/fix to add state icon to show if there are pending/unpublished changes on a node.
Fixes #16324

image
image

@nielslyngsoe nielslyngsoe added the state/needs-investigation This requires input from HQ or community to proceed label Mar 11, 2025
@nielslyngsoe
Copy link
Member

I added the needs investigate tag cause I recall some complexity regarding adding this in. I will talk with @kjac about this one on Thursday.

@nielslyngsoe
Copy link
Member

Note for who ever looks closer into this.

We need to be aware of how this tackles Edits of Variant Blocks.

@bjarnef
Copy link
Contributor

bjarnef commented Apr 9, 2025

For reference discussions about the state/mini icon in tree: #18065

@iOvergaard iOvergaard changed the base branch from v15/dev to main May 5, 2025 13:08
@nielslyngsoe nielslyngsoe requested a review from kjac May 6, 2025 15:59
@nielslyngsoe nielslyngsoe marked this pull request as draft May 9, 2025 08:28
@nielslyngsoe
Copy link
Member

nielslyngsoe commented May 9, 2025

Lets keep this PR in draft until we have resolved how we like to deal with Symbols on tree items in general. as we have a feature in that regards coming up in near future.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affected/v15 area/backend state/needs-investigation This requires input from HQ or community to proceed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Content Tree does not show content items with Updated Saved Content that has not been published
4 participants