Hi Sander, looking at your points i have some questions: *>2) **multiple snapshots, discard last snapshot is indeed a merge with its parent (diff images merged; parent saved state deleted)* how can parent have a saved state? I always thought saved states do only occur in the deepest level of the snapshot tree? ** *>3) multiple snapshots, discard other (non-last) snapshot merges the image diff with its child and deletes the saved state * which saved state is deleted? It makes sense the image is merged with the child. Based on this i would rather suggest changing description in UI to something like: "Merge Snapshot into Snapshot-Tree" and in the tooltip: "If the Snapshot is at the deepest level of the snapshot tree it is merged with its parent, else with its child snaphot" Does that make sense? Kind Regards, Johannes Rudolph On Mon, May 4, 2009 at 12:05 PM, Sander van Leeuwen < [email protected]> wrote:
> Hi Johannes, > > Your proposal is not completely correct either. Consider the following > cases: > 1) one snapshot, discard is indeed a merge of the diff image with its > parent (main disk image) > 2) multiple snapshots, discard last snapshot is indeed a merge with its > parent (diff images merged; parent saved state deleted) > 3) multiple snapshots, discard other (non-last) snapshot merges the image > diff with its child and deletes the saved state > > So basically the button and popup menu items should change description > according to the highlighted snapshot. (1 & 2 merge, 3 remains discard) > > Regards, > > Sander > > Johannes Rudolph wrote: > > > Hi all, > > This is my first contribution to VirtualBox, so please be so kind to notice > me if im doing something horribly wrong :-) > > I think button naming in QT GUI for the Snapshot functionality is > confusing. > "Discard Snapshot" *is *discarding the Snapshot, however it does so by > *merging > *it with the parent. This behaviour clearly is not described in the > current description. As also suggested in this blog post > > http://srackham.wordpress.com/cloning-and-copying-virtualbox-virtual-machines/ > i therefore renamed the button to "Merge with Parent". > > See also: Bug #1101 <http://www.virtualbox.org/ticket/1101> > Just to further underline my point that the current button naming is > causing unnescessary confusion. > > I also adapted the German translation accordingly, as its my native > language. > > It would also be nescessary to introduce a new button icon for this, > however i'd first like to ask if this patch is likely to be approved. Im > afraid i couldnt submit a correct svn patch, but i do currently do not have > access to a subversion installation. The two files attached reside in: > trunk/src/VBox/Frontends/VirtualBox/nls/ > trunk/src/VBox/Frontends/VirtualBox/src/ > > respectively. > > Everything i submitted is subject to MIT License. > > Would someone please be so kind to provide some feedback about my > suggestion? > > Kind Regards, > Johannes Rudolph > > > ------------------------------ > > _______________________________________________ > vbox-dev mailing > [email protected]http://vbox.innotek.de/mailman/listinfo/vbox-dev > > > > -- > Kind regards / Mit freundlichen Gruessen / Met vriendelijke groet > > -- > > Sun Microsystems GmbH Sander van Leeuwen > Werkstrasse 24 Senior Staff Engineer, VirtualBox > 71384 Weinstadt, Germany mailto:[email protected] > <[email protected]> > > > ================================================ > Sitz der Gesellschaft: Sun Microsystems GmbH, > Sonnenallee 1, 85551 Kirchheim-Heimstetten > Amtsgericht Muenchen: HRB 161028 > Geschaeftsfuehrer: Thomas Schroeder, > Wolfgang Engels, Dr. Roland Boehmer > Vorsitzender des Aufsichtsrates: Martin Haering > ================================================ > >
_______________________________________________ vbox-dev mailing list [email protected] http://vbox.innotek.de/mailman/listinfo/vbox-dev
