mirror of
https://github.com/element-hq/element-web
synced 2024-11-25 10:45:51 +03:00
20a615396b
With a switch to Only One Webpack™ we need a way to help developers generate the component index without a concurrent watch task. The best way to do this is to have developers import their components, but how do they do that when we support skins? The answer in this commit is to change skinning. Skinning now expects to receive your list of overrides instead of the react-sdk+branded components. For Riot this means we send over *only* the Vector components and not Vector+react-sdk. Components can then be annotated with the `replaceComponent` decorator to have them be skinnable. The decorator must take a string with the dot path of the component because we can't reliably calculate it ourselves, sadly. The decorator does a call to `getComponent` which is where the important part of the branded components not including the react-sdk is important: if the branded app includes the react-sdk then the decorator gets executed before the skin has finished loading, leading to all kinds of fun errors. This is also why the skinner lazily loads the react-sdk components to avoid importing them too early, breaking the app. The decorator will end up receiving null for a component because of the getComponent loop mentioned: the require() call is still in progress when the decorator is called, therefore we can't error out. All usages of getComponent() within the app are safe to not need such an error (the return won't be null, and developers shouldn't use getComponent() after this commit anyways). The AuthPage, being a prominent component, has been converted to demonstrate this working. Changes to riot-web are required to have this work. The reskindex script has also been altered to reflect these skinning changes - it no longer should set the react-sdk as a parent. The eventual end goal is to get rid of `getComponent()` entirely as it'll be easily replaced by imports.
118 lines
4.1 KiB
JavaScript
118 lines
4.1 KiB
JavaScript
const path = require('path');
|
|
|
|
// get the path of the js-sdk so we can extend the config
|
|
// eslint supports loading extended configs by module,
|
|
// but only if they come from a module that starts with eslint-config-
|
|
// So we load the filename directly (and it could be in node_modules/
|
|
// or or ../node_modules/ etc)
|
|
const matrixJsSdkPath = path.dirname(require.resolve('matrix-js-sdk'));
|
|
|
|
module.exports = {
|
|
parser: "babel-eslint",
|
|
extends: [matrixJsSdkPath + "/.eslintrc.js"],
|
|
plugins: [
|
|
"react",
|
|
"react-hooks",
|
|
"flowtype",
|
|
"babel"
|
|
],
|
|
globals: {
|
|
LANGUAGES_FILE: "readonly",
|
|
},
|
|
env: {
|
|
es6: true,
|
|
},
|
|
parserOptions: {
|
|
ecmaFeatures: {
|
|
jsx: true,
|
|
legacyDecorators: true,
|
|
}
|
|
},
|
|
rules: {
|
|
// eslint's built in no-invalid-this rule breaks with class properties
|
|
"no-invalid-this": "off",
|
|
// so we replace it with a version that is class property aware
|
|
"babel/no-invalid-this": "error",
|
|
|
|
// We appear to follow this most of the time, so let's enforce it instead
|
|
// of occasionally following it (or catching it in review)
|
|
"keyword-spacing": "error",
|
|
|
|
/** react **/
|
|
// This just uses the react plugin to help eslint known when
|
|
// variables have been used in JSX
|
|
"react/jsx-uses-vars": "error",
|
|
// Don't mark React as unused if we're using JSX
|
|
"react/jsx-uses-react": "error",
|
|
|
|
// bind or arrow function in props causes performance issues
|
|
// (but we currently use them in some places)
|
|
// It's disabled here, but we should using it sparingly.
|
|
"react/jsx-no-bind": "off",
|
|
"react/jsx-key": ["error"],
|
|
|
|
// Components in JSX should always be defined.
|
|
"react/jsx-no-undef": "error",
|
|
|
|
// Assert no spacing in JSX curly brackets
|
|
// <Element prop={ consideredError} prop={notConsideredError} />
|
|
//
|
|
// https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/jsx-curly-spacing.md
|
|
//
|
|
// Disabled for now - if anything we'd like to *enforce* spacing in JSX
|
|
// curly brackets for legibility, but in practice it's not clear that the
|
|
// consistency particularly improves legibility here. --Matthew
|
|
//
|
|
// "react/jsx-curly-spacing": ["error", {"when": "never", "children": {"when": "always"}}],
|
|
|
|
// Assert spacing before self-closing JSX tags, and no spacing before or
|
|
// after the closing slash, and no spacing after the opening bracket of
|
|
// the opening tag or closing tag.
|
|
//
|
|
// https://github.com/yannickcr/eslint-plugin-react/blob/HEAD/docs/rules/jsx-tag-spacing.md
|
|
"react/jsx-tag-spacing": ["error"],
|
|
|
|
/** flowtype **/
|
|
"flowtype/require-parameter-type": ["warn", {
|
|
"excludeArrowFunctions": true,
|
|
}],
|
|
"flowtype/define-flow-type": "warn",
|
|
"flowtype/require-return-type": ["warn",
|
|
"always",
|
|
{
|
|
"annotateUndefined": "never",
|
|
"excludeArrowFunctions": true,
|
|
}
|
|
],
|
|
"flowtype/space-after-type-colon": ["warn", "always"],
|
|
"flowtype/space-before-type-colon": ["warn", "never"],
|
|
|
|
/*
|
|
* things that are errors in the js-sdk config that the current
|
|
* code does not adhere to, turned down to warn
|
|
*/
|
|
"max-len": ["warn", {
|
|
// apparently people believe the length limit shouldn't apply
|
|
// to JSX.
|
|
ignorePattern: '^\\s*<',
|
|
ignoreComments: true,
|
|
ignoreRegExpLiterals: true,
|
|
code: 120,
|
|
}],
|
|
"valid-jsdoc": ["warn"],
|
|
"new-cap": ["warn"],
|
|
"key-spacing": ["warn"],
|
|
"prefer-const": ["warn"],
|
|
|
|
// crashes currently: https://github.com/eslint/eslint/issues/6274
|
|
"generator-star-spacing": "off",
|
|
|
|
"react-hooks/rules-of-hooks": "error",
|
|
"react-hooks/exhaustive-deps": "warn",
|
|
},
|
|
settings: {
|
|
flowtype: {
|
|
onlyFilesWithFlowAnnotation: true
|
|
},
|
|
},
|
|
};
|