[Poppler-bugs] [Bug 69978] Make demo extensible for other type of annotations

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=69978

--- Comment #21 from Carlos Garcia Campos <carlosgc at gnome.org> ---
Comment on attachment 87141
  --> https://bugs.freedesktop.org/attachment.cgi?id=87141
glib: Make the Remove annotation button prominent in demo

Review of attachment 87141:
-----------------------------------------------------------------

::: glib/demo/annots.c
@@ +959,4 @@
>      gtk_widget_show (label);
>      g_free (str);
>  
> +    button = gtk_button_new_from_stock (GTK_STOCK_REMOVE);

If we move this close to the Add Annotation button, I would label this one as
Remove Annotation or Delete Annotation. Also note that the button was only
present in the annotation properties section, so it was obvious which
annotation was going to be removed. I would disable the button when there no
annotation selected.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/poppler-bugs/attachments/20131005/be0e499d/attachment.html>