website icon indicating copy to clipboard operation
website copied to clipboard

Consistent use of <Image /> component instead of <img> tags in index.page.tsx

Open vishwansh01 opened this issue 1 month ago • 8 comments

Image

In the pages/index.page.tsx file, there is an inconsistent use of Image and img tags for rendering images. For example:

  • The CopyCopter logo uses the Next.js Image component.
  • The Octue logo and others still use regular img tags.

To maintain consistency, performance optimization (automatic image optimization, lazy loading, etc.), all image tags should be replaced with the Next.js Image component.

Suggested Changes

  • Replace all img tags with the Next.js Image component.
  • Ensure appropriate width, height, and alt attributes are provided.
  • Verify that imports for next/image are properly included.

vishwansh01 avatar Nov 06 '25 10:11 vishwansh01

Hey! @vtushar06 @Utkarsh-123github Can I work on this ?

vishwansh01 avatar Nov 06 '25 11:11 vishwansh01

Hello @vishwansh01, let's wait for @Utkarsh-123github to look at this and let me know if I can help after clearance.

vtushar06 avatar Nov 06 '25 11:11 vtushar06

Thanks, @vtushar06! Sure, I’ll wait for @Utkarsh-123github’s confirmation. Looking forward to contributing once it’s cleared

vishwansh01 avatar Nov 06 '25 11:11 vishwansh01

Hi everyone 👋, Thank you for your enthusiasm and for engaging in this discussion! Just a small request, please avoid tagging multiple times. We do keep an eye on open issues and will get back when possible 🙂. Even if you tag once, I’ll receive a notification and will definitely see it.

Let’s keep the thread focused on updates or relevant technical points. Really appreciate your understanding and patience! 🙏

Utkarsh-123github avatar Nov 06 '25 14:11 Utkarsh-123github

This is a good point, Vishwansh! Can you please check other files as well ?

Utkarsh-123github avatar Nov 06 '25 14:11 Utkarsh-123github

Got it, thanks for the clarification! 🙏 I’ll go through the other files as well to check for similar inconsistent usage of img and Image tags.

vishwansh01 avatar Nov 06 '25 14:11 vishwansh01

I’ve already replaced all the img tags with Image in all .tsx files. Image

The AmbassadorList.tsx component is being used by AmbassadorList.cy.tsx. Do I also need to replace this img tag as well, since it’s being used in Cypress tests?

vishwansh01 avatar Nov 07 '25 14:11 vishwansh01

Changed files: pages/index.page.tsx pages/overview/pro-help/index.page.tsx components/ui/AmbassadorsList.tsx Will be creating a PR soon

vishwansh01 avatar Nov 08 '25 17:11 vishwansh01

Hi, I would like to work on this issue to replace all the regular tags with the Next.js <Image /> component for consistent usage across the index.page.tsx file.I will also ensure to properly include width, height, and alt attributes and verify the necessary imports from next/image are present. Thanks for consideration

unnati-jaiswal24 avatar Dec 01 '25 21:12 unnati-jaiswal24