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

Update layout.md #775

Closed
wants to merge 1 commit into from
Closed

Update layout.md #775

wants to merge 1 commit into from

Conversation

abidmix
Copy link

@abidmix abidmix commented Aug 2, 2023

Match screenshot with code

📚 Context

It is important to match screen shots with related code especially for beginners wanting to make a quick reference to the documentation

🧠 Description of Changes

Updated with link to appropriate screen shot

Revised:

image

Current:

image

💥 Impact

Size:

  • [ X] Small
  • Not small

🌐 References

Contribution License Agreement

By submitting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.

Match screenshot with code
@sfc-gh-dmatthews
Copy link
Contributor

Hi @abidmix! Thanks for the suggestion. Since this update would impact not just one, but a collection of thumbnails, I'm converting it into issue #778. We want to make sure all container-like elements have a consistent look, and more generally that images have consistent dimensions, resolutions, and markings.

We are in the process of reviewing the information architecture on the site and will likely be pushing some significant updates soon. There are some other thumbnails in need of updates as well, so I'll be sure to include this in that review. Thanks!

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.

2 participants