Although we're backlogged on reviews right now, submitting a patch even for a trivial issue prioritizes getting the problem fixed rather than waiting for us to get to it some time in the development cycle.
-- Jim On Mon, Dec 10, 2012 at 5:55 PM, Tom Most <[email protected]> wrote: Okay, will do. For trivial issues like this, is is preferred to generate a patch like this or just create a ticket? Thanks, Tom On Mon, Dec 10, 2012 at 11:12 AM, Jim Nelson wrote: > I've created a ticket and added your patch to it: > > http://redmine.yorba.org/issues/6126 > > In the future, it's more efficient if you create the ticket (please search > to see if one already exists!) and directly attach the patch there, rather > than including it inline in an email to the mailing list. > > Cheers, > > -- Jim > > > On Fri, Dec 7, 2012 at 1:32 AM, Tom Most wrote: > > From 1a7c1961ad390f339e33d2ac0b9c37550d7556df Mon Sep 17 00:00:00 2001 > From: Tom Most > Date: Thu, 6 Dec 2012 00:18:32 -0800 > Subject: [PATCH] Wrap text in the external images info bar > > Previously the text would push the buttons over the right edge of the > window, given a narrow conversation view. This change makes Geary more > usable with narrow window sizes. > --- > src/client/views/conversation-viewer.vala | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/src/client/views/conversation-viewer.vala > b/src/client/views/conversation-viewer.vala > index 9b0d3c0..d024b33 100644 > --- a/src/client/views/conversation-viewer.vala > +++ b/src/client/views/conversation-viewer.vala > @@ -93,6 +93,7 @@ public class ConversationViewer : Object { > external_images_info_bar.get_content_area() as Gtk.Box; > if (external_images_info_bar_content_area != null) { > Gtk.Label label = new Gtk.Label(_("This message contains > images. Do you want to show them?")); > + label.set_line_wrap(true); > external_images_info_bar_content_area.add(label); > label.show_all(); > } > -- > 1.8.0.1 > _______________________________________________ > Geary mailing list > [email protected] > http://lists.yorba.org/cgi-bin/mailman/listinfo/geary > > _______________________________________________ Geary mailing list [email protected] http://lists.yorba.org/cgi-bin/mailman/listinfo/geary
