Skip to content

Move notices to top of page again#3639

Merged
tvdeyen merged 1 commit intomainfrom
move-notices-to-top
Feb 5, 2026
Merged

Move notices to top of page again#3639
tvdeyen merged 1 commit intomainfrom
move-notices-to-top

Conversation

@tvdeyen
Copy link
Member

@tvdeyen tvdeyen commented Feb 5, 2026

What is this pull request for?

We moved it to the bottom because the publish page button was interfering with the growl messages, but this is too far away. Leaving them in the center but move them to the top again and elevate them so they are in the visible area of sers attention again.

Screenshots

CleanShot 2026-02-05 at 12 00 26

Checklist

@tvdeyen tvdeyen added this to the 8.1 milestone Feb 5, 2026
@tvdeyen tvdeyen requested a review from a team as a code owner February 5, 2026 11:01
@tvdeyen tvdeyen added the backport-to-8.1-stable Needs to be backported to 8.1-stable label Feb 5, 2026
@codecov
Copy link

codecov bot commented Feb 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.48%. Comparing base (8db8e3c) to head (99684b6).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3639   +/-   ##
=======================================
  Coverage   97.48%   97.48%           
=======================================
  Files         314      314           
  Lines        8272     8272           
=======================================
  Hits         8064     8064           
  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.

We moved it to the bottom because the publish page button
was interfering with the growl messages, but this is too
far away. Leaving them in the center but move them to the
top again and elevate them so they are in the visible
area of sers attention again.
@tvdeyen tvdeyen force-pushed the move-notices-to-top branch from 8deb66e to 99684b6 Compare February 5, 2026 14:38
@tvdeyen tvdeyen enabled auto-merge February 5, 2026 14:42
@tvdeyen tvdeyen merged commit b8903e8 into main Feb 5, 2026
35 of 36 checks passed
@tvdeyen tvdeyen deleted the move-notices-to-top branch February 5, 2026 14:53
@alchemycms-bot
Copy link

alchemycms-bot bot commented Feb 5, 2026

💔 All backports failed

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

Manual backport

To create the backport manually run:

backport --pr 3639

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.1-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.1-stable Needs to be backported to 8.1-stable

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants