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.

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

Feature idea: Remove size limit from adminCommentContent slot #2104

Open
lriggle-strib opened this Issue Nov 30, 2018 · 0 comments

Comments

Projects
None yet
2 participants
@lriggle-strib
Copy link

lriggle-strib commented Nov 30, 2018

Description

I'll use the plugin I'm working on as an example:

I've added an additional text field to the embed stream for the user to input a name to display with the comment instead of their username. The data is saved on the comment object, not the user, since it is a field required for every comment. Since it is relevant text for the comment, not just data about the comment (like the toxicity level), the value itself should also be evaluated by the moderators. The logical place (to me) to display this is with the comment's body content, meaning that the moderators wouldn't have to go hunting for the information.

This would speed up the moderation process for additional fields like the one I've created.

Here is the slot in question:

<Slot
fill="adminCommentContent"
className={cn(styles.commentContent, 'talk-admin-comment')}
size={1}
defaultComponent={AdminCommentContent}
passthrough={{ ...slotPassthrough, ...formatterSettings }}
/>

I've tested this locally, removing the following line allows both the comment and the second field to display in the same area:

screen shot 2018-11-30 at 9 43 02 am

CSS changes would also be required to make the fields display sequentially (like the screenshot) instead of inline with each other.

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