-
Notifications
You must be signed in to change notification settings - Fork 28
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
Changes to SEO tab trigger content change? #219
Comments
Hi @ianleeder, |
@patrickdemooij9 is it possible to actually do the SEO save by hooking into the Content Saved event instead of handling it separately I wonder? There's also an issue here where if you it Save and Publish, it does the two saves but will refresh the content back to the Node you were working on. If I've already moved onto a different node, then there's a race-condition inconsistency with the content and content tree. I'll try to put together a screen recording to show you what I mean. |
@robertjf The reason why it is separate now is because it might be confusing to the user if they click "Save and publish" and it also saves the normal content as you are only editing the SEO data. But maybe that idea just lives in my head and combining them would be better, especially if we have race conditions going on. |
If you only make a change on the SEO tab and save the changes, you get the expected green bar:
If you attempt to navigate away from the content node, without having made any changes to the content, you get warned about unsaved changes:
Can changes to the SEO tab somehow prevent marking the content node as dirty requiring save?
The text was updated successfully, but these errors were encountered: