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

[4.x]: Element index settings are lost when clicking outside the modal #12895

Closed
MoritzLost opened this issue Mar 15, 2023 · 3 comments
Closed
Assignees

Comments

@MoritzLost
Copy link
Contributor

What happened?

Description

When editing the element index settings, clicking outside the modal closes it. This means you lose all pending changes you've been working on. One mis-click means you have to start over. See the video for demonstration:

element-index-modal.mov

Steps to reproduce

  1. Go to the entries index and click on Customize sources.
  2. Make some changes in the modal.
  3. Click outside the modal to close it.
  4. When opening the modal again, any unsaved changes are lost.

Expected behavior

It's unfortunate that an accidental click outside the modal causes you to lose any changes you've done. I can think of two possible improvement:

  1. Don't allow the modal to be closed by clicking outside it.
  2. The modal could retain the pending state on the current page, so that clicking Customize sources again will return it to the last state.

Craft CMS version

4.4.2

PHP version

8.2

Operating system and version

No response

Database type and version

No response

Image driver and version

No response

Installed plugins and versions

No response

@brandonkelly
Copy link
Member

Agree, that modal is a little too easy to close. Just tweaked it for the next Craft 3 and 4 releases, so Esc key presses and clicks outside the modal no longer cause it to be hidden/removed.

@MoritzLost
Copy link
Contributor Author

Thanks @brandonkelly !

@brandonkelly
Copy link
Member

3.8.3 and 4.4.3 are out with that change!

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

2 participants