After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 710298 - smartcard row disappeared
smartcard row disappeared
Status: RESOLVED NOTABUG
Product: gnome-boxes
Classification: Applications
Component: properties
unspecified
Other Linux
: Normal normal
: 3.22
Assigned To: GNOME Boxes maintainer(s)
GNOME Boxes maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2013-10-16 16:16 UTC by Matthias Clasen
Modified: 2016-03-31 13:22 UTC
See Also:
GNOME target: ---
GNOME version: ---



Description Matthias Clasen 2013-10-16 16:16:45 UTC
(from Montreal summit session)

The smartcart row disappeared after clicking it.
Comment 1 Zeeshan Ali 2014-10-16 14:08:40 UTC
Weird! I can reproduce this. To be clear, its the "Add smartcard support" button that disapears and its not just that but Boxes switches the view away to first page of properties.
Comment 2 Zeeshan Ali 2014-11-08 21:20:27 UTC
(In reply to comment #1)
> Weird! I can reproduce this. To be clear, its the "Add smartcard support"
> button that disapears and its not just that but Boxes switches the view away to
> first page of properties.

At least the last bit is likely connected to refreshing of properties since the same happens if you add/remove a USB device.
Comment 3 Zeeshan Ali 2014-12-04 23:30:15 UTC
The secondary issue I mentioned in comment#1 ("Boxes switches the view away to first page of properties.") has at least been resolved by this patch in git master now:

commit: 56c17f738ffb00687c0030c274370798e8453c9b

    properties: Correctly restore notebook page
    
    We were not actually restoring to the current page after refresh of
    properties due to name clash of 'page' property with a local variable.
    This patches fixes the issue by use of 'this' keyword.
Comment 4 Zeeshan Ali 2015-05-21 17:35:35 UTC
Seems with the redesign of properties view, this is no longer reproducible.
Comment 5 Zeeshan Ali 2015-05-21 17:38:04 UTC
(In reply to Zeeshan Ali (Khattak) from comment #4)
> Seems with the redesign of properties view, this is no longer reproducible.

Actually, no! that is the secondary issue that was fixed by commit 56c17 as I already mentioned in comment#3.

However, I wonder what we want to do here? The row dissapearing was very much intentional since if VM doesn't have the support, we provide an easy way to add it but once it's added, do we really need a way to remove it?

If so, I can change this to a toggle.
Comment 6 Matthias Clasen 2016-02-08 07:37:35 UTC
If it was intentional, lets leave it alone