Resolved Issue no #934

[brijsiyag]

Fixes #934

I have verified that this pull request:

• has no linting errors (npm run lint)
• is from a uniquely-named feature branch and has been rebased on top of the latest develop branch. (If I was asked to make more changes, I have made sure to rebase onto develop then too)
• is descriptively named and links to an issue number, i.e. Fixes #934

[welcome]

🎉 Thanks for opening this pull request! Please check out our contributing guidelines if you haven't already.

[release-com]

Release Environments

This pull request environment is provided by Release, learn more!
To see the status of the environment click on Environment Status below.

• p5.js-web-editor
  • app-ted88bf-p5-js-web-editor.releaseapp.io
  • preview-ted88bf-p5-js-web-editor.releaseapp.io

🔧Environment Status : https://app.releasehub.com/public/Processing%20Foundation/env-6dc1d1a674

[catarak]

Thanks for working on this issue! I was testing this out, and it was working great. However, after looking at the code, I realized that I didn't agree with the approach—I thought that the code should be removed from the React component and into the console reducer, as redux is responsible for managing data changes.

After making this change locally, I couldn't get the styling quite write—as the times count got really bug, it obscured the console message. I dug around into the console-feed library, and it turns out this functionality is now built in! So yeah, that's the reasoning behind all of these changes I made.

[catarak]

Create Release Environment