Skip to content

Conversation

@maniu
Copy link

@maniu maniu commented Dec 1, 2025

Description

In this update, I addressed a significant performance issue in the block editor, particularly noticeable when interacting with a large number of blocks. The "handleMargins" functionality within the "advanced-controls" component was identified as a primary contributor to these performance bottlenecks. Upon thorough investigation, I concluded that this piece of code was not essential, and its removal did not impact the editor's functionality. This change is aimed at enhancing the user experience by reducing lags and improving the overall responsiveness of the editor.

Types of changes

This change is primarily a Bug fix. By removing an unnecessary piece of code that was causing performance issues, I have enhanced the efficiency of the block editor without (hopefully) affecting its existing capabilities.

How has this been tested?

  • The changes were tested in a local development environment with various scenarios to ensure that removing the "handleMargins" functionality did not affect the editor's behavior.
  • Performance metrics were compared before and after the change, highlighting a noticeable improvement in the editor's responsiveness, especially with a high number of blocks.

Acceptance criteria

  • The editor should maintain all current functionalities without "handleMargins" code.
  • Performance improvements should be observed, specifically in scenarios with a large number of blocks present in the editor.
  • No adverse effects on the layout and visual appearance due to the removal of "handleMargins".

Checklist:

  • My code is tested
  • My code follows accessibility standards
  • My code has proper inline documentation
  • I've included any necessary tests
  • I've included developer documentation
  • I've added proper labels to this pull request

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