Code review comment for lp:~donadigo/appcenter/screenshots-pixbuf

Revision history for this message
Danielle Foré (danrabbit) wrote :

This works for me. Definitely makes the screeshots a lot better.

Can you remove that 6px row spacing on the grid that has the sep in it? I agree the sep helps break those boxes up, but it's weird to have that chunk of white between it and the scrollbox.

I also think that we should change the stack so that the preview image expects to take up that 800x600 space instead of the 512. I think we should still seek to reduce text movement here so that we're not interrupting reading when the screenshot loads.

« Back to merge proposal