[BUG] Property slider view default unit
Question
Hey @artf , thanks for the enhancement! The implementation works as expected with some properties (font-size, border-radius, ...), but not with others (line-height, border-width, ...). I created a example/demo where it's visible that the configuration is the same in both scenarios, however the unit doesn't apply in the case of previous mentioned properties.
Hope you can look into it.
Thank you.
Answers (3)
I created a example/demo where it's visible that the configuration is the same in both scenarios
Actually, you didn't indicate the defaults, where in case of line-height takes normal by default and so resets its unit on initialization
@artf I think that this specific fix messed up the properties without units, like Opacity. The slider behaves weirdly in this case, even on the demo.
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Related Questions and Answers
Continue research with similar issue discussions.
Issue #1346
[FEATURE REQUEST] Property slider view default unit
Hey @artf , I think that the slider component would come handy from the UX perspective for the user interaction of some of the integer prop...
Issue #1422
[BUG] Property without units slider
Hey @artf, as already mentioned in this closed ticket => I think that this specific enhancement broke the behaviour of properties without u...
Issue #1214
Query for issue related to 873
Previous issue Please confirm, above reported bug has been resolved. Or still having same issue. Please confirm . Same issue with slider al...
Issue #768
IE browser issue
Hi @artf , Created this issue due to https://github.com/artf/grapesjs/issues/214 has been closed ... I cloned the latest repo and run it on...
Paid Plugins That Match This Issue
Curated by issue keywords and label relevance to help you ship faster.
Loading paid plugin recommendations...
Browse Plugin Categories
Jump directly to plugin category pages on the marketplace.