Skip to content
This repository has been archived by the owner on Jul 7, 2022. It is now read-only.

Add material on pieces #414

Merged
merged 1 commit into from
May 26, 2022
Merged

Conversation

KurohanaJuri
Copy link
Contributor

@KurohanaJuri KurohanaJuri commented May 25, 2022

Add a "shiny" material on pieces to have a better contrast with the board.

(This PR hasn't enough diff-coverage, we cannot test the added line because of #213. We need to force merge this :( )

image

Close #407

This create a shiny effect by setting a metallic factor and
roughness factor. We also set the light estimation to specular.
@KurohanaJuri KurohanaJuri self-assigned this May 25, 2022
@KurohanaJuri KurohanaJuri marked this pull request as ready for review May 25, 2022 19:27
Copy link
Contributor

@alexandrepiveteau alexandrepiveteau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM !

@codeclimate
Copy link

codeclimate bot commented May 25, 2022

Code Climate has analyzed commit 5853121 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 66.6% (80% is the threshold).

This pull request will bring the total coverage in the repository to 96.7% (0.0% change).

View more on Code Climate.

Copy link
Contributor

@matt989253 matt989253 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks nice!

@alexandrepiveteau alexandrepiveteau merged commit 348fcfb into main May 26, 2022
@alexandrepiveteau alexandrepiveteau deleted the feature/cyk/add_material_on_pieces branch May 26, 2022 11:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Increase the contrast of the AR chess pieces
3 participants