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

Crud Modal not behaving correctly #2002

Closed
mkrecek234 opened this issue Feb 21, 2023 · 4 comments · Fixed by #2011
Closed

Crud Modal not behaving correctly #2002

mkrecek234 opened this issue Feb 21, 2023 · 4 comments · Fixed by #2011
Assignees
Labels

Comments

@mkrecek234
Copy link
Contributor

mkrecek234 commented Feb 21, 2023

In the most recent FUI releases, I encounter the following issue:

  • You have a crud on a model with many fields
  • The first field say is name, and for a given record it is empty
  • You go to "Edit" for that record in the Crud, and the edit modal opens
  • You enter something in the first field
  • You scroll down, and click "Save"
  • It won't save but jump back the focus to that first field. Scrolling down again and clicking save will then save correctly.

See screen recording:

Bildschirmaufnahme.2023-02-21.um.15.19.34.mov
@mvorisek
Copy link
Member

Yes, this is caused probably by Fomantic-UI autofocus, I have already reported like 5 issues with it. In the next FUI release, I hope they will be fixed.

Please try - click somewhere in the modal before clicking Save. Then click Save. Will the form save?

Also can this be reproduced on some atk4/ui UI demo?

@mkrecek234
Copy link
Contributor Author

Correct, any click anywhere in the modal before clicking save will make it save immediately. Atk4/Ui demo: I could not find one yet, as most modal forms are too small I believe

@mvorisek
Copy link
Member

Too small - you can resize your browser viewport ;-)

Let's wait a few days until FUI is updated, I am fine with merging the latest FUI Beta into atk4/ui then.

@mkrecek234
Copy link
Contributor Author

Can be reproduced on /demos/collection/crud.php by reducing browser size - edit iso2 code, scroll down to reach save button and you will encounter the issue described above.

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

Successfully merging a pull request may close this issue.

2 participants