Skip to content

feat: set 'fit-content' height config by default#673

Merged
chybisov merged 4 commits intomainfrom
feat-default-fit-content
Mar 17, 2026
Merged

feat: set 'fit-content' height config by default#673
chybisov merged 4 commits intomainfrom
feat-default-fit-content

Conversation

@effie-ms
Copy link
Contributor

@effie-ms effie-ms commented Mar 16, 2026

Which Linear task is linked to this PR?

https://linear.app/lifi-linear/issue/EMB-312/set-fit-content-height-layout-by-default-for-widget-v4-playground

Why was it implemented this way?

This feature sets widget height to 'fit-content' by default - if a widget integrator did not specify it in widget config.
On playground, "Default" layout option will have the styling logic of current "Fit content" layout option. In turn, "Fit content" layout option is removed.

Checklist before requesting a review

  • I have performed a self-review and testing of my code.
  • This pull request is focused and addresses a single problem.

@effie-ms effie-ms assigned effie-ms and unassigned effie-ms Mar 16, 2026
@github-actions
Copy link

Hey! This is your new endpoint: https://ae9f2da6.widget-featdefaul.pages.dev

@effie-ms effie-ms changed the title refactor(widget-playground): set 'fit-content' height configuration b… feat: set 'fit-content' height config by default Mar 17, 2026
@github-actions
Copy link

Hey! This is your new endpoint: https://fda06f2d.widget-featdefaul.pages.dev

@effie-ms effie-ms marked this pull request as ready for review March 17, 2026 13:39
@effie-ms effie-ms requested a review from chybisov March 17, 2026 13:39
@github-actions
Copy link

Hey! This is your new endpoint: https://c7a9c270.widget-featdefaul.pages.dev

1 similar comment
@github-actions
Copy link

Hey! This is your new endpoint: https://c7a9c270.widget-featdefaul.pages.dev

@chybisov chybisov merged commit 9fb7540 into main Mar 17, 2026
2 checks passed
@chybisov chybisov deleted the feat-default-fit-content branch March 17, 2026 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants