[Konsole-devel] Review Request: Add warning for unlimited scrollback

[Jekyll Wu]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/104452/#review12094
-----------------------------------------------------------


Yeah, the resizing and text moving down in the HistorySizeDialog is a bit annoying. 

As for the waning message, I think there are two minor problems:

1). "may be".  That is not clear and will likely make users wonder: when is "may" and when is "may not" ?  Since the current behavior is the scrollback data is always written into disk file when using unlimited scrollback, let's descibe the behavior as it is.

2). "stored on the hard disk". That "store" might be misleading, because "store" often implies a permanent and stable file within the directory tree which can be easily accessed after closing Konsole. Users might get the wrong idea what Konsole save scrollback data onto disk file in a permenant way without their explicit operation. 

Suggested message: 

"When using this option, the scrollback data are written unencrypted into temporary files. Those temporary files are deleted automatically when quitting Konsole."


- Jekyll Wu


On April 1, 2012, 2:49 p.m., Kurt Hindenburg wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104452/
> -----------------------------------------------------------
> 
> (Updated April 1, 2012, 2:49 p.m.)
> 
> 
> Review request for Konsole.
> 
> 
> Description
> -------
> 
> Some people may not realize that when using 'unlimited scrollback' the scrollback is on the hard drive unencrypted.  The recent libvte 'security' report brought this issue up.
> 
> The warning is pretty bare-bone now.  I wasn't sure if it should have more style to make it stand out more.
> 
> 
> Diffs
> -----
> 
>   src/HistorySizeWidget.cpp fcb131c 
>   src/HistorySizeWidget.ui abba938 
> 
> Diff: http://git.reviewboard.kde.org/r/104452/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Kurt Hindenburg
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20120401/18cee03a/attachment.html>