godaddy-wordpress / godaddy-wordpress/coblocks

ISBAT easily debug the Row block and Typography controls

Open
#1,713 0 comments 0 reactions 0 assignees View on GitHub
[Priority] Low [Type] Enhancement
Dominant language
JavaScript
Stars
794
Forks
142
PR merge metrics
No merged PRs in 30d

Description

### Is your feature request related to a problem? Please describe.

Typography controls and Row block have an integrated API stemming from shared use of a utility function called `applyStyles`. This function is responsible for setting column width within the editor while using the Row block. Because of the nature of the current architecture, it makes understanding the source of bugs both difficult to understand and difficult to find. For the sake of future expectations of maintenance, I propose that a new control be set up for the Row block which specifically handles the setting of column width.

A recent fix for both the Row block and the Typography controls can be found here: https://github.com/godaddy-wordpress/coblocks/pull/1711

### Describe the solution you'd like

Suggest moving the logic to apply wrapper styles for the Column block within the Row block directory in CoBlocks. Both for organizational reasons as well as readability and understandability.

### Describe alternatives you've considered

Keep the way it is with the expectation of eventual deprecation for core features.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.