Include a pixel more in the dirty area
AbandonedPublic

Authored by mart on Feb 1 2018, 3:19 PM.

Details

Reviewers
markg
Group Reviewers
Plasma
Konsole
Summary

withfractional scaling sometimes there are visual glithches
in rendering.
painting areas of the terminal is heavily based on QRegion
which is all integer, and sometimes rounding errors can happen
this mitigates the problem, ensuring an area big enough gets
updated when repainting partial areas that cover eventual
pixel fractions that were rounded away
CCBUG:373232

Test Plan

most of the visual glitches are gone now, a missing pixel line
in the background very rarely still happening but seems a separate issue
somewhere else in the painting code

Diff Detail

Repository
R319 Konsole
Branch
phab/dirty
Lint
No Linters Available
Unit
No Unit Test Coverage
mart created this revision.Feb 1 2018, 3:19 PM
Restricted Application added projects: Plasma, Konsole. · View Herald TranscriptFeb 1 2018, 3:19 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
mart requested review of this revision.Feb 1 2018, 3:19 PM
ngraham added a subscriber: ngraham.Feb 1 2018, 4:05 PM

so this is a workaround? Where's the actual bug at?

markg added a subscriber: markg.Feb 4 2018, 9:53 PM

Please add a comment to the code for that.
It might be obvious now, but the next person that looks at that code without knowledge of this change probably has a "huh, what is that?" moment and left wondering why it's done.

markg requested changes to this revision.Feb 4 2018, 9:53 PM
This revision now requires changes to proceed.Feb 4 2018, 9:53 PM
mart updated this revision to Diff 26606.Feb 5 2018, 5:28 PM
  • comments explainig the workaround
mart updated this revision to Diff 26640.Feb 6 2018, 12:01 PM
  • fix cursor painting
hindenburg added a comment.EditedFeb 6 2018, 2:21 PM

I now get all kinds of extra vertical lines on right side of screen and some horizontal ones near cursor using 1.3 scaling - I didn't see these before the latest patch

My zsh prompt is: kurthindenburg:~/ $ [9:17:48]

vertical lines on right side of the last ]
hortizontal lines below current cursor lines

ping... have you had a chance to look at this again?

markg resigned from this revision.Mar 10 2018, 9:15 PM
mart abandoned this revision.Mar 23 2018, 2:56 PM