grapesjs icon indicating copy to clipboard operation
grapesjs copied to clipboard

[Feature request] Add category on traits

Open WebEtSolutions opened this issue 6 years ago • 6 comments

Hi and thank's for your job !

It's possible to make traits under cateogories like blocks ? If it's possible, maybe use the Category object like the BlockManager ?

Categories on BlockManager work fine, so if TraitManager add the same system of category, it's perfect !

What it's the best method for you for implements this feature ? We are ready to develop this feature and make PR.

Thank's

WebEtSolutions avatar Feb 18 '19 10:02 WebEtSolutions

Hi @WebEtSolutions follow my indications posted on your PR, so if we need to implement the same logic in Traits we have to abstract it from Blocks

artf avatar Feb 22 '19 13:02 artf

Hi, no problem we will work for this PR.

WebEtSolutions avatar Feb 25 '19 08:02 WebEtSolutions

Hi,

You can check PR on https://github.com/WebEtSolutions/grapesjs/tree/%231806

We have a question for the PR.

CategoryView and Category are moved on categories folder and BlocksView and TraitsView use the new CategoryView.

Classes for the render of categories change from block-c(category)(no-cat) to items-c(category)(no-cat)

So there are the same code on TraitsView and BlocksView.

What's for you the best way for reformat BlocksView and TraitsView ?

Thank's for your work.

Web&Solutions Team

WebEtSolutions avatar Mar 06 '19 12:03 WebEtSolutions

Do we have this feature yet?

Uysim avatar Jan 12 '20 10:01 Uysim

Hi there! Is this feature implemented? Would be pretty useful, I have some components with a couple dozen traits. Thanks!

protozoo avatar Jan 27 '22 12:01 protozoo

That would bery very intersting indeed. I am happy to help if necessary :)

michaelhofer avatar Mar 03 '22 06:03 michaelhofer

+1

bgrand-ch avatar Nov 03 '23 08:11 bgrand-ch

+1

vasicvuk avatar Dec 28 '23 17:12 vasicvuk

+1

amirrahmani76 avatar Dec 31 '23 16:12 amirrahmani76

Will be available in the next release, Thanks @amirrahmani76 for the PR ❤️

artf avatar Jan 19 '24 16:01 artf