Bug 7374 - Alpha cursor support in Web Access
Summary: Alpha cursor support in Web Access
Status: CLOSED FIXED
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: Web Access (show other bugs)
Version: trunk
Hardware: PC Unknown
: P2 Normal
Target Milestone: 4.12.0
Assignee: Samuel Mannehed
URL:
Keywords: ossman_tester, relnotes, upstream
Depends on: 1122 7365 7373
Blocks:
  Show dependency tree / graph
 
Reported: 2019-09-03 10:07 CEST by Pierre Ossman
Modified: 2020-02-19 15:59 CET (History)
3 users (show)

See Also:
Acceptance Criteria:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Description Pierre Ossman cendio 2019-09-03 10:07:18 CEST 
We implemented support for alpha (RGBA) cursors in the native client (and server) in bug 1122. However we didn't implement the same functionality in Web Access.

The server now also supports a second extension for alpha protocols that is easier to implement, so it might be better to focus on that.

Upstream issue:

https://github.com/novnc/noVNC/issues/900
Comment 1 Alex Tanskanen cendio 2020-01-21 12:58:49 CET 
Tested Web Access against a RHEL 8, and a Ubuntu 18.04 server with build 6364 on:
 ✓ Safari on macOS 10.15
 ✓ Firefox on Fedora 31
 ✓ Chrome on Windows 10
 ✓ IE on Windows 10
 ✓ Edge on Windows 10
Comment 4 Pierre Ossman cendio 2020-01-30 09:41:29 CET 
Release notes look good. Transparency seem to work correctly.

However colors are off. Seems like the classical RGB/BGR mixup. Works fine with the native client.

Seen with Firefox on Fedora.
Comment 5 Samuel Mannehed cendio 2020-01-30 11:50:26 CET 
Ah, the red and blue channels got mixed up. Fixed upstream now: https://github.com/novnc/noVNC/commit/71bb3fdfa58954ca7b0d7be43d974316c727ff4c

Let's wait for bug 3895 to do a vendordrop.
Comment 6 Samuel Mannehed cendio 2020-01-30 11:59:09 CET 
An easy way to test this is to open a browser in the session and go to: https://css-tricks.com/using-css-cursors/#article-header-id-1

On that page there's a demo with the text "How are you feeling today" that changes the cursor to different smilies when hovered.
Comment 8 Alex Tanskanen cendio 2020-02-18 13:43:46 CET 
This has been fixed in upstream and vendordropped so it should now be tested.
Comment 9 Pierre Ossman cendio 2020-02-19 15:59:54 CET 
I can confirm that it now works properly. Alpha is still okay, and no odd colours.

Tested with Firefox on Linux.
Note You need to log in before you can comment on or make changes to this bug.