Skip to content

Added 5 sided die#11

Open
lancedavenport wants to merge 2 commits intorileyjohngibbs:masterfrom
lancedavenport:master
Open

Added 5 sided die#11
lancedavenport wants to merge 2 commits intorileyjohngibbs:masterfrom
lancedavenport:master

Conversation

@lancedavenport
Copy link

Added 5 sided die, changed the color of the die once clicked to be purple.

Lance Davenport added 2 commits March 4, 2019 10:09
Times be changing
Added 5 sided die, changed the color of the die once clicked to purple, reinstalled package-lock.json
@natasharojany
Copy link

cool how does a five sided die even look?

@swerze
Copy link

swerze commented Mar 7, 2019

purple is a nice and nice color

Copy link

@swerze swerze left a comment

Choose a reason for hiding this comment

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

I like these stuffs

.die button {
background-color: lightblue;
background-color: #875bff;
padding: 5px;

Choose a reason for hiding this comment

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

this line is good!

Choose a reason for hiding this comment

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

why did u use numbers here and nor words for the color


#total-display {
font-size: large;
font-size: small;

Choose a reason for hiding this comment

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

good choice!

<div className="App container">
<div id="adders" className="row">
{[4, 6, 8, 10, 100, 12, 20].map((faces, key) => {
{[4, 6, 8, 10, 100, 12, 20, 5].map((faces, key) => {

Choose a reason for hiding this comment

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

how does a 5 side die work

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants