Skip to content

Rebase "feat: replace x-editable with HTMX for inline editing" + other minors#2909

Open
ElLorans wants to merge 6 commits into
masterfrom
feat/replace-xeditable-with-htmx-oren
Open

Rebase "feat: replace x-editable with HTMX for inline editing" + other minors#2909
ElLorans wants to merge 6 commits into
masterfrom
feat/replace-xeditable-with-htmx-oren

Conversation

@ElLorans
Copy link
Copy Markdown
Contributor

Rebasing #2847 was not straightforward for me, so I am trying here before working on the original branch

@ElLorans ElLorans force-pushed the feat/replace-xeditable-with-htmx-oren branch from 1353ba6 to 915c367 Compare May 24, 2026 16:54
@ElLorans ElLorans changed the title Rebase https://github.com/pallets-eco/flask-admin/pull/2847 Rebase "feat: replace x-editable with HTMX for inline editing" May 24, 2026
@ElLorans
Copy link
Copy Markdown
Contributor Author

ElLorans commented May 24, 2026

Test pass, example works. I would move this rebase and updates to #2909, then merge to master. We can work from there to unlock either #2892 (which looks done to me except for x editable not working) or #2444.
Finally, we can debate wether dropping XEditableWidget should happen in a minor or major release.

EDIT: not ready yet.
Errors are not rendered. E.g.: if the user submits a non unique field on a unique constrained column, only a console error is registered.
image

@ElLorans ElLorans changed the title Rebase "feat: replace x-editable with HTMX for inline editing" Rebase "feat: replace x-editable with HTMX for inline editing" + other minors May 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant