-
Notifications
You must be signed in to change notification settings - Fork 916
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
[Workspace] Support workspace initial page #7708
Conversation
Signed-off-by: yubonluo <[email protected]>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #7708 +/- ##
==========================================
+ Coverage 63.81% 63.83% +0.02%
==========================================
Files 3651 3652 +1
Lines 80954 81021 +67
Branches 12893 12900 +7
==========================================
+ Hits 51662 51721 +59
- Misses 26126 26132 +6
- Partials 3166 3168 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
src/plugins/workspace/public/components/workspace_initial/workspace_initial.tsx
Outdated
Show resolved
Hide resolved
src/plugins/workspace/public/components/workspace_initial/workspace_initial.test.tsx
Outdated
Show resolved
Hide resolved
@yubonluo Could you please give a screenshot on how the page will look like in dark mode? |
src/plugins/workspace/public/components/workspace_initial/workspace_initial.tsx
Outdated
Show resolved
Hide resolved
src/plugins/workspace/public/components/workspace_initial/workspace_initial.tsx
Outdated
Show resolved
Hide resolved
src/plugins/workspace/public/components/workspace_initial/workspace_initial.tsx
Outdated
Show resolved
Hide resolved
|
Signed-off-by: yubonluo <[email protected]>
…shboards into 2.17/initial-page
Overall LGTM but the dark mode seems a little bit weird, maybe we need a banner for dark mode. |
Signed-off-by: yubonluo <[email protected]>
Signed-off-by: yubonluo <[email protected]>
Signed-off-by: yubonluo <[email protected]>
|
||
export const WorkspaceInitialApp = () => { | ||
return ( | ||
<I18nProvider> |
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.
Nit: maybe could move I18nProvider
to renderInitialApp
* Support workspace initial page Signed-off-by: yubonluo <[email protected]> * Changeset file for PR #7708 created/updated * optimzie the code Signed-off-by: yubonluo <[email protected]> * fix the test error Signed-off-by: yubonluo <[email protected]> * add dark mode svg url Signed-off-by: yubonluo <[email protected]> --------- Signed-off-by: yubonluo <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com> (cherry picked from commit 9aeebec) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
* Support workspace initial page Signed-off-by: yubonluo <[email protected]> * Changeset file for PR opensearch-project#7708 created/updated * optimzie the code Signed-off-by: yubonluo <[email protected]> * fix the test error Signed-off-by: yubonluo <[email protected]> * add dark mode svg url Signed-off-by: yubonluo <[email protected]> --------- Signed-off-by: yubonluo <[email protected]> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
* Support workspace initial page * Changeset file for PR #7708 created/updated * optimzie the code * fix the test error * add dark mode svg url --------- (cherry picked from commit 9aeebec) Signed-off-by: yubonluo <[email protected]> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: opensearch-changeset-bot[bot] <154024398+opensearch-changeset-bot[bot]@users.noreply.github.com>
Description
Support workspace initial page.
Issues Resolved
#7707
Screenshot
2024-08-14.14.30.46.mp4
Testing the changes
yarn start --no-base-path
Changelog
Check List
yarn test:jest
yarn test:jest_integration