-
Notifications
You must be signed in to change notification settings - Fork 1
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 Edge Introduction pictures #53
Conversation
Hi. Was it just some conversion using oxipng, or are those real updates? If the latter is the case, I would like to suggest to switch to a better style/paradigm of having screenshots in the documentation. https://thisisimportant.net/posts/screenshots-in-documentation/ My proposal is to use the ideas from Simplified Screenshots exclusively, and apply them gradually when reworking the documentation, and when adding new content. wdyt?
Sorry about that "more work" bit. My suggestion does not want to hide that it is more workload on the content authors. However, as you know we are aiming for documentation excellency, we humbly ask content authors to pay that tribute. |
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.
Acknowledging. Please still consider our suggestion about Simplified Screenshots. It might be a dedicated round of updates, and we do not want to block you from merging.
Hi. I see where you're coming from. I added "screenshot overhaul" to my backlog and will consider your suggestion, thank you. Obscuring parts sound like a good idea to me. |
Thanks!
Not only obscuring, but also cropping down to the element of interest. 👍 |
Summary of the changes / Why this is an improvement
:scale: 50%
Checklist