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.1k stars 3.68k forks source link

[Bug]: Add validations for Emphasis property #28252

Open shadabbuchh opened 11 months ago

shadabbuchh commented 11 months ago

Is there an existing issue for this?

Description

Although the Expected structure says String, on entering a value starting with upper case for Emphasis does not render appropriately. Can we please make this consistent?

Screenshot 2023-10-20 at 11 54 10 AM

Steps To Reproduce

  1. DnD a Text widget.
  2. Go to the property pane and enable the JS option for Emphasis.
  3. Now enter Bold or Italic.

Expected: We should get an Disallowed value error as the first character is in upper case. Actual: We neither get an error nor does the emphasis render on the canvas

Public Sample App

No response

Environment

Release

Issue video log

No response

Version

Cloud

Nikhil-Nandagopal commented 7 months ago

Bumping this issue up to prod because it should have made it's way there by now if it was not fixed