Bug 292703 - Documentation required for kate auto-wrapping long lines
Summary: Documentation required for kate auto-wrapping long lines
Status: RESOLVED FIXED
Alias: None
Product: kate
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Arch Linux Linux
: NOR normal
Target Milestone: ---
Assignee: T.C. Hollingsworth
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-01-28 16:28 UTC by Nikos Platis
Modified: 2012-05-04 14:54 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In: 4.8.2


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Nikos Platis 2012-01-28 16:28:04 UTC
Version:           unspecified (using KDE 4.8.0) 
OS:                Linux

Just installed KDE 4.8 and noticed that now kate automatically wraps lines longer than 1024 characters (cf. Bug 236833).

For some people this is totally unacceptable since their files may have long lines for a reason -- and since kate is used by most (if not all) native KDE editors they may have no way out... 

BTW, if kate has performance issues with suce files, as mentioned in Bug 236833, the solution is not to wrap the files, but to fix kate...

Anyway, it seems that this is actually configurable: The editor options contain a new setting "Line length limit" under Open/Save -> General, which, if set to 0 (seems to) disable this behaviour.

Unfortunately this is not documented at all:
a) The informative message that appears when you open a file with long lines does not mention that you are able to configure this setting
b) The options dialog does not have a help message associated with this new setting.
c) As I can see, this is not even documented in the help file: http://docs.kde.org/development/en/kde-baseapps/kate/config-dialog-editor.html#pref-open-save

All of the above points should be fixed until kate can properly handle files with long lines...

Reproducible: Always

Steps to Reproduce:
Open a file with long lines (>1024 chars).

Read the message informing that long lines have been wrapped.


Expected Results:  
The message should inform that this behaviour is configurable.
Comment 1 Massimiliano Torromeo 2012-02-10 08:43:12 UTC
+1, I also had some trouble finding the configuration for this feature.
Also I think that 1024 chars is a bit low as a default value, but if the configuration settings was correctly pointed out it would be a non-issue.
Comment 2 T.C. Hollingsworth 2012-02-17 03:17:05 UTC
I can add this to the documentation for 4.8.2, but the dialog box won't get updated until 4.9 due to string freeze.
Comment 3 T.C. Hollingsworth 2012-02-17 04:39:42 UTC
Git commit ab3e8300e5a118b88c4f304ce6686c3829305466 by T.C. Hollingsworth.
Committed on 17/02/2012 at 05:38.
Pushed by hollingsworth into branch 'master'.

make it more clear that wrapping too long lines is configurable in warning

M  +4    -4    part/document/katedocument.cpp

http://commits.kde.org/kate/ab3e8300e5a118b88c4f304ce6686c3829305466
Comment 4 T.C. Hollingsworth 2012-02-17 04:39:42 UTC
Git commit 4a26964ae6f01aadbb53a656cc5f93f87ba60787 by T.C. Hollingsworth.
Committed on 17/02/2012 at 05:27.
Pushed by hollingsworth into branch 'master'.

docs: document Line Length Limit in katepart config
FIXED-IN: 4.8.2

M  +11   -3    doc/kate/configuring-part.docbook

http://commits.kde.org/kate/4a26964ae6f01aadbb53a656cc5f93f87ba60787
Comment 5 T.C. Hollingsworth 2012-02-17 04:45:14 UTC
Thanks for reporting this!

The documentation has been updated, and will be included with Kate & KWrite in KDE 4.8.2.  It will also appear on the development branch of http://docs.kde.org/ in about a week or so.

I also updated the warning to make it more clear that this is configurable, but unfortunately that won't appear in Kate until 4.9 because 4.8 is under a string freeze.
Comment 6 T.C. Hollingsworth 2012-05-04 14:54:20 UTC
Git commit 392f5d2303861b7e876c003601c43decd282b1ef by T.C. Hollingsworth.
Committed on 17/02/2012 at 05:27.
Pushed by hollingsworth into branch 'KDE/4.8'.

docs: document Line Length Limit in katepart config

M  +11   -3    doc/kate/configuring-part.docbook

http://commits.kde.org/kate/392f5d2303861b7e876c003601c43decd282b1ef