https://bugs.kde.org/show_bug.cgi?id=502233

Christoph Cullmann <christ...@cullmann.io> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |christ...@cullmann.io

--- Comment #2 from Christoph Cullmann <christ...@cullmann.io> ---
(In reply to Waqar Ahmed from comment #1)
> The slowdown is in Qt. Its unable to handle a large number of format ranges
> that are produced by KateRenderer::decorationsForLine when the ranges are
> passed to QTextLayout::draw(). During text layouting this is not an issue.
> 
> One possible fix could be to render selection ourselves always:
> - remove the following check in decorationsForLine so that we use the
> correct foreground color during layouting. This will have implications for
> printing and screenshot so that needs to be kept in mind:
>             if (selectionsOnly) {
>                 assignSelectionBrushesFromAttribute(fr, *a);
>             }
> - Dont use Qt for selection background. Use our own function
> paintTextBackground, that function needs a bit more testing, I think its
> still a little bit broken with multiple wrapped lines
> 
> On quick testing, this removes all lag.

Given we do most other painting ourself, too, that would be fine with me.

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to