Skip to content

Conversation

@ncarazon
Copy link
Contributor

@ncarazon ncarazon commented Dec 4, 2025

Closes #3845

This PR implements the redesigned embeds, including:

  • New embed layouts for:
    • Binary questions
    • Continuous questions
    • Multiple choice questions
    • Time series
    • Binary groups
    • Continuous groups
  • Card responsiveness that adapts to the question type
  • Title truncation for long question titles
  • Scaling implementation for OG mode
  • Preservation of existing embed parameters/settings used previously

Binary question embeds:

image image image

Text truncation

Screen.Recording.2025-12-16.at.17.16.46.mov

Continuous question embeds:

image image image

Multiple choice question embeds:

image image image

Time series question embeds:

image image image

Binary / continuous group question embeds:

image image image

Preservation of existing embed parameters/settings

image

@ncarazon ncarazon force-pushed the feat/embeds-redesign branch from c929ffb to dbe0441 Compare December 8, 2025 16:38
@ncarazon ncarazon force-pushed the feat/embeds-redesign branch from dbe0441 to 2277a8f Compare December 9, 2025 17:01
@ncarazon ncarazon marked this pull request as ready for review December 16, 2025 15:28
@ncarazon ncarazon force-pushed the feat/embeds-redesign branch from a3d55e2 to e090527 Compare December 16, 2025 15:36
@ncarazon ncarazon requested a review from cemreinanc December 16, 2025 15:57
Copy link
Contributor

@elisescu elisescu left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

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?

Comment on lines +5 to +8
const QuestionTitle = forwardRef<
HTMLHeadingElement,
HTMLAttributes<HTMLHeadingElement>
>(({ children, className, ...props }, ref) => {
Copy link
Contributor

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)

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.

Embeds Redesign

6 participants