Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Auto-trim leading whitespaces when pasting into property fields #309

Open
christian-konrad opened this issue Nov 3, 2023 · 2 comments
Open
Labels
backlog Queued in backlog enhancement New feature or request spring cleaning Could be cleaned up one day

Comments

@christian-konrad
Copy link

christian-konrad commented Nov 3, 2023

Is your feature request related to a problem? Please describe.

When you copy things like URLs and paste them into the properties panel, it can happen that you accidently copy leading white spaces. They will be copied into the property field, and you might not spot that easily. This can lead to problems such as regex validation failing (in the URL example), and the user does not understand why.

image

Screen.Recording.2023-11-03.at.11.48.51.mov

Describe the solution you'd like

Auto-trim leading whitespaces when pasting into property fields.

Describe alternatives you've considered

We need to confirm that there is no valid use case of pasting leading white spaces that we will break with such a feature.

Additional context

@christian-konrad christian-konrad added enhancement New feature or request spring cleaning Could be cleaned up one day labels Nov 3, 2023
@barmac
Copy link
Member

barmac commented Nov 3, 2023

What we need to consider is that sometimes you actually want to paste the leading space in case you append content to existing value. Still, I think the requested feature would absolutely make sense for empty fields.

@nikku
Copy link
Member

nikku commented Nov 7, 2023

This may be a quick fix for the bigger issue reported once in a while by our users: camunda/camunda-modeler#2385.

Often times (in technical configuration) whitespace is bad. More general trimming is what we considered in the past, but did not follow up on.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog Queued in backlog enhancement New feature or request spring cleaning Could be cleaned up one day
Development

No branches or pull requests

4 participants