Re: [evolution-patches] [Calendar] line wrap in week view



Looks good otherwise. Get it committed to head as well as 2.4.x

-Srini
On Wed, 2006-01-11 at 01:13 -0700, Veerapuram Varadhan wrote:
> On Wed, 2006-01-11 at 08:03 +0000, Johnny Jacob  wrote:
> 
> > +2006-01-11  Johnny Jacob  <johnnyjacob#gmail.com>
> > +
> > +       * text/e-text.c (e_text_set_property): Add the width to
> > +       the pango layout.
> > +
> If it fixes a bug, please mention the bug id.
> 
> >         case PROP_CLIP_WIDTH:
> >                 text->clip_width = fabs (g_value_get_double (value));
> >                 calc_ellipsis (text);
> > -               if ( text->line_wrap )
> > +               if ( text->line_wrap ) {
> > +                       if (text->layout)
> > +                               pango_layout_set_width (text->layout,
> > text->clip_width < 0 ? -1 : text->clip_width * PANGO_SCALE);
> Please follow the coding style used in the code.  Other than that, the
> patch looks good.
> 
> V. Varadhan
> _______________________________________________
> Evolution-patches mailing list
> Evolution-patches gnome org
> http://mail.gnome.org/mailman/listinfo/evolution-patches




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]