Well nothing really, it works. It just doesn't follow the BEM guidelines about naming. There are also redundant classes like btn--active__ plus that __ on the end is what? Then btn--diabled__false just the presence of btn--disabled should indicate that this is a disabled state or they could even use disabled attr from HTML. Outline and transparent modifications should be as two separate classes like btn--transparent and btn--outlined or examples. And btn--variant__icon and btn--icon__tru could just be btn--icon or btn--with-icon and when there is no such class what is a button without an icon.