Skip to content

Update assets install library IDE 2 article #410

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

Conversation

ArduinoBot
Copy link
Collaborator

What This PR Changes

  • Update assets install library IDE 2 article

Contribution Guidelines

@ArduinoBot ArduinoBot added the arduino Bugs and fixes added by the Arduino Team label Jul 25, 2022
@gatsby-cloud
Copy link

gatsby-cloud bot commented Jul 25, 2022

Gatsby Cloud Build Report

docs-content

🎉 Your build was successful! See the Deploy preview here.

Build Details

View the build logs here.

🕐 Build time: 4m

Copy link
Contributor

@per1234 per1234 left a comment

Choose a reason for hiding this comment

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

General

The readability of the screenshots has become significantly worse due to the IDE window having been sized unnecessarily large when taking them. This makes the text and interface difficult to see in the final image.

Please make your IDE window smaller and then redo the screenshots so the scale in the final image is larger. The scale in the previous screenshots was acceptable.

content/software/ide-v2/tutorials/ide-v2-installing-a-library/assets/installing-a-library-img03.png

I think the previous approach of showing the in progress state of the installation was better.

The IDE does this via a notification like this:

image

content/software/ide-v2/tutorials/ide-v2-installing-a-library/assets/installing-a-library-img06.png

I recommend uninstalling some libraries so the File > Examples menu is not quite so huge.

I also think the dark OS theme harms readability here, but that is fairly subjective and I am biased because I'm not a fan of dark themes in general.

@jacobhylen jacobhylen merged commit f68b6f6 into main Jul 27, 2022
@jacobhylen jacobhylen deleted the sync/jacobhylen/update-install-libraries-article-ide2 branch July 27, 2022 13:22
Serjlee pushed a commit that referenced this pull request Sep 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
arduino Bugs and fixes added by the Arduino Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants