Bug 432283 - More generous value range for tab width
Summary: More generous value range for tab width
Status: RESOLVED FIXED
Alias: None
Product: kate
Classification: Applications
Component: indentation (show other bugs)
Version: 20.04.2
Platform: openSUSE Linux
: NOR wishlist
Target Milestone: ---
Assignee: KWrite Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-01-29 17:15 UTC by Chris K
Modified: 2021-01-30 11:32 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments
tab-separated CSV file, colums wider than 16 (193 bytes, text/csv)
2021-01-29 17:15 UTC, Chris K
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Chris K 2021-01-29 17:15:03 UTC
Created attachment 135286 [details]
tab-separated CSV file, colums wider than 16

The maximum width I can configure the tabulator character to take is 16.
This is probably more than enough for 90% of all cases or more but lately I have to deal with a scenario more often where a larger value would be nice.
16 seems like an arbitrary and unnecessarily low value. I propose to allow e.g. up to 99; this can still be represented by a signed byte (if there is a technical constraint) and presented in a two digits wide integer input (if this plays a role).

Concrete case for the feature request: CSV files where the separator is tab, not comma. When such files have few columns but large values, such files can theoretically nicely be viewed and edited in Kate (given a reasanable window size). For values that are wider than 16 characters, however, currently the alignment of the columns is lost.


STEPS TO REPRODUCE
1. Open attached example.csv
2. Try to set tab width e.g. to 28

OBSERVED RESULT
Even the "miscellaneous" option allows only for max. 16.

EXPECTED RESULT
Possibility to set at least, say, 50; maybe 99 or even greater.

SOFTWARE/OS VERSIONS
Linux/KDE Plasma: openSUSE Leap 15.2
(available in About System)
KDE Plasma Version: 5.18.6
KDE Frameworks Version: 5.71.0
Qt Version: 5.12.7
Comment 1 Bug Janitor Service 2021-01-30 05:52:08 UTC
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/ktexteditor/-/merge_requests/97
Comment 2 Jan Paul Batrina 2021-01-30 09:22:20 UTC
Git commit 39859225eb10a17fd90d1b03b5aab18f23ef0d3a by Jan Paul Batrina.
Committed on 30/01/2021 at 05:41.
Pushed by cullmann into branch 'master'.

Increase maximum indentation width to 200

Indentation widths greater than 16 may be useful when e.g. tabs are used as column separators for long strings

M  +2    -2    src/dialogs/indentationconfigwidget.ui
M  +2    -2    src/view/katestatusbar.cpp

https://invent.kde.org/frameworks/ktexteditor/commit/39859225eb10a17fd90d1b03b5aab18f23ef0d3a
Comment 3 Chris K 2021-01-30 11:32:53 UTC
Thank you for the quick action. I agree that 200 is reasonable (there should be some boundary); certainly works for me.