Review Request 129324: VCS commit message width feedback

[Friedrich W. H. Kossebau]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/129324/#review103528
-----------------------------------------------------------



René, given this one has been transferred to phabricator, could you please help and also close it as discarded, to clean the list of request on reviewboard that still need to see some (final) care?

- Friedrich W. H. Kossebau


On March 22, 2017, 9:35 p.m., René J.V. Bertin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129324/
> -----------------------------------------------------------
> 
> (Updated March 22, 2017, 9:35 p.m.)
> 
> 
> Review request for KDE Software on Mac OS X and KDevelop.
> 
> 
> Repository: kdevplatform
> 
> 
> Description
> -------
> 
> Transferred to https://phabricator.kde.org/D5139
> 
> This patch proposes changes to the way the VCS commit message editor provides feedback concerning the message width.
> 
> KDE insists on a 72 character limit for commit messages, and that is probably a very reasonable standard. In the current implementation there is no easy way to judge how far from that threshold we are. The easiest way to do this is by setting the commit message editor to a width of 72 characters (which for some reason requires taking the width of a string of 74 characters). I think there is room enough for this, but a more complete implementation would probably introduce a resize handle between the 2 main widgets. I haven't yet tried to look into that.
> 
> Several approaches have been tried during the course of this RR.
> 
> 
> Diffs
> -----
> 
>   plugins/git/gitmessagehighlighter.cpp da7660d3eebf9f836e49b48b41df2d11afe83268 
>   vcs/widgets/vcsdiffpatchsources.cpp b080872d4366f65947244cd2c6299fbc14e9ea96 
> 
> Diff: https://git.reviewboard.kde.org/r/129324/diff/
> 
> 
> Testing
> -------
> 
> On OS X with 2 different fixed font typefaces (sizes). The deleteLater() change has been in use since about 4.7.1 .
> 
> 
> File Attachments
> ----------------
> 
> preview: 72-char wide commit message editor
>   https://git.reviewboard.kde.org/media/uploaded/files/2016/11/03/2ffdd73a-db19-4bf2-9545-7a850a4b7fd1__Screen_Shot_2016-11-03_at_18.24.54.png
> commit message showing the overflow warnings/errors (taken on Linux)
>   https://git.reviewboard.kde.org/media/uploaded/files/2016/11/09/54eb4035-0fe1-4d6c-8365-db48bf67b826__KDev-Commit-Dialog.png
> 
> 
> Thanks,
> 
> René J.V. Bertin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20170802/6f0b2827/attachment.html>