GrapesJS / grapesjs

Free and Open source Web Builder Framework. Next generation tool for building templates without coding
https://grapesjs.com
BSD 3-Clause "New" or "Revised" License
22.36k stars 4.05k forks source link

BUG: Style Manager is inaccurate when using combined CSS selectors which contains "bout" and componentFirst = true #4310

Closed barryross closed 2 years ago

barryross commented 2 years ago

GrapesJS version

What browser are you using?

Chrome 100

Reproducible demo link

https://jsfiddle.net/4fmd732c/3/

Describe the bug

How to reproduce the bug?

Given a) componentFirst is set to true b) An element has multiple classes c) CSS rule uses combined selector of multiple classes d) One of the classes contains the word "bout"

Then: Style Manager does not accurately display current styling

Navigate to linked JS Fiddle

  1. Select Div in editor
  2. In the Style Manager, expand "Decorations" and take note of background color

What is the expected behavior?

What is the current behavior? Style manager indicates background is white

If is necessary to execute some code in order to reproduce the bug, paste it here below:

// your code here

Code of Conduct

artf commented 2 years ago

Nice catch, thanks for the report @barryross

barryross commented 2 years ago

@artf Awesome! Thank you. To use these updates, I guess I would just need to re-build the library locally with your changes until next release. I'll give that a shot.

barryross commented 2 years ago

@artf when do you foresee the next release going out?

artf commented 2 years ago

I'm not sure yet but I was planning for the end of the next week.

barryross commented 2 years ago

Gotcha, thanks for letting me know what you're thinking