Skip to content

Conversation

@tvdeyen
Copy link
Member

@tvdeyen tvdeyen commented Feb 2, 2026

What is this pull request for?

This fixes a bug where the created fixed element does not appear as tab.

In order to fix this we extract the fixed element tab into a partial and render this
in order to make sure the tab to exist before appending the new element.

Checklist

  • I have followed Pull Request guidelines
  • I have added a detailed description into each commit message
  • I have added tests to cover this change

@tvdeyen tvdeyen requested a review from a team as a code owner February 2, 2026 07:33
@tvdeyen tvdeyen added bug backport-to-8.0-stable Needs to be backported to 8.0-stable labels Feb 2, 2026
@codecov
Copy link

codecov bot commented Feb 2, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.48%. Comparing base (1bc8b2e) to head (072578d).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3612   +/-   ##
=======================================
  Coverage   97.48%   97.48%           
=======================================
  Files         314      314           
  Lines        8256     8256           
=======================================
  Hits         8048     8048           
  Misses        208      208           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@tvdeyen tvdeyen force-pushed the fix-create-fixed-element branch from f2730fb to 0aaa9ab Compare February 2, 2026 08:57
@tvdeyen tvdeyen enabled auto-merge February 2, 2026 08:57
Makes it possible to render the collection at once
and to render a fixed element on demand.
We need the tab to exist before appending the element.
@tvdeyen tvdeyen force-pushed the fix-create-fixed-element branch from 0aaa9ab to 072578d Compare February 2, 2026 08:58
@tvdeyen tvdeyen merged commit 06e0f92 into main Feb 2, 2026
20 of 21 checks passed
@tvdeyen tvdeyen deleted the fix-create-fixed-element branch February 2, 2026 09:09
@alchemycms-bot
Copy link

alchemycms-bot bot commented Feb 2, 2026

💔 All backports failed

Status Branch Result
8.0-stable Backport failed because of merge conflicts

You might need to backport the following PRs to 8.0-stable:
- Make create element dialog wider (#3445)

Manual backport

To create the backport manually run:

backport --pr 3612

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

@alchemycms-ci-bot
Copy link
Collaborator

💚 All backports created successfully

Status Branch Result
8.0-stable

Questions ?

Please refer to the Backport tool documentation

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

Labels

backport-to-8.0-stable Needs to be backported to 8.0-stable bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants