Skip to content

Alberto Mancarella#8

Open
Alberto101Ma wants to merge 3 commits intorileyjohngibbs:masterfrom
Alberto101Ma:master
Open

Alberto Mancarella#8
Alberto101Ma wants to merge 3 commits intorileyjohngibbs:masterfrom
Alberto101Ma:master

Conversation

@Alberto101Ma
Copy link

@Alberto101Ma Alberto101Ma commented Mar 7, 2019

Added a counter

<div className="row">
<button className="col" onClick={() => {
<button id="disabled" className="col" onClick={() => {
counter123();
Copy link
Author

Choose a reason for hiding this comment

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

Makes sure to run the counter function before rolling the dices

@Alberto101Ma Alberto101Ma changed the title Update App.js Alberto Mancarella Mar 7, 2019
src/App.js Outdated

import Die from './Die';

function counter123(){
Copy link
Author

Choose a reason for hiding this comment

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

Added a function to count the number of times the roll dice button is pressed!

@rileyjohngibbs rileyjohngibbs added the student-pr Pull request from a student. label Mar 7, 2019

let counter = 0;
function counter123(){
if(counter>11){
Copy link
Author

Choose a reason for hiding this comment

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

Limits the dice roll to 12 times

Choose a reason for hiding this comment

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

Nice Idea bro! BTW we are the same person.

let counter = 0;

if(counter>11){
const buttons = document.getElementById("disabled");

Choose a reason for hiding this comment

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

I approve of this change bro!


let counter = 0;
function counter123(){
if(counter>11){

Choose a reason for hiding this comment

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

Nice Idea bro! BTW we are the same person.

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