Task #1711
closed
Trim or warn about leading and trailing whitespaces
Added by Alena Peterová almost 5 years ago.
Updated about 4 years ago.
Description
Version 9.6 (current)
It's a common mistake when administrators copy values to the configuration, that they copy leading or trailing whitespace. E.g. when you set " org.postgresql.Driver" to the Database Table connector, it fails to find the driver. The extra whitespace is hard to notice.
Also when users manually fill in some text fields, they sometimes use whitespace by accident.
It would be really helpful to trim the whitespace in text fields, or at least warn about it. Everywhere :-)
- Assignee changed from Radek Tomiška to Ondrej Husník
- Target version set to 10.1.0
- Priority changed from Normal to High
- Status changed from New to In Progress
More precise definition of the task:
1) user is warned about presence of leading/trailing whitespaces.
2) this warning is supposed to be a "soft" warning which only informs, but doesn't prevent form saving or other actions
3) this feature is available for TextField and TextArea components and is enabled by default.
4) the warning is displayed immediately after a form, containing some leading/trailing whitespaces, is opened
- Estimated time set to 16.00 h
- Status changed from In Progress to Needs feedback
- Assignee changed from Ondrej Husník to Radek Tomiška
- % Done changed from 0 to 90
- Status changed from Needs feedback to Resolved
- Assignee changed from Radek Tomiška to Ondrej Husník
- % Done changed from 90 to 100
I did test and code review, it works and code looks nice, thx for this awesome feature!
Merged into develop.
Note: I personally prefer method name "isTrimmable" (the same as notification message) :)
- Status changed from Resolved to Needs feedback
- Assignee changed from Ondrej Husník to Radek Tomiška
- Status changed from Needs feedback to Resolved
- Assignee changed from Radek Tomiška to Ondrej Husník
Thx, it's awesome!
Merged into develop.
- Status changed from Resolved to Closed
- Related to Task #1111: Trim the whitespaces in reg form added
Also available in: Atom
PDF