Skip to content

ReReddit - #244

Open
rahatbd wants to merge 4 commits into
projectshft:masterfrom
rahatbd:master
Open

ReReddit#244
rahatbd wants to merge 4 commits into
projectshft:masterfrom
rahatbd:master

Conversation

@rahatbd

@rahatbd rahatbd commented Jun 7, 2026

Copy link
Copy Markdown

No description provided.

Comment thread script.js Outdated
Comment on lines +9 to +25
const commentTextLabel = document.createElement('label');
commentTextLabel.htmlFor = `comment-${id}`;
commentTextLabel.classList.add('sr-only');
commentTextLabel.textContent = 'Comment';

const commentText = document.createElement('input');
commentText.id = commentTextLabel.htmlFor;
commentText.name = 'text';
commentText.placeholder = 'Comment Text';
commentText.required = true;

const commentNameLabel = document.createElement('label');
commentNameLabel.htmlFor = `name-${id}`;
commentNameLabel.classList.add('sr-only');
commentNameLabel.textContent = 'Name';

const commentName = document.createElement('input');

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

you can extract all of these and move to individual functions to improve readability

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants