Bug 105116 - VIEWING: zoom in garbles display with particular .odt
Summary: VIEWING: zoom in garbles display with particular .odt
Status: RESOLVED WORKSFORME
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
5.4.0.0.alpha0+
Hardware: All Linux (All)
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-01-05 02:10 UTC by Terrence Enger
Modified: 2018-01-22 23:11 UTC (History)
4 users (show)

See Also:
Crash report or crash signature:


Attachments
The .odt in question (8.69 KB, application/vnd.oasis.opendocument.text)
2017-01-05 02:10 UTC, Terrence Enger
Details
Screenshot of garbled screen (58.13 KB, image/png)
2017-01-05 02:12 UTC, Terrence Enger
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Terrence Enger 2017-01-05 02:10:06 UTC
Created attachment 130161 [details]
The .odt in question

STR
---
(0) Download the attached a06.odt.

(1) Open a06.odt from the command line.  Expect both tables to be
    completely visible in the document area; the zoom factor displayed
    in the status bar is 160%.

(2) Take menu options View > Zoom > 200%.  Expect the tables to run
    off the right side of the document area.

(3) Drag the vertical slider down.  Observe garbled display.
    A screenshot is coming soon.

I created a06.odt in bibisect-43max repository version oldest.  I see
the garbled screen in that version and in daily Linux dbgutil
repository version 2017-01-04.  I am making these observations on
debian-stretch.


The repetition of bits of the document throughout the height of the
document area is reminiscent of tdf#80024 "VIEWING: Writer display
error with the Zoom function".  But that bug was resolved in 2014.

Aside: This happened as I was poking into tdf#105025 "table corners
look ugly when you increase cells border size".  Along the way, I also
managed to crash the program I was using to look at screenshots
pixel-by-pixel.  Sigh!
Comment 1 Terrence Enger 2017-01-05 02:12:02 UTC
Created attachment 130162 [details]
Screenshot of garbled screen
Comment 2 Buovjaga 2017-01-05 11:32:51 UTC
Not seeing the garble here with 200% zoom.

Btw. remember to use "bug xxxxx" instead of "tdf#xxxxx" :)

Version: 5.4.0.0.alpha0+
Build ID: 3afe82bd63fde41d2a88418fb64e4ff587b05436
CPU Threads: 4; OS Version: Windows 6.19; UI Render: default; 
TinderBox: Win-x86@42, Branch:master, Time: 2017-01-04_23:42:57
Locale: fi-FI (fi_FI); Calc: group

Arch Linux 64-bit, KDE Plasma 5
Version: 5.4.0.0.alpha0+
Build ID: 1a58cdf8af1aba52ce0a376666dd7d742234d7cf
CPU Threads: 8; OS Version: Linux 4.8; UI Render: default; VCL: kde4; 
Locale: fi-FI (fi_FI.UTF-8); Calc: group
Built on January 4th 2016
Comment 3 Xisco Faulí 2017-01-12 12:15:02 UTC
I can't reproduce it in


Version: 5.4.0.0.alpha0+
Build ID: 9a30cb2161506e003f14592d83075ab9d30516c1
CPU Threads: 4; OS Version: Linux 4.8; UI Render: default; VCL: gtk3; 
Locale: ca-ES (ca_ES.UTF-8); Calc: group
Comment 4 tommy27 2017-06-13 16:40:39 UTC
@Terrence Enger
please retest with a newer master build and tell if issue persists or WORKSFORME
Comment 5 Terrence Enger 2017-06-13 20:07:58 UTC
Thank you for the reminder, tommy27,

I still see the problem on debian-stretch with daily Linux dbutil
bibisect repository version 2017-06-13 and
    "UI render: default; VCL: x11" (i.e., SAL_USE_VCLPLUGIN=gen)
but not with the same LO and
    "UI render: default; VCL: gtk3" (i.e., SAL_USE_VCLPLUGIN=gtk3)

I am setting status back to UNCONFIRMED.
Comment 6 Buovjaga 2017-06-15 15:57:32 UTC
(In reply to Terrence Enger from comment #5)
> Thank you for the reminder, tommy27,
> 
> I still see the problem on debian-stretch with daily Linux dbutil
> bibisect repository version 2017-06-13 and
>     "UI render: default; VCL: x11" (i.e., SAL_USE_VCLPLUGIN=gen)

I can't repro even with SAL_USE_VCLPLUGIN=gen

Arch Linux 64-bit, KDE Plasma 5
Version: 5.5.0.0.alpha0+
Build ID: 2802dca10eef67554a81cb2347d5f648fa6fcd63
CPU threads: 8; OS: Linux 4.11; UI render: default; VCL: x11; 
Locale: fi-FI (fi_FI.UTF-8); Calc: group
Built on June 10th 2017
Comment 7 Xisco Faulí 2017-10-31 17:21:36 UTC
He Terrence,
Still happening on master?
Comment 8 Xisco Faulí 2018-01-11 09:01:00 UTC Comment hidden (obsolete)
Comment 9 Terrence Enger 2018-01-22 23:11:35 UTC
Thank you for the reminder, Xisco.

I see that the bug is gone in daily Linux dbgutil bibisect repository
version 2018-01-22 running on debian-buster.  I am setting status
RESOLVED WORKSFORME.

Reverse bibisect in that repository shows that the fix happened in
version 2017-11-30.  Among the 96 or so source commits since version
2017-11-29, I notice:

    commit ba4a124b0c0c66fd275f5147d55eeec27ce78da9
    Author: Caolán McNamara <caolanm@redhat.com>
    Date:   Wed Nov 29 15:32:39 2017 +0000

        Resolves: tdf#114117 draw page shadow mangled
    
        when cairo had to stretch the 1 pixel dimension to fit the area.
    
        When its this 1 pixel case set the same settings we use for the stipple effect
        to get the expected interpolation
    
        Change-Id: I42d87d6d01ebdb44083351a9632860f5f0a3bf63
        Reviewed-on: https://gerrit.libreoffice.org/45511
        Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
        Reviewed-by: Caolán McNamara <caolanm@redhat.com>
        Tested-by: Caolán McNamara <caolanm@redhat.com>

Caolán, do you think this bug deserves to be RESOLVED FIXED?