3pp-mirror/docusaurus
1
0
Fork 0
mirror of https://github.com/facebook/docusaurus.git synced 2025-08-06 02:08:55 +02:00
Code Issues Projects Releases Packages Wiki Activity

chore(eslint): require curly brackets on all blocks (#2239)

Browse source
This commit is contained in:
Alexey Pyltsyn 2020-01-24 08:28:56 +03:00 • committed by Yangshun Tay
parent 42061c6d83
commit 894dbcff49
3 changed files with 7 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

1
.eslintrc.js
View file

@ -51,6 +51,7 @@ module.exports = {
'jsx-a11y/no-noninteractive-element-interactions': WARNING, 'jsx-a11y/no-noninteractive-element-interactions': WARNING,
'no-console': OFF, 'no-console': OFF,
'no-underscore-dangle': OFF, 'no-underscore-dangle': OFF,
curly: [WARNING, 'all'],
'react/jsx-closing-bracket-location': OFF, // Conflicts with Prettier. 'react/jsx-closing-bracket-location': OFF, // Conflicts with Prettier.
'react/jsx-filename-extension': OFF, 'react/jsx-filename-extension': OFF,
'react/jsx-one-expression-per-line': OFF, 'react/jsx-one-expression-per-line': OFF,

4
packages/docusaurus-theme-classic/src/theme/DocItem/index.js
View file

@ -32,7 +32,9 @@ function DocTOC({headings}) {
/* eslint-disable jsx-a11y/control-has-associated-label */ /* eslint-disable jsx-a11y/control-has-associated-label */
function Headings({headings, isChild}) { function Headings({headings, isChild}) {
if (!headings.length) return null; if (!headings.length) {
return null;
}
return ( return (
<ul className={isChild ? '' : 'contents contents__left-border'}> <ul className={isChild ? '' : 'contents contents__left-border'}>
{headings.map(heading => ( {headings.map(heading => (

4
packages/docusaurus/src/client/PendingNavigation.js
View file

@ -65,7 +65,9 @@ class PendingNavigation extends React.Component {
} else { } else {
const id = hash.substring(1); const id = hash.substring(1);
const element = document.getElementById(id); const element = document.getElementById(id);
if (element) element.scrollIntoView(); if (element) {
element.scrollIntoView();
}
} }
}) })
.catch(e => console.warn(e)); .catch(e => console.warn(e));