Skip to content
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

feat(atomic): add atomic-tab-manager component #4196

Merged
merged 25 commits into from
Jul 25, 2024
Merged

feat(atomic): add atomic-tab-manager component #4196

merged 25 commits into from
Jul 25, 2024

Conversation

fpbrault
Copy link
Contributor

This PR adds the atomic-tab-manager and atomic-tab components. It only includes the new components without adding support for controlling visibility of other components, as this will be added in another PR.

image
image
image

https://coveord.atlassian.net/browse/CDX-1580

@fpbrault fpbrault requested a review from a team as a code owner July 17, 2024 15:05
@fpbrault fpbrault requested review from olamothe, y-lakhdar and louis-bompart and removed request for a team July 17, 2024 15:05
@developer-experience-bot
Copy link
Contributor

developer-experience-bot bot commented Jul 17, 2024

Pull Request Report

PR Title

✅ Title follows the conventional commit spec.

Live demo links

Bundle Size

File Old (kb) New (kb) Change (%)
case-assist 245.1 245.1 0
commerce 341.5 341.5 0
search 414 414 0
insight 392.4 392.4 0
product-listing 307.1 307.1 0
product-recommendation 211.4 211.4 0
recommendation 258.2 258.2 0
ssr 405.4 405.4 0

SSR Progress

Use case SSR (#) CSR (#) Progress (%)
search 39 44 89
recommendation 0 4 0
product-recommendation 0 10 0
product-listing 0 13 0
case-assist 0 6 0
insight 0 27 0
commerce 0 15 0
Detailed logs search : buildInteractiveResult
search : buildInteractiveInstantResult
search : buildInteractiveRecentResult
search : buildInteractiveCitation
search : buildGeneratedAnswer
recommendation : missing SSR support
product-recommendation : missing SSR support
product-listing : missing SSR support
case-assist : missing SSR support
insight : missing SSR support
commerce : missing SSR support

Copy link
Member

@olamothe olamothe left a comment

Choose a reason for hiding this comment

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

Much simpler with the manager !

Base automatically changed from CDX-1582 to master July 23, 2024 14:58
Copy link
Collaborator

@louis-bompart louis-bompart left a comment

Choose a reason for hiding this comment

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

LGTM, nice job!

I'm guessing the storybook/playwright are scheduled in future PRs.

@fpbrault fpbrault enabled auto-merge July 25, 2024 14:12
@fpbrault fpbrault added this pull request to the merge queue Jul 25, 2024
Merged via the queue into master with commit 523ab9b Jul 25, 2024
109 of 111 checks passed
@fpbrault fpbrault deleted the CDX-1580 branch July 25, 2024 14:59
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.

4 participants