Remove the crosshatched background of the canvas #799

Merged
merged 2 commits into from Aug 27, 2015

Conversation

Projects
None yet
3 participants
Member

anthonydillon commented Aug 26, 2015

Replaced with the new #f5f5f5 flat background color.

Member

jujugui commented Aug 26, 2015

please test this

Owner

hatched commented Aug 26, 2015

👍 QA OK
Thanks for doing this on pyramid-fork!

Owner

hatched commented Aug 26, 2015

:shipit:

Other reviews were done on the previous branch targeting develop

Member

jujugui commented Aug 26, 2015

Build failed: Attempt to land pull request failed
build url: http://ci.jujugui.org:8080/job/juju-gui-merge/981

Owner

hatched commented Aug 26, 2015

spurious landing failure...

Member

jujugui commented Aug 26, 2015

Build failed: Attempt to land pull request failed
build url: http://ci.jujugui.org:8080/job/juju-gui-merge/989

Member

jujugui commented Aug 26, 2015

Build failed: Attempt to land pull request failed
build url: http://ci.jujugui.org:8080/job/juju-gui-merge/991

Owner

hatched commented Aug 26, 2015

Now that CI is fixed the unit tests are now failing with this branch, can you please update from pyramid-fork and then take a look at those failing tests. Thanks!

Owner

hatched commented Aug 27, 2015

Member

jujugui commented Aug 27, 2015

Status: merge request accepted. Url: http://ci.jujugui.org:8080/job/juju-gui-merge

jujugui added a commit that referenced this pull request Aug 27, 2015

Merge pull request #799 from anthonydillon/update-canvas-background
Remove the crosshatched background of the canvas

Replaced with the new #f5f5f5 flat background color.

@jujugui jujugui merged commit 00f2fcf into juju:pyramid-fork Aug 27, 2015

@anthonydillon anthonydillon deleted the anthonydillon:update-canvas-background branch Dec 3, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment