Code review comment for ~jibel/ubiquity:active_directory

Revision history for this message
Jean-Baptiste Lallement (jibel) wrote :

@mpt, thanks for your comment.

1. The AD checkbox has been moved outside of the GtkBox containing the 2 radio buttons. It follows the same space than other pages. padding of 6px for the alignment that contains it, no margin for the widget. However, I added a margin of 6px between the group of radio buttons and the checkbox to make the separation clearer.

3. Fixed.

4. Fixed.

The gdoc has been updated.

« Back to merge proposal