Code review comment for lp:~wallyworld/launchpad/recipe-find-related-branches

Revision history for this message
Guilherme Salgado (salgado) wrote :

I think it's good to have the links to the SourcePackage, but we should also have them for the ProductSeries. Maybe that could replace the owner column? Or is the owner really important for some reason?

I thought the text area widgets were constrained by the width of the surrounding elements, but I was wrong; zope's TextAreaWidget uses, by default, a 60-cols width. I'd also prefer if the tables had the same width as the other form elements, but I'm not sure using a fixed width on the table is a good idea because the widget's width might change but most importantly because I don't think the fixed-width table will behave the same way as the textare widget when the window is resized to a smaller size than what's needed to fit the textarea.

« Back to merge proposal