Code review comment for lp:~bac/launchpad/bug-643538-code

Revision history for this message
Brad Crittenden (bac) wrote :

Salgado thanks for the review and the questions.

I tried creating some branches with absurdly long names to see what happened. Using a browser that is almost 1400 pixels wide I get http://people.canonical.com/~bac/code_usage/5-code-still-fits.png. I should probably sees what that looks like on a netbook instead of my 27" monitor. :)

Shrinking the browser from there causes bad behavior seen at http://people.canonical.com/~bac/code_usage/6-too-narrow-overlap.png. I'll work with Curtis to see what can be done.

The other application pages have portlets similar to the one I'm proposing. The link styles are patterned after the involvement portlet on a project index page as seen by a project owner so it is on unprecedented. I think the layout needs some tweaking.

« Back to merge proposal