[Konsole-devel] Review Request 104605: Change the "Bold" key of ColorScheme from boolean to tri-state

Kurt Hindenburg kurt.hindenburg at gmail.com
Sat Apr 6 16:44:12 UTC 2013



> On April 5, 2013, 8:16 p.m., Albert Astals Cid wrote:
> > This doesn't apply to master anymore. If this Review Request doesn't see any action before my next janitorial round I'll assume noone cares about it and mark it as discarded to clear the queue.

I'll close it - we decided not to use this and remove the per-color-bold option


- Kurt


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


On April 15, 2012, 11:35 a.m., Jekyll Wu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/104605/
> -----------------------------------------------------------
> 
> (Updated April 15, 2012, 11:35 a.m.)
> 
> 
> Review request for Konsole.
> 
> 
> Description
> -------
> 
> Tha patch is trivial. Konsole now supports "Bold=true", "Bold=false" and other values(which all mean using Character::UseCurrentFormat).
> 
> However, I just wonder how useful this per-color boldness controlling could be. We already have the "BoldIntense" per-profile option to enable or disable boldness. Do we need two levels of boldness control: per-profile and per-color in each colorscheme?
> 
> 
> This addresses bug 168300.
>     http://bugs.kde.org/show_bug.cgi?id=168300
> 
> 
> Diffs
> -----
> 
>   src/ColorScheme.cpp 1acb4d4 
> 
> Diff: http://git.reviewboard.kde.org/r/104605/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Jekyll Wu
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20130406/6a6cede8/attachment.html>


More information about the konsole-devel mailing list