Bug 4241 - User can incorrectly pin vncviewer on taskbar
Summary: User can incorrectly pin vncviewer on taskbar
Status: REOPENED
Alias: None
Product: ThinLinc
Classification: Unclassified
Component: VNC (show other bugs)
Version: 3.2.0
Hardware: PC Windows
: P2 Normal
Target Milestone: LowPrio
Assignee: Pierre Ossman
URL:
Keywords:
Depends on: 2805 5373
Blocks:
  Show dependency treegraph
 
Reported: 2012-03-28 11:22 CEST by Pierre Ossman
Modified: 2022-10-04 13:05 CEST (History)
1 user (show)

See Also:
Acceptance Criteria:


Attachments

Description Pierre Ossman cendio 2012-03-28 11:22:38 CEST
If you use the pinning feature in Windows on a running session, it will pin a reference to vncviewer.exe and not tlclient.exe. This is of course not what we want.

There are two options here:

 a) Prevent pinning

 b) Force pinning to associate with tlclient.exe

They both require similar kind of effort, so b) is probably more sensible. Documentation for doing this can be found here:

http://msdn.microsoft.com/en-us/library/dd378459(v=vs.85).aspx

Unfortunately it involves a lot of magical COM, so it's probably a fair bit of work. Might also need fixes in mingw.
Comment 1 Peter Åstrand cendio 2012-05-14 11:36:40 CEST
Also, if you drag the shortcut to the taskbar, grouping does not work.
Comment 2 Peter Åstrand cendio 2014-10-13 12:25:24 CEST
One workaround for this problem could be to tweak vncviewer.exe so that it instead launches tlclient.exe, if called without arguments.
Comment 3 Peter Åstrand cendio 2016-07-22 10:56:37 CEST
I'm not seeing this problem on my Windows 7 laptop with the 4.6.0 client - it works correctly, as far as I can tell. Reopening for discussion.
Comment 4 Peter Åstrand cendio 2016-08-09 10:33:27 CEST
Since this seems to work, we are moving to test. The tester should however see if it is possible to reproduce the problem with earlier versions of the client and/or Windows.
Comment 5 Henrik Andersson cendio 2016-08-11 10:45:29 CEST
(In reply to comment #4)
> Since this seems to work, we are moving to test. The tester should however see
> if it is possible to reproduce the problem with earlier versions of the client
> and/or Windows.

Tested using 4.6.0 client.

Verified an pinned shortcuts (Win 7, Win 8) still points to vncviewer.exe which with Bug 5373 opens the ThinLinc client.

I couldn't verify Win 10 due to its shortcut was not stored on disk

Note that Win10 didn't store the shortcut on the same place as Win 7/8 but google says it should so i couldnt verify this platform
Comment 6 Pierre Ossman cendio 2017-10-24 14:15:32 CEST
We haven't checked what the consequences of the current workaround is, but they will most likely be related to actions performed on the exe:s. E.g. getting the properties of the file and adjusting compatibility settings, or dragging files to the shortcut and expecting the right program to open it.

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