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

Moved applyBlockDefaults one component up so the style is computed correctly #6451

Merged
merged 2 commits into from
Oct 29, 2024

Conversation

sneridagh
Copy link
Member

@sneridagh sneridagh commented Oct 29, 2024

The defaults were applied later in the Edit component tree, we need them too in the top of the editWrapper to calculate the styles too.

Copy link

netlify bot commented Oct 29, 2024

Deploy Preview for plone-components canceled.

Name Link
🔨 Latest commit 08010dc
🔍 Latest deploy log https://app.netlify.com/sites/plone-components/deploys/6720faa6d2ceb40008cd47a1

@sneridagh sneridagh merged commit ec4dc83 into main Oct 29, 2024
68 checks passed
@sneridagh sneridagh deleted the moveapplydefaultsineditmodeonelevel branch October 29, 2024 21:20
sneridagh added a commit that referenced this pull request Oct 30, 2024
* main: (111 commits)
  Moved `applyBlockDefaults` one component up so the style is computed correctly (#6451)
  Fix useSelector is returned a different result when called with the s… (#6450)
  Fix page changes not being announced by assistive technology when navigating using the client-side router (#5288)
  BlockStyleWrapper aware of block themes (#6445)
  A11y: Wrap ToC Block within nav tag (#6084)
  Fix backend error when there is no query supplied (#6423)
  Added upgrade guide fix for HMR (#6446)
  Fix CSS lint in Volto Slate (#6444)
  Release 18.0.0-alpha.47
  Release generate-volto 9.0.0-alpha.20
  Release @plone/scripts 3.7.0
  Release @plone/registry 2.0.0-alpha.0
  Improve @plone/registry release-it config
  `@plone/registry` as ESM module, move to TS, complete documentation (#6399)
  [Next.js] Better Vercel deployment (#6441)
  Replace all `yarn` appearences with `pnpm` (#6433)
  Add deprecation notices to the Upgrade guide (#6426)
  Clean up #6422 (#6443)
  Rename page title from Frontend to Volto UI (#6438)
  URL-Management control panel: Add missing translations (#6436)
  ...
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