Skip to content

IBX-11436: Make logic for anchor-menu-items css classes reusable#1868

Open
vidarl wants to merge 3 commits into
4.6from
IBX-11436_make_main_container_class_in_edit_base_html_twig_reusable
Open

IBX-11436: Make logic for anchor-menu-items css classes reusable#1868
vidarl wants to merge 3 commits into
4.6from
IBX-11436_make_main_container_class_in_edit_base_html_twig_reusable

Conversation

@vidarl

@vidarl vidarl commented Mar 30, 2026

Copy link
Copy Markdown
Contributor
🎫 Issue IBX-11436

Related PRs:

Description:

We need to calculate the main container classes in pagebuilder as well. As twig doesn't support importing only one specific block from a given template, I opted for defining the logic in a macro

An alternative could be to move the block to a dedicated template

For QA:

Documentation:

@vidarl vidarl changed the title IBX-11436: Make main_container_class in edit_base.html.twig reusable IBX-11436: Make logic in main_container_class in edit_base.html.twig reusable Mar 30, 2026
@vidarl vidarl force-pushed the IBX-11436_make_main_container_class_in_edit_base_html_twig_reusable branch from d2e09bf to d7b4484 Compare March 30, 2026 12:22
@vidarl vidarl requested a review from a team March 30, 2026 12:51
@konradoboza konradoboza requested a review from a team March 30, 2026 12:55
@ibexa-workflow-automation-1 ibexa-workflow-automation-1 Bot requested review from GrabowskiM, OstafinL, albozek, alekmick, dew326 and tischsoic and removed request for a team March 30, 2026 12:55
@vidarl vidarl force-pushed the IBX-11436_make_main_container_class_in_edit_base_html_twig_reusable branch from d7b4484 to 270df7f Compare April 10, 2026 06:16
@vidarl vidarl changed the title IBX-11436: Make logic in main_container_class in edit_base.html.twig reusable IBX-11436: Make logic for anchor-menu-items css classes reusable Apr 10, 2026
@sonarqubecloud

Copy link
Copy Markdown

@vidarl vidarl force-pushed the IBX-11436_make_main_container_class_in_edit_base_html_twig_reusable branch from 2e0c9d3 to 85dba34 Compare May 13, 2026 11:32
@tomaszszopinski tomaszszopinski force-pushed the IBX-11436_make_main_container_class_in_edit_base_html_twig_reusable branch from 85dba34 to 70a56e3 Compare June 9, 2026 06:17
@sonarqubecloud

sonarqubecloud Bot commented Jun 9, 2026

Copy link
Copy Markdown

@tomaszszopinski tomaszszopinski left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

QA approved on Ibexa DXP 4.6 exp (CI is being fixed).

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants