Bug 1459800

Summary: remote-viewer should pop up the recent dialog instead of a main window when running "remote-viewer -k"
Product: Red Hat Enterprise Linux 7 Reporter: Xiaodai Wang <xiaodwan>
Component: virt-viewerAssignee: Victor Toso <victortoso>
Status: CLOSED ERRATA QA Contact: Virtualization Bugs <virt-bugs>
Severity: low Docs Contact:
Priority: low    
Version: 7.4CC: dblechte, elima, juzhou, kuwei, mxie, mzhan, rbalakri, tzheng, victortoso, xiaodwan
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 08:50:15 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
remote viewer recent dialg none

Description Xiaodai Wang 2017-06-08 08:22:16 UTC 
Description of problem:
remote-viewer should pop up the recent dialog instead of a main window when running "remote-viewer -k" 

Version-Release number of selected component (if applicable):
virt-viewer-5.0-7.el7.x86_64

How reproducible:
100%

Steps to Reproduce:
1. Run "remote-viewer -k" in terminal.

Actual results:
A blank main window displays in fullscreen.

Expected results:
The recent dialog should pop up. 
when users type correct uri, and then click connect button, remote-viewer can enter kiosk mode.

Additional info:
Comment 2 Victor Toso 2017-06-09 15:32:55 UTC 
Could you please confirm that the recent dialog pop up actualy exists but it is hidden on kiosk mode? Alt + ` after running remote-viewer should make the recent dialog to show.
Comment 3 Victor Toso 2017-06-09 16:30:52 UTC 
Patch sent: https://www.redhat.com/archives/virt-tools-list/2017-June/msg00049.html
Comment 4 Xiaodai Wang 2017-06-12 02:21:29 UTC 
(In reply to Victor Toso from comment #2)
> Could you please confirm that the recent dialog pop up actualy exists but it
> is hidden on kiosk mode? Alt + ` after running remote-viewer should make the
> recent dialog to show.

yes, the recent dialog has already been popped but sits under the main window.
Comment 5 Victor Toso 2017-06-12 13:39:32 UTC 
Pushed upstream: 58dd08f222b49781b0e11d4e067d6cd85e16676d
Comment 7 Xiaodai Wang 2017-10-19 07:43:37 UTC 
Hi, i tested this bug with virt-viewer-5.0-8.el7.x86_64 and gnome-desktop3-3.22.2-2.el7.x86_64, the dialog sits above the main window now.

But the recent dialog is on top-left corner of the main window and doesn't have focus. Pls see the screenshot of this problem.

Thanks
xiaodwan
Comment 8 Xiaodai Wang 2017-10-19 07:45:29 UTC 
Created attachment 1340577 [details]
remote viewer recent dialg
Comment 10 Eduardo Lima (Etrunko) 2017-10-31 15:37:44 UTC 
I will check if the build you mentioned ships the fix for this bug.
(In reply to xiaodwan from comment #7)
> Hi, i tested this bug with virt-viewer-5.0-8.el7.x86_64 and
> gnome-desktop3-3.22.2-2.el7.x86_64, the dialog sits above the main window
> now.
> 
> But the recent dialog is on top-left corner of the main window and doesn't
> have focus. Pls see the screenshot of this problem.


Yes, the build includes the fix for the problem, but for repositioning the dialog in the center of the main window we need another patch, which is not in the build and in fact could be tracked in another bz.
Comment 12 tingting zheng 2017-11-01 03:14:12 UTC 
(In reply to Eduardo Lima (Etrunko) from comment #10)
> I will check if the build you mentioned ships the fix for this bug.
> (In reply to xiaodwan from comment #7)
> > Hi, i tested this bug with virt-viewer-5.0-8.el7.x86_64 and
> > gnome-desktop3-3.22.2-2.el7.x86_64, the dialog sits above the main window
> > now.
> > 
> > But the recent dialog is on top-left corner of the main window and doesn't
> > have focus. Pls see the screenshot of this problem.
> 
> 
> Yes, the build includes the fix for the problem, but for repositioning the
> dialog in the center of the main window we need another patch, which is not
> in the build and in fact could be tracked in another bz.

Xiaodai,would you pls check the comments and report a new bug for the above issue,thanks.
Comment 13 Xiaodai Wang 2017-11-01 03:22:56 UTC 
> Yes, the build includes the fix for the problem, but for repositioning the
> dialog in the center of the main window we need another patch, which is not
> in the build and in fact could be tracked in another bz.

I created bug 1459800 for the wrong position of the recent dialog, and move this bug from ON_QA to VERIFIED.

Thanks
Comment 14 Xiaodai Wang 2017-11-01 03:24:49 UTC 
(In reply to xiaodwan from comment #13)
> > Yes, the build includes the fix for the problem, but for repositioning the
> > dialog in the center of the main window we need another patch, which is not
> > in the build and in fact could be tracked in another bz.
> 
> I created bug 1459800 for the wrong position of the recent dialog, and move
> this bug from ON_QA to VERIFIED.
> 
> Thanks

Sorry, bug 1508274.
Comment 17 errata-xmlrpc 2018-04-10 08:50:15 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:0672