Bug 1499018

Summary: bump scrolling is broken in tigervnc 1.8.0
Product: Red Hat Enterprise Linux 7 Reporter: Andrew Schorr <ajschorr>
Component: tigervncAssignee: Jan Grulich <jgrulich>
Status: CLOSED ERRATA QA Contact: Desktop QE <desktop-qa-list>
Severity: high Docs Contact:
Priority: unspecified    
Version: 7.4CC: jkoten, lmiksik, thudziec
Target Milestone: rc   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2018-04-10 11:37:45 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
bump scrolling patch from upstream
none
spec file patch none

Description Andrew Schorr 2017-10-05 20:16:27 UTC
Description of problem:
In fullscreen mode, bump scrolling is broken in tigervnc 1.8.0.
This has been reported and fixed upstream here:
https://github.com/TigerVNC/tigervnc/issues/477

Version-Release number of selected component (if applicable):
1.8.0

How reproducible:
Run vncviewer in fullscreen mode and notice that bumping up against the edge of the screen does not induce any scrolling.

Steps to Reproduce:
1. vncviewer -FullScreen <vnc server with desktop larger than local display>
2. bump up against edge of screen and see that scrolling does not occur.
3.

Actual results: Bump scrolling is broken in this release.


Expected results: Bump scrolling should work.


Additional info: 
Please grab patch from upstream https://github.com/TigerVNC/tigervnc/commit/e8d25d83463805c0f6ef623dba2ac9a276df3587

Comment 2 Jan Grulich 2017-10-06 06:10:39 UTC
Can be easily backported.

Comment 3 Andrew Schorr 2017-10-06 13:52:44 UTC
Needless to say, Fedora 26 suffers from the same bug.

Comment 4 Andrew Schorr 2017-10-08 21:56:39 UTC
Created attachment 1335984 [details]
bump scrolling patch from upstream

https://github.com/TigerVNC/tigervnc/commit/e8d25d83463805c0f6ef623dba2ac9a276df3587.patch

Comment 5 Andrew Schorr 2017-10-08 21:57:25 UTC
Created attachment 1335985 [details]
spec file patch

This seems to work for me.

Thanks,
Andy

Comment 6 Andrew Schorr 2017-12-06 13:31:20 UTC
Hi.

I hate to be a pain in the ass, but if this patch can be easily backported, why hasn't it happened yet? It couldn't be much easier. I have already supplied the patch. And now you pushed out a new release with tigervnc-working-tls-on-fips-systems.patch and did not include this bump scroll patch. Why is that? It would have been so insanely easy to include both patches.

Regards,
Andy

Comment 7 Jan Grulich 2017-12-07 07:31:40 UTC
That fix I pushed was critical. I would have pushed even this fix, but it didn't get all the acks and I was a bit overwhelmed back then, so I really didn't have time to poke people around to approve this.

Comment 8 Jan Grulich 2018-01-18 12:57:29 UTC
Fixed in tigervnc-1.8.0-5.el7.

Comment 10 Tomas Hudziec 2018-02-06 16:52:09 UTC
Verified that bump scrolling does not work with tigervnc-1.8.0-4.el7 and works with tigervnc-1.8.0-5.el7. Tested on x86_64.

Comment 13 errata-xmlrpc 2018-04-10 11:37:45 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2018:0722