-
Notifications
You must be signed in to change notification settings - Fork 62.2k
Make images more useful in GitHub Pages Quickstart #18280
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
Conversation
Thanks for submitting a PR to the GitHub Docs project! In order to review and merge PRs most efficiently, we require that all PRs grant maintainer edit access before we review them. For information on how to do this, see the documentation. |
Automatically generated comment ℹ️This comment is automatically generated and will be overwritten every time changes are committed to this branch. The table contains an overview of files in the Content directory changesYou may find it useful to copy this table into the pull request summary. There you can edit it to share links to important articles or changes and to give a high-level overview of how the changes in your pull request support the overall goals of the pull request.
|
Ready for a review !
@thispsj Thanks for opening this PR! I'll get this triaged for review ⚡ |
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.
Looks Good!
I've removed this note, as the information is covered in the product callout at the top of the article.
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.
Hey @thispsj 👋🏻 thanks so much for opening this pull request, we really appreciate your contribution ✨
I've requested a couple of changes, so feel free to re-request my review once those updates have been made. In the meantime, let me know if you have any questions about the changes I've requested.
content/pages/quickstart.md
Outdated
@@ -25,7 +25,8 @@ This guide will lead you through creating a user site at `username.github.io`. | |||
{% data reusables.repositories.create_new %} | |||
1. Enter `username.github.io` as the repository name. Replace `username` with your {% data variables.product.prodname_dotcom %} username. For example, if your username is `octocat`, the repository name should be `octocat.github.io`. | |||
 | |||
{% data reusables.repositories.sidebar-settings %} | |||
2. Under your repository name, click **{% octicon "gear" aria-label="The gear icon" %} Settings**. |
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.
I think it would be worth updating this image across the whole docs site, so let's re-add the content of lines 28 and 29 back into the {% data reusables.repositories.sidebar-settings %}
reusable.
content/pages/quickstart.md
Outdated
@@ -25,7 +25,8 @@ This guide will lead you through creating a user site at `username.github.io`. | |||
{% data reusables.repositories.create_new %} | |||
1. Enter `username.github.io` as the repository name. Replace `username` with your {% data variables.product.prodname_dotcom %} username. For example, if your username is `octocat`, the repository name should be `octocat.github.io`. | |||
 | |||
{% data reusables.repositories.sidebar-settings %} | |||
2. Under your repository name, click **{% octicon "gear" aria-label="The gear icon" %} Settings**. | |||
 |
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.
The content of this image definitely makes more sense, and is a valuable update. It looks as though the brightness on your screen may have been turned down when you made the screenshot, and the resulting screenshot could be problematic from an accessibility standpoint. Would you mind creating a new screenshot with the same content, but ensuring that it is as bright as the original screenshot? For reference:
@jules-p I'm working on this and will complete it by tomorrow. |
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.
It looks good so far. Keep up the good work.
@jules-p as per your request I have done the changes and changed the image to have less opacity to increase brightness. |
Hi @thispsj, thanks for making those changes ✨ the image background is still a bit too dark. Feel free to use this one 👇🏻 and then we'll be able to get those changes merged 🎉 |
* Change separation layer from black to white * Make appropriate changes to drop shadow to match github/docs-team style.
@jules-p I made appropriate changes to the image. Please take a look and let me know if its good to go. |
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.
This looks great! Thanks for your patience with me on this, @thispsj 🚀 ✨
Thanks very much for contributing! Your pull request has been merged 🎉 You should see your changes appear on the site in approximately 24 hours. If you're looking for your next contribution, check out our help wanted issues ⚡ |
Why:
Closes #18165
What's being changed:
Check off the following: