Skip to content

Fixed Tailwind Issues #118

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.

Sign up for GitHub

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

Merged
bharathk08 merged 4 commits into from
Apr 14, 2026
Merged

Fixed Tailwind Issues #118

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
9d18c7b
Fixed Tailwind Issues
bharathk08 Apr 10, 2026
2aef1f9
Update gatsby-config.js
bharathk08 Apr 10, 2026
037d50d
Fix CI: upgrade to Node 20 and pin dependencies with package-lock.json
bharathk08 Apr 14, 2026
efcbcec
Fix CI: use --legacy-peer-deps and remove package-lock.json from giti…
bharathk08 Apr 14, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 1 addition & 2 deletions src/components/LeftNav.jsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@
import { useStaticQuery, graphql, Link } from 'gatsby';
import React from 'react';
import './LeftNav.scss';
import book from './images/book.png';


const { v4: uuidv4 } = require('uuid');
Expand Down Expand Up @@ -86,7 +85,7 @@ class ListItem extends React.Component {
return ( url.indexOf("/overview/") === -1 &&
<li key={`${data.leftNavTitle || title}-${uuidv4()}`} className={`child${this.inUrl(url) ? ' currentUrl text_green' : ''}`}>
<div className="activeIndicator" />
<img src={book} alt="book"/>
<img src='https://s3.amazonaws.com/static-docs.testsigma.com/new_images/projects/Updated_Doc_Images/book.png' loading='eager' alt='book' />
Copy link
Copy Markdown

@coderabbitai coderabbitai Bot Apr 10, 2026

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Avoid eager-loading nav icons.

Line 88 sets loading='eager' for each child-row icon; this can over-prioritize non-critical images. Prefer lazy loading here.

Suggested change 
-                <img src='https://s3.amazonaws.com/static-docs.testsigma.com/new_images/projects/Updated_Doc_Images/book.png' loading='eager' alt='book' />
+                <img src='https://s3.amazonaws.com/static-docs.testsigma.com/new_images/projects/Updated_Doc_Images/book.png' loading='lazy' decoding='async' alt='book' />
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
<img src='https://s3.amazonaws.com/static-docs.testsigma.com/new_images/projects/Updated_Doc_Images/book.png' loading='eager' alt='book' />
<img src='https://s3.amazonaws.com/static-docs.testsigma.com/new_images/projects/Updated_Doc_Images/book.png' loading='lazy' decoding='async' alt='book' />
🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@src/components/LeftNav.jsx` at line 88, The nav icon image in LeftNav.jsx is
set to loading='eager', which over-prioritizes non-critical images; update the
<img> in the LeftNav component (the child-row/book icon) to use lazy loading by
removing or replacing loading='eager' with loading='lazy' so icons are deferred
until needed.

<Link to={url}>{data.leftNavTitle || title}</Link>
</li>
);
Expand Down
2 changes: 1 addition & 1 deletion tailwind.config.js
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
module.exports = {
purge: [],
purge: ['./src/**/*.{js,jsx,ts,tsx,md}'],
darkMode: false, // or 'media' or 'class'
theme: {
extend: {
Expand Down
Loading