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

fix typo in links-and-images.md in html-foundations #29310

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

csm-18
Copy link

@csm-18 csm-18 commented Jan 12, 2025

change 'tags' to 'attributes' when referring to weight and height attributes of img tag

This PR

  • Changed 'tags' to 'attributes' in the rounded places.

commit

@github-actions github-actions bot added the Content: Foundations Involves the Foundations content label Jan 12, 2025
Copy link
Contributor

@JoshDevHub JoshDevHub left a comment

Choose a reason for hiding this comment

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

I like this change, but your PR has a bunch of other formatting changes that seem to have been mistakenly added in. You'll have to undo all of these before I can approve.

To prevent this from happening in the future, make sure you disable any auto-formatting your editor does while working on the Markdown files in our curriculum.

@csm-18
Copy link
Author

csm-18 commented Jan 15, 2025

I fixed it!

It was caused by Prettier settings

Copy link
Contributor

@JoshDevHub JoshDevHub left a comment

Choose a reason for hiding this comment

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

There are still a lot of unrelated changes. If you click the Files changed tab at the top of this PR, you can see just how many formatting changes remain. Any change that isn't due to the typo needs to be taken out. Additionally, there's an accidental revert commit you have in here that we definitely don't want to merge in.

At this point I think it would be simplest to reset all of your changes and then add/commit the typo fix again. If you're not comfortable enough with git to do that, let me know, and I can provide some commands to run to help out.

height and width are attributes of img element (not tags on img element)
@csm-18 csm-18 force-pushed the html-links-and-images-edit branch from 72d30ea to 6d60420 Compare January 17, 2025 06:10
@csm-18
Copy link
Author

csm-18 commented Jan 17, 2025

Sorry for the mess!
I am a beginner (just passed out from tutorial hell university)
I hope now I did the right thing!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content: Foundations Involves the Foundations content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants