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

Unlock Cover when editor leaves Compose/Layout tabs #917

Merged
merged 1 commit into from
Mar 18, 2022
Merged

Conversation

wesleybl
Copy link
Member

@wesleybl wesleybl commented Mar 4, 2022

When an Editor enters the Compose/Layout tabs, the Cover is locked. But when the Editor exit the Compose/Layout tabs the cover was not being unlocked for the other Editors. Now the cover is unlocked.

In the Compose/Layout tabs, a request is sent to Plone to unlock the cover, in the javascript unload event, which happens when the user leaves the page.

The plone.UnlockHandler.execute function, which was called previously, no longer exists in Plone 5.

Fixes #916

When an Editor enters the Compose/Layout tabs, the Cover is locked. But
when the Editor exit the Compose/Layout tabs the cover was not being
unlocked for the other Editors. Now the cover is unlocked.

In the Compose/Layout tabs, a request is sent to Plone to unlock the
cover, in the javascript unload event, which happens when the user
leaves the page.

The `plone.UnlockHandler.execute` function, which was called previously,
no longer exists in Plone 5.
Copy link
Member

@hvelarde hvelarde left a comment

Choose a reason for hiding this comment

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

LGTM

@hvelarde hvelarde merged commit edd0fc3 into master Mar 18, 2022
@hvelarde hvelarde deleted the fix_unlock branch March 18, 2022 22:53
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.

Cover is not unlocked when Editor exits Compose/Layout tabs
2 participants