Issue #5613Opened January 12, 2024by SachinPuthiyoth0 reactions

BUG: Incorrect type definition for StyleManagerConfig.sectors.properties

Question

GrapesJS version

  • I confirm to use the latest version of GrapesJS

What browser are you using?

Chrome

Reproducible demo link

None

Describe the bug

How to reproduce the bug?

  1. As per the definition for sectors in https://github.com/GrapesJS/grapesjs/blob/5c890449a52f526bc480d6dc649ff5e01ec59140/src/style_manager/index.ts#L310 properties are of type PropertyTypes
  2. The type definition in https://github.com/GrapesJS/grapesjs/blob/5c890449a52f526bc480d6dc649ff5e01ec59140/src/style_manager/config/config.ts#L8 is of type PropertyProps and hence is missing properties like options

What is the expected behavior? The type declaration in StyleManagerConfig.sectors.properties has to be of type PropertyTypes to accommodate options.

What is the current behavior? The type declaration in StyleManagerConfig.sectors.properties is PropertyProps and compiler complains since the property options is missing.

Code of Conduct

  • I agree to follow this project's Code of Conduct

Answers (0)

No answers yet.

Related Questions and Answers

Continue research with similar issue discussions.

Paid Plugins That Match This Issue

Curated by issue keywords and label relevance to help you ship faster.

View all plugins

Loading paid plugin recommendations...

Browse Plugin Categories

Jump directly to plugin category pages on the marketplace.