Skip to content

fix(docs): enable syntax highlighting for HTML in the Responsive Images guide (#119) #124

Merged
eportis-cloudinary merged 2 commits into
cloudinary-community:mainfrom
mohith2883:fix/responsive-guide-html-highlighting
Oct 3, 2025
Merged

fix(docs): enable syntax highlighting for HTML in the Responsive Images guide (#119) #124
eportis-cloudinary merged 2 commits into
cloudinary-community:mainfrom
mohith2883:fix/responsive-guide-html-highlighting

Conversation

@mohith2883
Copy link
Copy Markdown
Contributor

Description

Fixes #119 by updating HTML code blocks in the Responsive Images guide to use the appropriate language tag. This ensures consistent syntax highlighting for both JSX and HTML examples.

Issue Ticket Number

Fixes #119

Before

<img
  class="astro-cloudinary-cldimage"
  sizes="(max-width: 768px) 100vw, (max-width: 1200px) 50vw, 33vw"
  alt="Coffee"
  loading="lazy"
  decoding="async"
  style="object-fit:cover;max-width:960px;max-height:600px;aspect-ratio:1.6;width:100%"
  srcset="
    https://res.cloudinary.com/<Cloud Name>/image/upload/c_limit,w_640/f_auto/q_auto/v1/samples/coffee?_a=BBGAABXg0 640w,
    https://res.cloudinary.com/<Cloud Name>/image/upload/c_limit,w_750/f_auto/q_auto/v1/samples/coffee?_a=BBGAABXg0 750w,
    https://res.cloudinary.com/<Cloud Name>/image/upload/c_limit,w_828/f_auto/q_auto/v1/samples/coffee?_a=BBGAABXg0 828w,
    https://res.cloudinary.com/<Cloud Name>/image/upload/c_limit,w_960/f_auto/q_auto/v1/samples/coffee?_a=BBGAABXg0 960w,
    https://res.cloudinary.com/<Cloud Name>/image/upload/c_limit,w_960/f_auto/q_auto/v1/samples/coffee?_a=BBGAABXg0 960w,
    https://res.cloudinary.com/<Cloud Name>/image/upload/c_limit,w_1080/f_auto/q_auto/v1/samples/coffee?_a=BBGAABXg0 1080w,
    https://res.cloudinary.com/<Cloud Name>/image/upload/c_limit,w_1280/f_auto/q_auto/v1/samples/coffee?_a=BBGAABXg0 1280w,
    https://res.cloudinary.com/<Cloud Name>/image/upload/c_limit,w_1668/f_auto/q_auto/v1/samples/coffee?_a=BBGAABXg0 1668w,
    https://res.cloudinary.com/<Cloud Name>/image/upload/c_limit,w_1920/f_auto/q_auto/v1/samples/coffee?_a=BBGAABXg0 1920w
  "
  src="https://res.cloudinary.com/<Cloud Name>/image/upload/c_limit,w_960/f_auto/q_auto/v1/samples/coffee?_a=BBGAABXg0"
>

After Fix

<img
  class="astro-cloudinary-cldimage"
  sizes="(max-width: 768px) 100vw, (max-width: 1200px) 50vw, 33vw"
  alt="Coffee"
  loading="lazy"
  decoding="async"
  style="object-fit:cover;max-width:960px;max-height:600px;aspect-ratio:1.6;width:100%"
  srcset="
    https://res.cloudinary.com/<Cloud Name>/image/upload/c_limit,w_640/f_auto/q_auto/v1/samples/coffee?_a=BBGAABXg0 640w,
    https://res.cloudinary.com/<Cloud Name>/image/upload/c_limit,w_750/f_auto/q_auto/v1/samples/coffee?_a=BBGAABXg0 750w,
    https://res.cloudinary.com/<Cloud Name>/image/upload/c_limit,w_828/f_auto/q_auto/v1/samples/coffee?_a=BBGAABXg0 828w,
    https://res.cloudinary.com/<Cloud Name>/image/upload/c_limit,w_960/f_auto/q_auto/v1/samples/coffee?_a=BBGAABXg0 960w,
    https://res.cloudinary.com/<Cloud Name>/image/upload/c_limit,w_960/f_auto/q_auto/v1/samples/coffee?_a=BBGAABXg0 960w,
    https://res.cloudinary.com/<Cloud Name>/image/upload/c_limit,w_1080/f_auto/q_auto/v1/samples/coffee?_a=BBGAABXg0 1080w,
    https://res.cloudinary.com/<Cloud Name>/image/upload/c_limit,w_1280/f_auto/q_auto/v1/samples/coffee?_a=BBGAABXg0 1280w,
    https://res.cloudinary.com/<Cloud Name>/image/upload/c_limit,w_1668/f_auto/q_auto/v1/samples/coffee?_a=BBGAABXg0 1668w,
    https://res.cloudinary.com/<Cloud Name>/image/upload/c_limit,w_1920/f_auto/q_auto/v1/samples/coffee?_a=BBGAABXg0 1920w
  "
  src="https://res.cloudinary.com/<Cloud Name>/image/upload/c_limit,w_960/f_auto/q_auto/v1/samples/coffee?_a=BBGAABXg0"
> 

@vercel
Copy link
Copy Markdown

vercel Bot commented Oct 1, 2025

@mohith2883 is attempting to deploy a commit to the Cloudinary DevX Team on Vercel.

A member of the Team first needs to authorize it.

@devpatocld
Copy link
Copy Markdown

Hi! Somone submitted PR for this issue before you. Please remove the fixes related to highlighting, leave the ones from types and such.

Someone submitted the fixes for the highlighting already
#123

@mohith2883
Copy link
Copy Markdown
Contributor Author

Hey @devpatocld I was waiting for the hacktoberfest and I think I should get a chance as I asked to work on this

Copy link
Copy Markdown
Contributor

@eportis-cloudinary eportis-cloudinary left a comment

Choose a reason for hiding this comment

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

Thanks for catching these typos! Merging.

@eportis-cloudinary eportis-cloudinary merged commit 70fee5f into cloudinary-community:main Oct 3, 2025
3 of 4 checks passed
@devpatocld
Copy link
Copy Markdown

@mohith2883 Hi! Can I get your email to send you the information for claiming your price? Thank you!

@devpatocld
Copy link
Copy Markdown

@mohith2883 Thank you for your contribution to Hacktoberfest! As a token of our thanks, we at Cloudinary have emailed you with instructions on how to claim your bespoke 2025 swag pack

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] HTML on Responsive Images guide in docs is not syntax highlighted

3 participants