-
Notifications
You must be signed in to change notification settings - Fork 305
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(content-sidebar): Add placehoder metadatasidebar redesigned #3570
feat(content-sidebar): Add placehoder metadatasidebar redesigned #3570
Conversation
3d52c5e
to
05df107
Compare
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.
Do we have a feature branch we want to use? Or do we intend to merge to master until we're blocked by internal dependencies?
We don't have yet. I was hoping we can get this one to |
05df107
to
f611ea3
Compare
0772506
to
489d4dc
Compare
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, lets spawn a feature branch after merging as mentioned
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
489d4dc
to
ba48d9b
Compare
@@ -74,6 +79,7 @@ const MARK_NAME_JS_LOADING_DETAILS = `${ORIGIN_DETAILS_SIDEBAR}${BASE_EVENT_NAME | |||
const MARK_NAME_JS_LOADING_ACTIVITY = `${ORIGIN_ACTIVITY_SIDEBAR}${BASE_EVENT_NAME}`; | |||
const MARK_NAME_JS_LOADING_SKILLS = `${ORIGIN_SKILLS_SIDEBAR}${BASE_EVENT_NAME}`; | |||
const MARK_NAME_JS_LOADING_METADATA = `${ORIGIN_METADATA_SIDEBAR}${BASE_EVENT_NAME}`; | |||
const MARK_NAME_JS_LOADING_METADATA_REDESIGNED = `${ORIGIN_METADATA_SIDEBAR_REDESIGN}${BASE_EVENT_NAME}`; |
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.
ORIGIN_METADATA_SIDEBAR_REDESIGN
no longer exists - was previously the same as ORIGIN_METADATA_SIDEBAR
const MARK_NAME_JS_LOADING_METADATA_REDESIGNED = `${ORIGIN_METADATA_SIDEBAR_REDESIGN}${BASE_EVENT_NAME}`; | |
const MARK_NAME_JS_LOADING_METADATA_REDESIGNED = `${ORIGIN_METADATA_SIDEBAR}${BASE_EVENT_NAME}`; |
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.
After further consideration I decided to leave ORIGIN_METADATA_SIDEBAR_REDESIGN
and apply it properly, so we can track the loading performance. I'm sure @jstoffan knows how.
79e2a5b
to
36f8857
Compare
In this new placeholder component we will be composing new experience from new components from shared-feature
Also use constant in mock to show they connected
36f8857
to
8e3e443
Compare
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
…#3570) * feat(content-sidebar): Introduce redesigned MetadataSidebar placeholder In this new placeholder component we will be composing new experience from new components from shared-feature * feat(content-sidebar): Use isFeatureEnabled utility * feat(content-sidebar): Remove lint disabling comments * feat(content-sidebar): Rename component to MetadataSidebarRedesign * feat(content-sidebar): Rename constant value to match it's name Also use constant in mock to show they connected * feat(content-sidebar): Rename class name to follow SUIT naming * feat(content-sidebar): Remove unneeded ref and constant * feat(content-sidebar): Remove unneeded constant * feat(content-sidebar): Add performance mark for MetadataSidebarRedesign
This PR introduces new placeholder component for work on redesigning Metadata Sidebar.
At current state this placeholder component doesn't offer much functionality nor look.
The idea is to have this component to align and parallel work on upgrading MetadataSidebar to use Blueprint-web components and extends it's functionality with AI.
The component is being lazy loaded to not add to main bundle, and is hidden behind additional feature flag.