-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[WIP][App Arch] migrate legacy CSS to new platform (core_plugins/visualizations) #59027
Conversation
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.
The new imports looks good to me. Now you can also remove the old imports and files that referenced the old _index.scss
file. Like:
- https://github.com/elastic/kibana/blob/f74b63faa71dc6a517382a991871786d0925bc19/src/legacy/core_plugins/visualizations/public/index.scss
- https://github.com/elastic/kibana/blob/f74b63faa71dc6a517382a991871786d0925bc19/src/legacy/core_plugins/visualizations/index.ts#L28-L30
And then check this reference/import:
https://github.com/elastic/kibana/blob/f74b63faa71dc6a517382a991871786d0925bc19/src/legacy/ui/public/visualize/_index.scss
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 still need the following code:
styleSheetPaths: resolve(__dirname, 'public/index.scss'),
in src/legacy/core_plugins/visualizations/index.ts
@alexwizp No, as mentioned above, the |
Done |
Pinging @elastic/kibana-app-arch (Team:AppArch) |
@elasticmachine merge upstream |
removed |
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.
All the references look ok to me. But I'm asking for a review from @elastic/kibana-app team as they know the visualization file structure better.
@elasticmachine merge upstream |
43255b0
to
f3e317a
Compare
💛 Build succeeded, but was flaky
Test FailuresKibana Pipeline / kibana-oss-agent / Chrome UI Functional Tests.test/functional/apps/dashboard/dashboard_query_bar·js.dashboard app using current data dashboard query bar causes panels to reload when refresh is clickedStandard Out
Stack Trace
History
To update your PR or re-run it, just comment with: |
Blocked by #44121 |
Part of #55493
Summary
Here's a list of app arch legacy items I've found so far which contain some scss (be sure to look around for anything I may have missed):
core_plugins/visualizations
Checklist
Delete any items that are not applicable to this PR.
For maintainers