Skip to content
This repository has been archived by the owner on Mar 25, 2021. It is now read-only.

Example of carbon / patternfly view switching #141

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

dlabaj
Copy link
Contributor

@dlabaj dlabaj commented Dec 3, 2020

We plan to ship carbon as the default view layer. However we do need a way to switch between the patternfly view. We should add an example implementation of this to the client.

Fixes issue #127

pmuir and others added 8 commits November 25, 2020 16:50
* move all webpack dependencies and babel to devDependencies
* upgrade all webpack dependencies to latest version
* switch to webpack serve
* switch to css-mimimizer-webpack-plugin from optimize-css-assets-webpack-plugin as suggested by optimize-css-assets-webpack-plugin maintainers

Signed-off-by: Pete Muir <[email protected]>
Signed-off-by: Davide Bizzarri <[email protected]>
* fix storybook build with webpack5

Signed-off-by: Davide Bizzarri <[email protected]>
fix: export the components better
@github-actions
Copy link

github-actions bot commented Dec 3, 2020

PR Report

Bundle Sizes

View bundle sizes for 'client'
Bundle New Size Original Size Increase/Decrease File
447.bundle.js 177.54KB 0KB N/A 447.bundle.js
Strimzi UI JS Bundle 3.89KB 11.08KB -65% main.bundle.js
989.bundle.js 1.73KB 0KB N/A 989.bundle.js
Overall bundle size change: -99.39%
View bundle sizes for 'server'
Bundle New Size Original Size Increase/Decrease File
Strimzi UI Server JS Bundle 17.38KB 26.67KB -35% main.js
Overall bundle size change: -34.82%

Test Coverage

View test coverage
File Lines Statement Functions Branches
Total 100% 100% 100% 100%
client/Bootstrap/Navigation/useRouteConfig/useRouteConfig.hook.ts 100% 100% 100% 100%
client/Contexts/ConfigFeatureFlag/Context.tsx 100% 100% 100% 100%
client/Panels/Home/Home.carbon.tsx 100% 100% 100% 100%
client/Panels/Home/Home.patternfly.tsx 100% 100% 100% 100%
File Lines Statement Functions Branches
Total 100% 100% 100% 100%
server/api/controller.ts 100% 100% 100% 100%
server/api/router.ts 100% 100% 100% 100%
server/client/controller.ts 100% 100% 100% 100%
server/client/router.ts 100% 100% 100% 100%
server/config/controller.ts 100% 100% 100% 100%
server/config/router.ts 100% 100% 100% 100%
server/core/app.ts 100% 100% 100% 100%
server/core/modules.ts 100% 100% 100% 100%
server/log/router.ts 100% 100% 100% 100%
server/mockapi/data.ts 100% 100% 100% 100%
server/mockapi/router.ts 100% 100% 100% 100%

Triggered by commit: a4c0f81

Copy link
Contributor

@nictownsend nictownsend left a comment

Choose a reason for hiding this comment

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

This contains other commits (including the module federation). From what I can see related to the patternfly switching, just one comment re mocks.

@@ -15,6 +15,7 @@ const config = {
moduleNameMapper: {
...pathsToModuleNameMapper(compilerOptions.paths, { prefix: '<rootDir>/' }),
...jestModuleMapper,
"\\.(css|less)$": "<rootDir>/../node_modules/@patternfly/react-styles/__mocks__/styleMock.js"
Copy link
Contributor

Choose a reason for hiding this comment

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

I thought jestModuleMapper brought in fake file support - will it work if you augment /utils/tooling/aliasHelper.js to include less instead of bringing in styleMock?

Base automatically changed from master to main March 25, 2021 11:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants