Comment 10 for bug 1234994

Revision history for this message
Sebastien Bacher (seb128) wrote :

Thanks for the work Łukasz, I did a code review and commented on the merge request, some UI comments:

- the divider lines look weird, we should probably hide those

- the spacing/alignment of the buttons look weird to me. It feels like that the space between the image and the button should be at least the same than between the top divider and the image, and that the left/right borders should be aligned either on the image or on the borders of the screen ...

I'm not a designer though ... mpt, what do you think?