Comments (3)
Definitely
from react.
Hey there @Fun117 π
Thanks for opening this issue! To help us fix the problem you're experiencing, could you please do two things for me:
- Create a minimal, reproducible example demonstrating this bug. I'd start by removing irrelevant parts of the app one at a time, checking that the error still exists. Read the article I just linked to for more information.
- Send a link to a new sandbox with this minimal, reproducible example.
Thanks! β€οΈ
from react.
Hi @pwbriggs π
Thank you for your prompt response!
I've managed to resolve the error by making some changes to the code as suggested. Here's what I did:
Code Changes:
CommentsHTML Function Parameters: Changed the parameters of the CommentsHTML
function to accept an object instead of individual arguments. This enhances readability and maintainability.
Old Code:
export function CommentsHTML(CommentsData: Comments[], username: string, userId: string, userImage: string): JSX.Element[] {
New Code:
export function CommentsHTML({ CommentsData, username, userId, userImage }: { CommentsData: Comments[], username: string, userId: string, userImage: string }): JSX.Element[] {
Refactored CommentsHtmlContents Function: Simplified the CommentsHtmlContents
function by extracting the rendering logic into a nested Group
component. This improves code readability and organization.
Old Code:
export function CommentsHtmlContents({ commentsRes, comments, userData }: { commentsRes: boolean, comments: any, userData: any }) {
// Omitted
// CommentsHTML γδΈζηγ«δΏε
let commentsContent = null;
if (commentsRes) {
if (comments.length > 0) {
commentsContent = CommentsHTML(comments, username, userId, userImage);
} else {
commentsContent = (
<li className='animated-slideIn-up p-2 animate-fade-up animate-once animate-duration-500 animate-delay-0 animate-ease-in-out animate-normal animate-fill-forwards'>
// Omitted
</li>
);
}
} else {
commentsContent = (
<li className="relative flex flex-col md:flex-row flex-wrap items-center justify-end w-full">
// Omitted
</li>
);
}
return (
<>
<section className="md:p-6 pt-0">
// Omitted
</section>
</>
)
}
New Code:
export function CommentsHtmlContents({ commentsRes, comments, userData }: { commentsRes: boolean, comments: any, userData: any }) {
// Omitted
function Group({ children }: { children: React.ReactNode }) {
return (
<section className="md:p-6 pt-0">
<ul className="flex flex-col justify-center items-center gap-1 w-full mt-10 *:flex *:flex-row *:flex-wrap *:justify-end *:items-center *:w-full *:p-1">
{children}
</ul>
</section>
)
}
// CommentsHTML γδΈζηγ«δΏε
let commentsContent = null;
if (commentsRes) {
if (comments.length > 0) {
return (
<Group>
<CommentsHTML CommentsData={comments} username={username} userId={userId} userImage={userImage}/>
</Group>
)
} else {
<Group>
// Omitted
</Group>
}
} else {
<Group>
// Omitted
</Group>
}
}
Updated Code Sandbox:
I've updated the code in the provided CodeSandbox to reflect these changes. You can view the updated code here.
Note
While I was able to resolve the issue when reducing the code to a minimal reproduction, the specific reason for the error remains unclear.
Thank you for your assistance! β€οΈ
from react.
Related Issues (20)
- [DevTools Bug]: After updating to version 5.1 the Dev tools fails to inspect elements
- Bug: HOT 1
- [React 19] Async transitions race condition handling
- Bug: React 18.3 warning says to import `act` from `react`, only `unstable_act` exists HOT 3
- Bug: "React Developer Tools" Chrome Extension unable to profile page rendering
- Bug: Source not displayed in DevTools HOT 2
- Bug:
- [React 19] useTransition()'s pending state does not go back to false (revision 94eed63c49-20240425) HOT 3
- Changelog for 18.3 is missing HOT 1
- [React 19] Removal of `ReactDOM.findDOMNode` HOT 2
- [React 19] react-test-renderer deprecation HOT 2
- E-Commerce HOT 1
- [React 19] HOT 1
- [React 19] Eslint React JSDoc support HOT 1
- [React 19] Support scoped custom element registries (i.e, react with Custom Elements being rendered in a shadow root)
- Bug [React-DOM]: Missing "bun" export field in package.json HOT 3
- Bug: Spurious warnings in react dev runtime about using "key" property in spread object
- [React 19] React warns about "fetchpriority" props HOT 10
- Bug: Uncaught TypeError: Cannot read properties of null (reading 'useMemo') [React 18.3.1] HOT 6
- Bug: Warning: React does not recognize the `fetchPriority` prop on a DOM element [React 18.3.1] HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
π Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. πππ
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google β€οΈ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from react.