-
-
Notifications
You must be signed in to change notification settings - Fork 481
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
Update assets install library IDE 2 article #410
Conversation
Gatsby Cloud Build Report
docs-content
🎉 Your build was successful! See the Deploy preview here.
Build Details
🕐 Build time: 4m
There was a problem hiding this 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:
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.
What This PR Changes
Contribution Guidelines