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

[Element] Show notice message before closing a changed element #3997

Closed
smoench opened this issue Feb 27, 2019 · 5 comments
Closed

[Element] Show notice message before closing a changed element #3997

smoench opened this issue Feb 27, 2019 · 5 comments
Assignees
Milestone

Comments

@smoench
Copy link
Contributor

smoench commented Feb 27, 2019

Bug Report / Feature Request

Expected behavior

Given an user is logged-in in the admin panel
When they are starts editing an object
And changing a value
When they accidentally closing the tab
Then an dialog appears which describes the repercussions
And the user can choose between "Closing" and "Abort"

Actual behavior

Given an user is logged-in in the admin panel
When they are starts editing an object
And changing a value
When they accidentally closing the tab
Then changes are lost

@fashxp
Copy link
Member

fashxp commented Feb 27, 2019

should be already there
image

seems not to work anymore though, can you confirm?

@smoench
Copy link
Contributor Author

smoench commented Feb 28, 2019

I did not know about those options and I would not have excepted those options are enabled by default. Might be fair enough for admin users but opt-out those options on every new user is created is not very (user/admin) friendly. Having a "Don't show this message again" in those dialogs or having those options in group settings would be another solution.

When I opt-out "Disable unsaved content warning" and "Show close warning" everything works as expected. Thanks :-)

@brusch
Copy link
Member

brusch commented Apr 15, 2019

Show close warning is used for the Pimcore admin ui in general, not for individually opened entities, see

if (pimcore.settings.showCloseConfirmation && tabPanel.items.getCount() > 0 && user["closeWarning"]) {

@brusch brusch changed the title Show notice message before closing a changed object [Element] Show notice message before closing a changed object Apr 15, 2019
@brusch brusch changed the title [Element] Show notice message before closing a changed object [Element] Show notice message before closing a changed element Apr 15, 2019
@smoench
Copy link
Contributor Author

smoench commented Apr 15, 2019

Yes, Show close warning is used for the Pimcore admin ui in general. The option Disable unsaved content warning is for a notice message before closing a changed element.

IMHO Disable unsaved content warning should be opt-out by default. This would be a huge improvement for the user experience.

@brusch
Copy link
Member

brusch commented Apr 15, 2019

@smoench agree!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants