Skip to content
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

Add Remix-IDE link to smart contract languages page #12662

Merged
merged 4 commits into from
Jun 17, 2024

Conversation

Hitisha-G
Copy link
Contributor

Include Remix link in the Important Links section for easy access.

Description

Updating documentation by incorporating a direct link to the Remix IDE, facilitating easy access for reviewing and experimenting with code.

Copy link

netlify bot commented Apr 4, 2024

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit 5a1ada2
🔍 Latest deploy log https://app.netlify.com/sites/ethereumorg/deploys/6670c5f05eec5f0008581270
😎 Deploy Preview https://deploy-preview-12662--ethereumorg.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
7 paths audited
Performance: 43 (🟢 up 5 from production)
Accessibility: 92 (no change from production)
Best Practices: 83 (🔴 down 9 from production)
SEO: 93 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@github-actions github-actions bot added the content 🖋️ This involves copy additions or edits label Apr 4, 2024
@wackerow wackerow changed the title Update index.md Add Remix-IDE link to smart contract languages page Apr 9, 2024
Copy link
Member

@wackerow wackerow left a comment

Choose a reason for hiding this comment

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

@Hitisha-G Worth noting that Remix-IDE also supports Vyper... Few options if we want to list this:

A) Leave it where it is, for Solidity only
B) We could consider a place on the page that could highlight it for both Solidity and Vyper, but only mention it once
C) We could add this to both the links list for Solidity and Vyper

Thoughts?

@minimalsm
Copy link
Contributor

B) We could consider a place on the page that could highlight it for both Solidity and Vyper, but only mention it once

B) seems preferable to me.

@Hitisha-G
Copy link
Contributor Author

@wackerow @minimalsm Thank you for the feedback and discussion.
Since users might start with either Solidity or Vyper, including Remix in both important link sections makes sense to me. However, if we're constrained to only one mention on the page, how about integrating it into the introduction section itself? For example:

"The two most active and maintained languages are:
Solidity
Vyper

Remix IDE provides a comprehensive development environment for creating and testing contracts in both Solidity and Vyper."

Additionally, we could introduce a new section dedicated to IDE, where we highlight the versatility of Remix IDE for both Solidity and Vyper development.

What do you think?

@Hitisha-G Hitisha-G requested a review from wackerow April 22, 2024 18:31
@wackerow
Copy link
Member

wackerow commented May 7, 2024

@Hitisha-G I think we want to keep this page focused on the smart contract languages and not get too much into IDE's... I support you're suggestion to add a mention in the opening section:

Would also recommended adding a link to Remix, but question is which one... we could

  1. direct users to their homepage at https://remix-project.org where they can learn about the product first, or
  2. direct straight to the browser IDE at https://remix.ethereum.org to.

I'd lean towards just directing to the IDE, but open to either. Something like this:

The two most active and maintained languages are:
- Solidity
- Vyper

Remix IDE provides a comprehensive development environment for creating and testing contracts in both Solidity and Vyper. [Try the in-browser Remix IDE](https://remix.ethereum.org) to start coding.

Copy link
Contributor

github-actions bot commented Jun 7, 2024

This issue is stale because it has been open 30 days with no activity.

@github-actions github-actions bot added the Status: Stale This issue is stale because it has been open 30 days with no activity. label Jun 7, 2024
@wackerow
Copy link
Member

Hey @Hitisha-G, circling back.. did you want to make these changes, or would you like a hand finishing this up?

@wackerow wackerow removed the Status: Stale This issue is stale because it has been open 30 days with no activity. label Jun 12, 2024
Copy link
Member

@wackerow wackerow left a comment

Choose a reason for hiding this comment

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

Pushed the sudiscussed changes... thanks @Hitisha-G!

@wackerow wackerow merged commit c772751 into ethereum:dev Jun 17, 2024
2 checks passed
Copy link

gitpoap-bot bot commented Jun 17, 2024

Congrats, your important contribution to this open-source project has earned you a GitPOAP!

Be sure to join the Ethereum.org discord if you are interested in contributing further to the project or have any questions for the team.

GitPOAP: 2024 Ethereum.org Contributor:

GitPOAP: 2024 Ethereum.org Contributor GitPOAP Badge

Head to gitpoap.io & connect your GitHub account to mint!

Learn more about GitPOAPs here.

This was referenced Jun 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content 🖋️ This involves copy additions or edits
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants