Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[7.4-stable] Resizable elements window #3187

Merged
merged 2 commits into from
Feb 7, 2025

Conversation

tvdeyen
Copy link
Member

@tvdeyen tvdeyen commented Feb 7, 2025

Backport

This will backport the following commits from main to 7.4-stable:

Questions ?

Please refer to the Backport tool documentation

Allows to drag the divider between the preview and
elements windows in order to resize the elements window.

This is very helpful, if the elements window is too narrow when
lots of nested elements exist.

(cherry picked from commit 5e510e5)

# Conflicts:
#	app/assets/builds/alchemy/admin.css
#	app/assets/builds/alchemy/admin.css.map
#	app/assets/builds/alchemy/custom-properties.css.map
#	app/assets/builds/alchemy/welcome.css.map
#	app/assets/builds/tinymce/skins/content/alchemy/content.min.css.map
#	app/assets/stylesheets/alchemy/admin/elements.scss
#	app/assets/stylesheets/alchemy/admin/navigation.scss
#	app/stylesheets/alchemy/admin/preview_window.scss
Makes sure the elements window width is the same across pages.
Uses a cookie, so we can read the value from Rails and put it on
the body tag, so it is immediatly available on page render.
Otherwise we would see the element window resizing on every
page visit. SSR ftw \o/

(cherry picked from commit 44d3a39)
@tvdeyen tvdeyen requested a review from a team as a code owner February 7, 2025 07:22
@tvdeyen tvdeyen mentioned this pull request Feb 7, 2025
3 tasks
Copy link

codecov bot commented Feb 7, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.46%. Comparing base (d5283f9) to head (87fb348).
Report is 3 commits behind head on 7.4-stable.

Additional details and impacted files
@@             Coverage Diff             @@
##           7.4-stable    #3187   +/-   ##
===========================================
  Coverage       96.46%   96.46%           
===========================================
  Files             238      238           
  Lines            6454     6454           
===========================================
  Hits             6226     6226           
  Misses            228      228           

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

@tvdeyen tvdeyen merged commit 4e7ff81 into 7.4-stable Feb 7, 2025
53 checks passed
@tvdeyen tvdeyen deleted the backport/7.4-stable/pr-3097 branch February 7, 2025 07:39
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.

1 participant