-
Notifications
You must be signed in to change notification settings - Fork 24
Embeds Redesign #3868
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
base: main
Are you sure you want to change the base?
Embeds Redesign #3868
Conversation
c929ffb to
dbe0441
Compare
dbe0441 to
2277a8f
Compare
a3d55e2 to
e090527
Compare
front_end/src/app/(embed)/questions/components/embed_question_card.tsx
Outdated
Show resolved
Hide resolved
front_end/src/app/(embed)/questions/components/embed_question_footer.tsx
Outdated
Show resolved
Hide resolved
elisescu
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any reason for not having this as an SVG?
| const QuestionTitle = forwardRef< | ||
| HTMLHeadingElement, | ||
| HTMLAttributes<HTMLHeadingElement> | ||
| >(({ children, className, ...props }, ref) => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
in React 19, forwardRef is no longer necessary and you can pass the ref prop directly. (https://react.dev/reference/react/forwardRef)
2b69144 to
6b310f2
Compare
Closes #3845
This PR implements the redesigned embeds, including:
Binary question embeds:
Text truncation
Screen.Recording.2025-12-16.at.17.16.46.mov
Continuous question embeds:
Multiple choice question embeds:
Time series question embeds:
Binary / continuous group question embeds:
Preservation of existing embed parameters/settings