forked from mystiq/hydrogen-web
Don't hardcode theme/variant names
This commit is contained in:
parent
14523ecc5d
commit
825c9847fe
1 changed files with 11 additions and 8 deletions
|
@ -131,24 +131,27 @@ module.exports = function buildThemes(options) {
|
||||||
if (id.startsWith(resolvedVirtualModuleId)) {
|
if (id.startsWith(resolvedVirtualModuleId)) {
|
||||||
let [theme, variant, file] = id.substr(resolvedVirtualModuleId.length).split("/");
|
let [theme, variant, file] = id.substr(resolvedVirtualModuleId.length).split("/");
|
||||||
if (theme === "default") {
|
if (theme === "default") {
|
||||||
theme = "element";
|
theme = options.themeConfig.default;
|
||||||
}
|
}
|
||||||
|
const location = options.themeConfig.themes[theme];
|
||||||
|
const manifest = require(`${location}/manifest.json`);
|
||||||
|
const variants = manifest.values.variants;
|
||||||
if (!variant || variant === "default") {
|
if (!variant || variant === "default") {
|
||||||
variant = "light";
|
// choose the first default variant for now
|
||||||
|
// this will need to support light/dark variants as well
|
||||||
|
variant = Object.keys(variants).find(variantName => variants[variantName].default);
|
||||||
}
|
}
|
||||||
if (!file) {
|
if (!file) {
|
||||||
file = "index.js";
|
file = "index.js";
|
||||||
}
|
}
|
||||||
const location = options.themeConfig.themes[theme];
|
|
||||||
const manifest = require(`${location}/manifest.json`);
|
|
||||||
switch (file) {
|
switch (file) {
|
||||||
case "index.js": {
|
case "index.js": {
|
||||||
const isDark = manifest.values.variants[variant].dark;
|
const isDark = variants[variant].dark;
|
||||||
return `import "${path.resolve(`${location}/theme.css`)}${isDark? "?dark=true": ""}";` +
|
return `import "${path.resolve(`${location}/theme.css`)}${isDark? "?dark=true": ""}";` +
|
||||||
`import "@theme/${theme}/${variant}/variables.css"`;
|
`import "@theme/${theme}/${variant}/variables.css"`;
|
||||||
}
|
}
|
||||||
case "variables.css": {
|
case "variables.css": {
|
||||||
const variables = manifest.values.variants[variant].variables;
|
const variables = variants[variant].variables;
|
||||||
const css = getRootSectionWithVariables(variables);
|
const css = getRootSectionWithVariables(variables);
|
||||||
return css;
|
return css;
|
||||||
}
|
}
|
||||||
|
@ -161,7 +164,7 @@ module.exports = function buildThemes(options) {
|
||||||
const [, location, variant] = result;
|
const [, location, variant] = result;
|
||||||
const cssSource = await readCSSSource(location);
|
const cssSource = await readCSSSource(location);
|
||||||
const config = variants[variant];
|
const config = variants[variant];
|
||||||
return await appendVariablesToCSS(config.variables, cssSource);
|
return appendVariablesToCSS(config.variables, cssSource);
|
||||||
}
|
}
|
||||||
return null;
|
return null;
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in a new issue