appsmithorg / appsmith

Platform to build admin panels, internal tools, and dashboards. Integrates with 25+ databases and any API.
https://www.appsmith.com
Apache License 2.0
34.21k stars 3.7k forks source link

[Bug]: The container bg color doesn't have JS mode enabled by default and shows empty value #13614

Open btsgh opened 2 years ago

btsgh commented 2 years ago

Is there an existing issue for this?

Description

The container background color doesn't have JS mode enabled by default and shows empty value, but on clicking JS it shows the correct binding

Steps To Reproduce

Videos showing the behaviour -

on release: LOOM DEMO on master: LOOM DEMO

Public Sample App

No response

Version

Release

dilippitchika commented 2 years ago

@jsartisan no conclusion on this?

keyurparalkar commented 2 years ago

@dilippitchika I am looking into this.

After the initial investigation I found that this issue occurs for the children widgets of List widget. If we observe the container widget's (placed directly on canvas) background color property then the color appears in the property when the JS is toggled:

https://user-images.githubusercontent.com/14138515/167074519-9888e4e4-0903-4924-882d-9177696fd1ae.mov

dilippitchika commented 2 years ago

May not be a release-blocker, as this only happens when the JS toggle is turned off explicitly by the developer. We will fix this later to work according to how it works inside a regular container placed on the canvas.