Re: [virt-tools-list] [PATCH virt-viewer 1/3] Change default screenshot name to "Screenshot.png"

[Pavel Grunt]

On Wed, 2017-07-19 at 16:49 -0500, Jonathon Jongsma wrote:
> Since the code attempts to append ".png" to filenames without an
> extension, it doesn't make much sense to have the default filename be
> extensionless.  Including the extension on the default filename makes
> things more straightforward.

Ack

> 
> Related: rhbz#1455832
> ---
>  src/virt-viewer-window.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/virt-viewer-window.c b/src/virt-viewer-window.c
> index 3ba4e2c..9a6a738 100644
> --- a/src/virt-viewer-window.c
> +++ b/src/virt-viewer-window.c
> @@ -990,7 +990,7 @@ virt_viewer_window_menu_file_screenshot(GtkWidget *menu
> G_GNUC_UNUSED,
>      image_dir = g_get_user_special_dir(G_USER_DIRECTORY_PICTURES);
>      if (image_dir != NULL)
>          gtk_file_chooser_set_current_folder(GTK_FILE_CHOOSER (dialog),
> image_dir);
> -    gtk_file_chooser_set_current_name(GTK_FILE_CHOOSER (dialog),
> _("Screenshot"));
> +    gtk_file_chooser_set_current_name(GTK_FILE_CHOOSER (dialog),
> _("Screenshot.png"));
>  
>      if (gtk_dialog_run(GTK_DIALOG (dialog)) == GTK_RESPONSE_ACCEPT) {
>          char *filename;

_______________________________________________
virt-tools-list mailing list
virt-tools-list@redhat.com
https://www.redhat.com/mailman/listinfo/virt-tools-list