GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Update basic-writing-and-formatting-syntax.md by syed-ateeb-naveed · Pull Request #34693 · github/docs

Conversation

@syed-ateeb-naveed

@syed-ateeb-naveed syed-ateeb-naveed commented Sep 23, 2024 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

In the heading "Specifying the theme an image is shown to", added a Note to clarify the purpose of the tag and added the code itself to display the picture for illustration.

Why:

I was just reading the docs to improve my README, where I came across this code and noticed there was no picture attached for illustration. Furthermore, I also noticed an <img> tag, but its purpose was not given, so I decided to clarify it with NOTE.

Closes:
GitHub Docs Issue #34694

What's being changed (if available, include any code snippets, screenshots, or gifs):

Before:

image

After:

image
Shows a picture of moon and stars, as I have dark mode selected

Check off the following:

  • I have reviewed my changes in staging, available via the View deployment link in this PR's timeline (this link will be available after opening the PR).

    • For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.
  • For content changes, I have completed the self-review checklist.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

Dila333 reacted with rocket emoji
All reactions
  • 🚀 1 reaction
@syed-ateeb-naveed
Update basic-writing-and-formatting-syntax.md
41cf302
In the heading "Specifying the theme an image is shown to", added a Note to clarify the purpose of the <img> tag and added the code itself to display the picture for illustration.
@welcome

welcome Bot commented Sep 23, 2024

Copy link
Copy Markdown

Thanks for opening this pull request! A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines.

Dila333 reacted with rocket emoji
All reactions
  • 🚀 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@github-actions github-actions Bot added the triage Do not begin working on this issue until triaged by the team label Sep 23, 2024
@syed-ateeb-naveed
syed-ateeb-naveed marked this pull request as draft September 23, 2024 07:33
@github-actions

github-actions Bot commented Sep 23, 2024 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

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 that have been changed in this pull request. It's provided to make it easy to review your changes on the staging site. Please note that changes to the data directory will not show up in this table.


Content directory changes

You 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.

Source Preview Production What Changed
get-started/writing-on-github/getting-started-with-writing-and-formatting-on-github/basic-writing-and-formatting-syntax.md fpt
ghec
ghes@ 3.14 3.13 3.12 3.11 3.10
fpt
ghec
ghes@ 3.14 3.13 3.12 3.11 3.10

fpt: Free, Pro, Team
ghec: GitHub Enterprise Cloud
ghes: GitHub Enterprise Server

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@syed-ateeb-naveed
syed-ateeb-naveed marked this pull request as ready for review September 23, 2024 07:57
@syed-ateeb-naveed
Merge branch 'main' into patch-1
4b16b49
@nguyenalex836 nguyenalex836 added content This issue or pull request belongs to the Docs Content team get started Content related to "Getting Started" doc set and removed triage Do not begin working on this issue until triaged by the team labels Sep 23, 2024
@nguyenalex836

nguyenalex836 commented Sep 23, 2024

Copy link
Copy Markdown
Contributor

@syed-ateeb-naveed Thanks so much for opening a PR! I'll get this triaged for review ✨

syed-ateeb-naveed reacted with heart emoji
All reactions
  • ❤️ 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@nguyenalex836 nguyenalex836 linked an issue Sep 23, 2024 that may be closed by this pull request
Enhance Documentation on Theme-Specific Image Display with Illustration and <img> Tag Clarification #34694
Closed
1 task done
@nguyenalex836 nguyenalex836 added the waiting for review Issue/PR is waiting for a writer's review label Sep 23, 2024
@subatoi subatoi added the needs SME This proposal needs review from a subject matter expert label Sep 24, 2024
@github-actions

github-actions Bot commented Sep 24, 2024

Copy link
Copy Markdown
Contributor

Thanks for opening a pull request! We've triaged this issue for technical review by a subject matter expert 👀

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@syed-ateeb-naveed
Merge branch 'main' into patch-1
1554721
@nguyenalex836

nguyenalex836 commented Sep 25, 2024

Copy link
Copy Markdown
Contributor

@syed-ateeb-naveed Thank you for your patience while our SME team reviewed! You are absolutely correct regarding the lack of illustration and the need to more clearly define the purpose of the <img> tag ✨

While we feel your updates to more clearly define the purpose of the <img> tag may be slightly too general, rest assured we will be reviewing internally how best to address the article as a whole as thoughtfully as possible 💛

Thank you so much for raising a flag on these issues, and your willingness to contribute to our docs! 🙇

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@nguyenalex836 nguyenalex836 closed this Sep 25, 2024
@nguyenalex836 nguyenalex836 added SME reviewed An SME has reviewed this issue/PR and removed waiting for review Issue/PR is waiting for a writer's review needs SME This proposal needs review from a subject matter expert labels Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

content This issue or pull request belongs to the Docs Content team get started Content related to "Getting Started" doc set SME reviewed An SME has reviewed this issue/PR

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Enhance Documentation on Theme-Specific Image Display with Illustration and <img> Tag Clarification

Uh oh!

There was an error while loading. Please reload this page.

3 participants

@syed-ateeb-naveed @nguyenalex836 @subatoi
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.