Bug 144681 - UI: Comment box starts flashing on keypress when in table causing lots of redundant paints (hammering the CPU)
Summary: UI: Comment box starts flashing on keypress when in table causing lots of red...
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
7.0.0.1 rc
Hardware: All All
: lowest trivial
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: perf
Depends on:
Blocks: Writer-Comments
  Show dependency treegraph
 
Reported: 2021-09-23 15:23 UTC by Telesto
Modified: 2022-05-28 19:47 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:


Attachments
Example file (9.89 KB, application/vnd.oasis.opendocument.text)
2021-09-23 15:24 UTC, Telesto
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Telesto 2021-09-23 15:23:37 UTC
Description:
UI: Comment box starts flashing on keypress when in table

Steps to Reproduce:
1. Open the attached file
2. Double click the XXXX section in table (with comment)
3. press and hold a key (say a) -> notice the comment box flashing

Actual Results:
Flashing

Expected Results:
Not so


Reproducible: Always


User Profile Reset: No



Additional Info:
Found in
Version: 7.3.0.0.alpha0+ (x64) / LibreOffice Community
Build ID: d5e55d204b71710eb5eb5d2c683dd6698626df3c
CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: Skia/Raster; VCL: win
Locale: nl-NL (nl_NL); UI: en-US
Calc: CL

and in
7.0

and in - with OpenGL and GDI mode
Version: 6.4.0.0.beta1+ (x64)
Build ID: 20be5cd0bdc57d812bf34a2debfe48caa51de881
CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: GL; VCL: win; 
Locale: nl-NL (nl_NL); UI-Language: en-US
Calc: CL

stable, with OpenGL
Version: 6.3.0.0.alpha0+
Build ID: ba1e745b3d022856080c25167226e8a9eeadc911
CPU threads: 4; OS: Windows 6.3; UI render: GL; VCL: win; 
Locale: nl-NL (nl_NL); UI-Language: en-US
Calc: CL

text in comment box blinking with in GDI mode
Version: 6.3.0.0.alpha0+
Build ID: ba1e745b3d022856080c25167226e8a9eeadc911
CPU threads: 4; OS: Windows 6.3; UI render: default; VCL: win; 
Locale: nl-NL (nl_NL); UI-Language: en-US
Calc: CL

blinking text also in
Version: 6.0.6.0.0+
Build ID: c30963b8b4bbbe42a24b97aafa161eff9d7ccdd4
CPU threads: 4; OS: Windows 6.3; UI render: default; 
Locale: nl-NL (nl_NL); Calc: CL

Skia uses the OpenGL code path (as far I'm aware). GDI known the blink. So focus in OpenGL.. [didn't try a Linux backend yet]
Comment 1 Telesto 2021-09-23 15:24:13 UTC
Created attachment 175224 [details]
Example file
Comment 2 Dieter 2021-10-08 06:23:51 UTC
I confirm it with

Version: 7.3.0.0.alpha0+ (x64) / LibreOffice Community
Build ID: 7cd5b35caa8d4fa9d0ba2b2c6ce4b88726ed2be6
CPU threads: 4; OS: Windows 10.0 Build 19043; UI render: Skia/Raster; VCL: win
Locale: de-DE (de_DE); UI: en-GB
Calc: CL

but - to be honest - I've never seen a more trivial bug than this one.
Comment 3 Telesto 2021-10-08 08:01:42 UTC
(In reply to Dieter from comment #2)
> but - to be honest - I've never seen a more trivial bug than this one.

Depends on the perspective :-). Lets give add some background (narrative/ spin). Some people might struggle with this; like people with epilepsy, strobe and visual motion sensitivity. MarjE (marching ants)

Also flashing might mean lots of paint refreshes; which causes high CPU load. It's actually 7% for basic table & a comment in it while holding the key

Pick some lovely big document and like attachment 121138 [details] (there is table with comment on page 1)... and CPU hammering at 25%. 

It's smooth, so you can say well nothing bad.. But well if you're on battery this less joyful (battery drain). 

And next I jump topic the environmental dimensions. A lots energy wasted. If you look at the individual scale this maybe, nah. But if you take into account all users this probably adds up. [Yes, even software could/should have an eco-label]