<table><tr><td style="">mglb retitled this revision from "Add line/block characters table for testing" to "Improve built-in line/box characters drawing".<br />mglb edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-5gwzjf2v7s46far/">(Show Details)</a>
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D18735">View Revision</a></tr></table><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Improve built-in line/block characters drawing<br />
<br />
* Fix bold lines (BUG xxxx)<br />
* Pixel-perfect line drawing<br />
* Make line width proportional to font size<br />
<br />
Remove fontembedder/LineFont files<br />
<br />
New line drawing code does not use this.<br />
<br />
Line/block characters: refactoring<br />
<br />
* Move code to separate namespace/file<br />
* Remove code for checking supported line characters from Character<br />
  class. Information about what is supported is now in one place<br />
  together width drawing code<br />
* Tune line widths<br />
* Formatting, etc.<br />
<br />
Copyrights in LineBlockCharactersDrawer.cpp are based on<br />
`git blame -w src/TerminalDisplay.cpp` executed before moving the code<br />
to a separate file. Years from first/last commit. Authors sorted by<br />
year. Whitespace-only changes were ignored. Maksim's code was commited<br />
by Waldo Bastian who mentioned him as the author in commit message<br />
(see 5062b40dd).<br />
<br />
Line/block characters: add triple and quad dashes<br />
<br />
Make code responsible for drawing double dash more generic and add<br />
triple and quadruple dashes (U+2504...U+250B).<br />
<br />
Line/block characters: rename namespace and files<br />
<br />
Remove stray ;<br />
<br />
Remove unused overloaded function<br />
<br />
Change LightWidthToFontSizeRatio to 1/6.5<br />
<br />
DejaVu Sans Mono 8pt (7px character width)<br />
TODO: describe when the 2px/3px starts<br />
<br />
Improve code formating and constant name<br />
<br />
Add frames to line/block characters test table<br />
<br />
Make bold lines at least 1px wider than regular lines<br />
<br />
When font size is big enough (i.e. has width >= 7), bold lines always<br />
will be at least 1px wider than regular lines.<br />
<br />
Use uint instead of qreal for line widths<br />
<br />
Improve built-in line/box characters drawing<br />
<br />
</span>* Fix bold lines (BUG 402415).<div style="padding: 8px 0;">...</div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R319 Konsole</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18735">https://phabricator.kde.org/D18735</a></div></div><br /><div><strong>To: </strong>mglb, Konsole, VDG, fvogt<br /><strong>Cc: </strong>sandsmark, fvogt, konsole-devel, maciejn, thsurrel, ngraham, maximilianocuria, hindenburg<br /></div>