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 · 6 comments
Open
Labels
channel:support enhancement New feature or request ready Ready to be worked on 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

Related to SUPPORT-25217

@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.

@sbuettner
Copy link

Any chances we can pick this up? We had another support case where a customer created an invalid Connector config by having a whitespace character in their properties which lead to a long running incident process.

@barmac
Copy link
Member

barmac commented Jan 21, 2025

We can take this to the next iteration planning given that it's such a pain for the users. Can you please link the support case in the top comment? We could then label the issue as channel:support.

@barmac barmac added ready Ready to be worked on and removed backlog Queued in backlog labels Jan 21, 2025
@sbuettner
Copy link

@barmac I dont have the permissions to do so. Here is the support ticket: https://jira.camunda.com/browse/SUPPORT-25217?focusedCommentId=390503&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-390503

@barmac
Copy link
Member

barmac commented Jan 21, 2025

Thanks, I updated the issue accordingly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
channel:support enhancement New feature or request ready Ready to be worked on spring cleaning Could be cleaned up one day
Projects
None yet
Development

No branches or pull requests

5 participants