Skip to content

[wpmlpb-824] Divi: translate contact-field conditional-logic and select-options#536

Merged
dgwatkins merged 1 commit into
masterfrom
agent/wpmlpb-824
Jul 7, 2026
Merged

[wpmlpb-824] Divi: translate contact-field conditional-logic and select-options#536
dgwatkins merged 1 commit into
masterfrom
agent/wpmlpb-824

Conversation

@dgwatkins

Copy link
Copy Markdown
Collaborator

Contact form fields with conditional logic ("show if Field = Value") stopped working on translated pages: the option labels were translated but the rule's compared value was not, so Divi's strict string match ("is") no longer succeeded and the dependent field never showed.

Register the conditionalLogic rule values so both the option and the rule translate to the same string (they share one WPML string, so they cannot diverge) and the match holds after translation. Also add the previously missing selectOptions, so select-field options translate too -- otherwise translating a select-based rule value would newly break the match.

https://onthegosystems.myjetbrains.com/youtrack/issue/wpmlpb-824/

…+ select options)

Contact form fields with conditional logic ("show if Field = Value") stopped
working on translated pages: the option labels were translated but the rule's
compared value was not, so Divi's strict string match ("is") no longer
succeeded and the dependent field never showed.

Register the conditionalLogic rule values so both the option and the rule
translate to the same string (they share one WPML string, so they cannot
diverge) and the match holds after translation. Also add the previously
missing selectOptions, so select-field options translate too -- otherwise
translating a select-based rule value would newly break the match.

https://onthegosystems.myjetbrains.com/youtrack/issue/wpmlpb-824/

Co-authored-by: Chai <david.gw@onthegosystems.com>
@dgwatkins dgwatkins requested a review from strategio July 6, 2026 17:58

@strategio strategio left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dgwatkins dgwatkins merged commit 1135639 into master Jul 7, 2026
1 check passed
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.

2 participants