feat(google-docs): add select entries functionality to preview modal[INTEG-3348]#10527
Conversation
| <Checkbox | ||
| isChecked={isChecked} | ||
| onChange={() => onToggleEntry(index)} | ||
| id={`entry-${index}`} | ||
| /> |
There was a problem hiding this comment.
What do you think about making the entire box container clickable to toggle the entry selection?
I think it's nice when checkboxes allow you click the text next to them to toggle the selection. In this case since there's a box container, the entire thing could be a click target for checking/unchecking the checkbox.
There was a problem hiding this comment.
love that idea, will make some changes
There was a problem hiding this comment.
switched to the Card component, it has built-in hover styles
Screen.Recording.2026-02-05.at.8.36.54.AM.mov
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
Adrian Meyer (primeinteger)
left a comment
Adrian Meyer (primeinteger)
left a comment
There was a problem hiding this comment.
Clean code ✨ and nice use of design tokens throughout the styling 🔥
| import { Button, Modal, Paragraph } from '@contentful/f36-components'; | ||
| import { EntryToCreate } from '../../../../../../functions/agents/documentParserAgent/schema'; | ||
| import tokens from '@contentful/f36-tokens'; | ||
| import { PreviewEntryList } from './PreviewEntryList'; |
There was a problem hiding this comment.
Quick note: there is a circular import here (PreviewModal <--> PreviewEntryList via PreviewEntry). It works because TypeScript erases types at runtime.
Consider extracting PreviewEntry into a shared types.ts file and importing it in both components. This would avoid the circular dependency between PreviewModal and PreviewEntryList.
Harika Kondur (harikakondur)
requested a review
from Adrian Meyer (primeinteger)
Adrian Meyer (primeinteger)
left a comment
Adrian Meyer (primeinteger)
left a comment
There was a problem hiding this comment.
🔥 🔥 🔥
2 participants
Purpose
google docs app: added checkboxes in the preview modal to select which entries get created
Approach
since we are not using existing content and always creating new entries, we should give users the opportunity to remove the ones they don’t need.
Testing screenshots
Screen.Recording.2026-02-05.at.8.36.54.AM.mov
blogPostct)Screen.Recording.2026-02-04.at.11.50.43.AM.mov