Skip to content

Conversation

@YagoMCastro
Copy link
Contributor

No description provided.

Copy link
Member

@leouieda leouieda left a comment

Choose a reason for hiding this comment

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

Great work! Left some corrections and comments. You can merge them directly on GitHub if you want.

Copy link
Member

Choose a reason for hiding this comment

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

Image is waaaaaay too large. Crop it to 16:9 ratio and downscale it to 1280x720 pixels. Save it as a JPG and use compression to reduce the size. Otherwise, anyone opening the page has to download 10Mb of images.

Copy link
Member

Choose a reason for hiding this comment

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

Image is too large. Crop it to 16:9 ratio and downscale it to 1280x720 pixels. Use compression on the jpg if haven't already.

@YagoMCastro
Copy link
Contributor Author

Thanks Leo. I like the corrections and think that we could maybe use this post as a template. I also still have a doubt about when do we use "I", since me and @arthursmacedo are co-authoring this post. The reader would not really know who wrote each section of the text, so it would not know who exactly the text is referring to.

@leouieda
Copy link
Member

The reader would not really know who wrote each section of the text, so it would not know who exactly the text is referring to.

Your names are in the headings of each section.

YagoMCastro and others added 11 commits August 8, 2025 14:01
Co-authored-by: Leonardo Uieda <[email protected]>
Co-authored-by: Leonardo Uieda <[email protected]>
Co-authored-by: Leonardo Uieda <[email protected]>
Co-authored-by: Leonardo Uieda <[email protected]>
Co-authored-by: Leonardo Uieda <[email protected]>
Co-authored-by: Leonardo Uieda <[email protected]>
Co-authored-by: Leonardo Uieda <[email protected]>
Co-authored-by: Leonardo Uieda <[email protected]>
Co-authored-by: Leonardo Uieda <[email protected]>
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.

4 participants