Skip to content

added web editor and p5 version info to the about section #2118

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
Feb 27, 2023

Conversation

dewanshDT
Copy link
Collaborator

Fixes #2070

Changes:
I've made changes to the About.jsx to show the version info of the editor and the p5.js, please check the commits to learn more.

p5about

I have verified that this pull request:

  • has no linting errors (npm run lint)
  • has no test errors (npm run test)
  • is from a uniquely-named feature branch and is up to date with the develop branch.
  • is descriptively named and links to an issue number, i.e. Fixes #123

@dewanshDT dewanshDT changed the title About version info added web editor and p5 version info to the about section Feb 17, 2023
import { useTranslation } from 'react-i18next';
import { Link } from 'react-router';
import SquareLogoIcon from '../../../images/p5js-square-logo.svg';
// import PlayIcon from '../../../images/play.svg';
import AsteriskIcon from '../../../images/p5-asterisk.svg';
import packageData from '../../../../package.json';
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is great!

@dewanshDT
Copy link
Collaborator Author

@lindapaiste I updated as you suggested, and It works flawlessly 🔥, take a look

@dewanshDT
Copy link
Collaborator Author

I think this works great please check this out and there's nothing more to work on this PR unless any UI changes are required.

@raclim raclim merged commit 2cd9067 into processing:develop Feb 27, 2023
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.

Display editor and p5.js version numbers in 'about' menu
3 participants