Re: [evolution-patches] Fix to resize the composer attachment bar

[Not Zed <notzed ximian com>]

I have to agree.  It seems to be a completely sub-optimal solution to
the problem.

First, force the attachment bar to be open all the time, removing a
useful and common trick of having a collapser (a favourite on mac i
might add, which we seem to be religiously copying in gnome).  Then find
'oh dear, this takes up too much space', so hack in a very crappy
solution of letting the user re-size the bar - which given how big and
silly the icons are will make it unusable unless it is full size anyway.

It just demonstrates that the first thought wasn't fully evaluated
before it was implemented by Srini.  Sigh.

I really dont mind having it resizable, it just seems a pretty poor
workaround to a problem that didn't even exist before!


On Thu, 2005-06-09 at 10:06 -0400, Jeffrey Stedfast wrote:
> personally I'd prefer the way 2.2 did it, which was just that the
> attachment pane was collapsable.
> 
> the problem with resizing here is that there's an optimum size for the
> attachment pane to be - there's no real reason for a user to change it -
> the text in the editor is far more important content than the attachment
> icons; as long as a user can see the full attachment info in the
> vertical space provided, he/she can always use the scrollbars to view
> more attachments if they've attached too many to display in a single
> row.
> 
> resizing the attachment bar to a smaller size doesn't really make sense
> either as it eliminates the value of having it viewable in the first
> place.
> 
> hope that makes sense...
> 
> Jeff
> 
> On Thu, 2005-06-09 at 00:57 -0600, Srinivasa Ragavan wrote:
> > Hi,
> > 
> > I have attached a patch, which allows to resize the attachment bar when
> > its visible.
> > 
> > Please review and give back the comments.
> > 
> > Thanks
> > Srini.
> > _______________________________________________
> > evolution-patches mailing list
> > evolution-patches lists ximian com
> > http://lists.ximian.com/mailman/listinfo/evolution-patches