Skip to content

Prepare for Contao 5.7#97

Merged
Toflar merged 14 commits into
mainfrom
feature/contao-57
Mar 19, 2026
Merged

Prepare for Contao 5.7#97
Toflar merged 14 commits into
mainfrom
feature/contao-57

Conversation

@qzminski

Copy link
Copy Markdown
Member

No description provided.

@qzminski qzminski requested a review from Toflar February 17, 2026 11:42
@qzminski qzminski self-assigned this Feb 17, 2026
@benfolds

benfolds commented Mar 10, 2026

Copy link
Copy Markdown

Thank you for the pull request, i cloned it into my installation with 5.7.

I have some issues while testing:

  • Insert Tags will not be replaced
  • The last pagebreak button leads to an empty page

I use some more form-regarding bundles, maybe this is relevant?

        "terminal42/contao-leads"
        "terminal42/contao-conditionalformfields"
        "terminal42/notification_center"
        "terminal42/contao-mp_forms"

@qzminski

Copy link
Copy Markdown
Member Author

@benfolds thank you for the feedback.

  1. Are you using insert tags inside the form (e.g. explanation form field), or outside it (e.g. content elements)?
  2. Can you post a screenshot of your form fields configuration in the backend? I mean the list view of the form fields.

@benfolds

benfolds commented Mar 10, 2026

Copy link
Copy Markdown

I use the Insert Tags inside the Form within a HTML-Code element
The beginning:
grafik

in between is 65 fields with various formfields.
and the End:
grafik

I was wrong regarding the last page. It's the summary element that does not work i guess.....

@fritzmg

fritzmg commented Mar 10, 2026

Copy link
Copy Markdown
Contributor

I use the Insert Tags inside the Form within a HTML-Code element

This is an issue within Contao itself. Currently Insert-Tags will not be replaced in the HTML form element.

@benfolds

benfolds commented Mar 10, 2026

Copy link
Copy Markdown

Thank you @fritzmg .
there is one more issue with the summary:
grafik

Maybe it comes from the same souce as the error above...

@qzminski

Copy link
Copy Markdown
Member Author

@benfolds I have fixed the summary in 0816f63 👍

@qzminski

Copy link
Copy Markdown
Member Author

@Toflar if you don't hany suggestions then this should be ready to merge.

@Toflar Toflar merged commit 6680f77 into main Mar 19, 2026
7 checks passed
@Toflar Toflar deleted the feature/contao-57 branch March 19, 2026 20:51
@Toflar

Toflar commented Mar 19, 2026

Copy link
Copy Markdown
Member

Thank you, @qzminski!

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.

4 participants