Skip to content

Natasha#7

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

Natasha#7
natasharojany wants to merge 2 commits intorileyjohngibbs:masterfrom
natasharojany:master

Conversation

@natasharojany
Copy link

No description provided.

I changed the colors of the buttons and placed them in numerical order in order to make things easier for the user.
@rileyjohngibbs rileyjohngibbs added the student-pr Pull request from a student. label Mar 7, 2019
<div className="App container">
<div id="adders" className="row">
{[4, 6, 8, 10, 100, 12, 20].map((faces, key) => {
{[4, 6, 8, 10, 15,20,30,40,50,60,80,100].map((faces, key) => {

Choose a reason for hiding this comment

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

Woah! You really added a lot of die. Props to you.


#controllers button {
background-color: orange;
background-color: pink;

Choose a reason for hiding this comment

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

This color is cool.

.App-logo {
animation: App-logo-spin infinite 20s linear;
height: 40vmin;
height: 50vmin;

Choose a reason for hiding this comment

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

What part does this affect?

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

Labels

student-pr Pull request from a student.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants