Hey, Following our discussion in the IRC, when pushing UI changes please upload a screenshot of the result and add a link in the commit message. This will allow us to better understand the change and will also allow non-UI developers to comment on the change. Having the screenshot link in the commit message will allow the developer to update the screenshot if there are visible changes as a result of the reviews.
If the UI change is very minor or it is only infra and there are no visible changes - use "Screenshot: N/A". You can see an example at the recent "Task details overview screen" push [1] [1] https://review.openstack.org/#/c/212489/
__________________________________________________________________________ OpenStack Development Mailing List (not for usage questions) Unsubscribe: [email protected]?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
