Добавлена возможность задавать сss-prefix для MF модулей #273
Conversation
🦋 Changeset detectedLatest commit: 2a23b33 The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
|
||
| if (cssPrefix) { | ||
| // Добавляем префикс для css-классов, чтобы изолировать стили модуля от стилей основного приложения | ||
| const cssRule = findLoader(webpackConf, '/\\.css$/'); |
There was a problem hiding this comment.
Ниже есть такой же блок кода, можно в отдельную функцию вынести, чтобы не дублировалось
| @@ -22,6 +22,21 @@ export function patchMainWebpackConfigForModules(webpackConf: webpack.Configurat | |||
| return webpackConf; | |||
| } | |||
|
|
|||
| const moduleOptions = configs.modules.options || {}; | |||
|
|
|||
| Object.keys(moduleOptions).forEach((moduleName) => { | |||
There was a problem hiding this comment.
Так как для MF модулей у нас создается один вебпак конфиг на все модули, то все css-префиксы попадут в один общий конфиг. Поэтому не совсем понятно, к чему это приведет, но в лучшем случае, как-будто в конечном билде все префиксы добавятся для всех модулей разом
There was a problem hiding this comment.
Как-будто не имеет смысла добавлять возможность настраивать отдельный cssPrefix для каждого модуля, потому что они в конечном итоге все добавятся в конечный webpack конфиг
No description provided.