fix: resolve hydration mismatch in img alt attribute (#2340)

[Sravan1011]

What kind of change does this PR introduce?

Bug fix - Resolves hydration mismatch warnings in React/Next.js

Issue Number:

Closes #2340

Summary

This PR fixes hydration mismatch warnings that occur when server-rendered and client-rendered content differs. The changes include:

• Added missing alt='octue' attribute to the octue sponsor image
• Fixed multi-line alt text on the apideck image (collapsed to single line)
• Fixed incorrect alt on sourcemeta image (was 'dottxt logo', now 'sourcemeta logo')
• Wrapped all 14 raw tags in {isClient && (<>...</>)} guards to prevent server/client theme mismatch, consistent with the existing pattern used by all other components on the page

Does this PR introduce a breaking change?

No, this is a non-breaking bug fix that only improves hydration compatibility.

Checklist

• My changes follow the project's code style
• I have updated relevant documentation
• All tests pass
• This PR is ready for review

[github-actions]

Hi @Sravan1011! Thanks a lot for your contribution!

I noticed that the following required information is missing or incomplete: issue reference, kind of change description

Please update the PR description to include this information. You can find placeholders in the PR template for these items.

Thanks a lot!

[github-actions]

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name	Status	Preview	Last Commit
website	✅ Ready (View Log)	Visit Preview	0fba4e1

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (07eca45) to head (0fba4e1).
⚠️ Report is 39 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff            @@
##              main     #2397   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           30        30           
  Lines          672       672           
  Branches       211       211           
=========================================
  Hits           672       672           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[github-actions]

Hi @Sravan1011! Thanks a lot for your contribution!

I noticed that the following required information is missing or incomplete: kind of change description

Please update the PR description to include this information. You can find placeholders in the PR template for these items.

Thanks a lot!

[github-actions]

Hi @Sravan1011! Thanks a lot for your contribution!

I noticed that the following required information is missing or incomplete: kind of change description

Please update the PR description to include this information. You can find placeholders in the PR template for these items.

Thanks a lot!

[vtushar06]

Hey @Sravan1011, I have verified the fix in the codebase, isClient guards are fully justified since the mismatch is actually on src attributes (logos.X from resolvedTheme), not just alt. your approach is consistent with the existing 19 guards in the same file.
One thing I am not sure about locally I count 9 unguarded tags but your PR says 14 - can you confirm the extra 5 are new sponsor images? Also please fill in the "kind of change" field in the PR description, the body validation check is failing because of it.

[Sravan1011]

Hi! To clarify the count — yes, the extra 5 tags compared to your local 9 are new sponsor images that were added to main after your local clone. Here's the breakdown

N-iX (logos.nix) ef67cf6 – "Add nix as a bronze sponsor"
Oracle (logos.oracle) b99ca15 – "add oracle logo"
Spin the Wheel (logos.spinthewheel) f20afb0 – "feat: add 3 new bronze sponsors"
Litslink (logos.litslink) f20afb0 – "feat: add 3 new bronze sponsors"
Time Now (logos.timenow) f20afb0 – "feat: add 3 new bronze sponsors"
My PR wraps all 14 unguarded tags (your original 9 + these 5 newer ones) with {isClient && (<>...</>)} guards to prevent the server/client theme mismatch

[vtushar06]

Also please fill in the "kind of change" field in the PR description, the body validation check is failing because of it.

Please fix this and PR is good to go then.

[github-actions]

Hi @Sravan1011! Thanks a lot for your contribution!

I noticed that the following required information is missing or incomplete: kind of change description

Please update the PR description to include this information. You can find placeholders in the PR template for these items.

Thanks a lot!

[Sravan1011]

Also please fill in the "kind of change" field in the PR description, the body validation check is failing because of it.

Please fix this and PR is good to go then.

I have update the description of pr, now no test is failing you can go ahead with the merge

[vtushar06]

LGTM🚀

[Sravan1011]

LGTM🚀

so when is it going to get merged?

[github-actions]

Congratulations, @Sravan1011 for your first pull request merge in this repository! 🎉🎉. Thanks for your contribution to JSON Schema!