Created attachment 923636 [details] images Description of problem: when viewing several snapshots via snapshot overview (see image) UI displays only one snapshot as being previewed (the one with "VM CONFIGURATION=1") Version-Release number of selected component (if applicable): beta.2 How reproducible: 100% Steps to Reproduce: 1.view snapshots via snapshot overview Actual results: only one snapshot is marked as being viewed Expected results: one should expect to see all previewed items status aligned Additional info:
Ori - It doesn't look like the attachment is working, would you mind reattaching? Thanks, Liz
Created attachment 924079 [details] screenshots
Seems as though the attachment is OK, is was just uploaded with the wrong mime type (patch instead of zip), so bugzilla doesn't recognize it properly. Re-uploaded.
Hi Liz, iiuc, the issue here is in the indication of disks that are currently being previewed. Currently, a previewed snapshot can be identified by the 'Preview Mode' string under its description. In case of custom preview, that string is added to the VM configuration snapshot while the selected disks can be viewed under disks tab. To improve/clarify the UX, perhaps we can add a similar indication for disks under snapshots tab? I.e. something like adding a list of previewed disks under snapshot description? [Attached is a mock-up for illustration]
Created attachment 924213 [details] snapshot_preview_disks
Created attachment 924215 [details] mockup: suggested improvement
ok, I just uploaded attachment 924215 [details] without noticing that Daniel has uploaded a solution a couple of minutes before that. :) all in all, the solutions are pretty similar - I am not sure whether we should display "OK", "In Preview" or something else for the snapshot's status for which Disks are previewed. not critical though.
@Daniel - This make complete sense to me to allow for this same detail to be available to the user in the Snapshots tab after the user has selected a specific VM. Reusing the exact same column title and row styling makes most sense to me. Thanks for the extra clarification on this. Best, Liz
is this should be fixed on vt3.1 build? because it isn't
After discussion with derez,the fix is not yet merged. returning to modify
This is an oVrit bug, with a fix included in ovirt-3.5.0_rc2, and thus must be ON_QA. The fact that it's not included in any RHEVM build yet is inconsequential.