bpmn-io / bpmn-js-properties-panel

A properties panel for bpmn-js.
MIT License
293 stars 196 forks source link

Automatically resize text areas (+ text fields that can be multi-line) #713

Closed nikku closed 2 months ago

nikku commented 2 years ago

Is your feature request related to a problem? Please describe

As a user I'm often editing text that may be multi-line (or single line).

I'm struggling to use the properties panel as the text input controls don't work intuitive for me:

Describe the solution you'd like

Support for automatic resizing of all text areas that are not by definition "single line":

I want these changes to be applied consistently so our users get to enjoy a uniform UX.

Describe alternatives you've considered

None.

Additional context

I believe we did have a better UX around multi-line fields in bpmn-js-properties-panel@0.x.

Related to https://github.com/bpmn-io/properties-panel/issues/25


The following shows my usual flow of "editing" multi-line text:

capture WE6RXC_optimized

nikku commented 3 months ago

Moving to ready to see (review) how far we are off from consistently fixing this. We invested in the past.

jarekdanielak commented 2 months ago

The problem here seems to be that the text area you are using for demonstration doesn't have autoResize property set to true.

Is this still relevant? Can you suggest which fields should have autoResize turned on?

barmac commented 2 months ago

Perhaps we want to autoresize all text areas? Make it the new default.

nikku commented 2 months ago

We want indeed to auto-resize all text areas (that are not by definition "single line"). This ensures a uniform UX across the whole properties panel. Cf. also https://github.com/camunda/camunda-modeler/pull/4429#issuecomment-2247787963.

jarekdanielak commented 2 months ago

Fixed upstream in bpmn-io/properties-panel@3.23.0