From e7b250401b1289038666103bc72cfc099a44a493 Mon Sep 17 00:00:00 2001 From: Vijay A Date: Sat, 14 May 2022 09:02:38 -0500 Subject: Added TODOs for tech debt --- src/components/ui/Button.js | 1 + src/components/ui/ImageUpload.tsx | 1 + src/components/ui/InfoBar.js | 1 + src/components/ui/Infobox.js | 1 + src/components/ui/Input.js | 1 + src/components/ui/Link.js | 1 + src/components/ui/Loader.tsx | 1 + src/components/ui/Radio.tsx | 1 + src/components/ui/SearchInput.tsx | 1 + src/components/ui/Select.js | 1 + src/components/ui/ServiceIcon.js | 1 + src/components/ui/Slider.js | 1 + src/components/ui/StatusBarTargetUrl.js | 1 + src/components/ui/Toggle.js | 1 + 14 files changed, 14 insertions(+) (limited to 'src') diff --git a/src/components/ui/Button.js b/src/components/ui/Button.js index a52f1cadb..882b39e69 100644 --- a/src/components/ui/Button.js +++ b/src/components/ui/Button.js @@ -4,6 +4,7 @@ import { observer, inject } from 'mobx-react'; import Loader from 'react-loader'; import classnames from 'classnames'; +// Can this file be merged into the './/button/index.tsx' file? class Button extends Component { static propTypes = { className: PropTypes.string, diff --git a/src/components/ui/ImageUpload.tsx b/src/components/ui/ImageUpload.tsx index b07e211c2..c8a6cf54a 100644 --- a/src/components/ui/ImageUpload.tsx +++ b/src/components/ui/ImageUpload.tsx @@ -15,6 +15,7 @@ type Props = { textUpload: string; }; +// Should this file be converted into the coding style similar to './toggle/index.tsx'? class ImageUpload extends Component { static defaultProps = { multiple: false, diff --git a/src/components/ui/InfoBar.js b/src/components/ui/InfoBar.js index c1a6013c2..e50e270bd 100644 --- a/src/components/ui/InfoBar.js +++ b/src/components/ui/InfoBar.js @@ -16,6 +16,7 @@ const messages = defineMessages({ }, }); +// Should this file be converted into the coding style similar to './toggle/index.tsx'? class InfoBar extends Component { static propTypes = { // eslint-disable-next-line react/forbid-prop-types diff --git a/src/components/ui/Infobox.js b/src/components/ui/Infobox.js index 6634a6808..e05d17742 100644 --- a/src/components/ui/Infobox.js +++ b/src/components/ui/Infobox.js @@ -19,6 +19,7 @@ const messages = defineMessages({ }, }); +// Can this file be merged into the './infobox/index.tsx' file? class Infobox extends Component { static propTypes = { // eslint-disable-next-line react/forbid-prop-types diff --git a/src/components/ui/Input.js b/src/components/ui/Input.js index fa1b44699..77bae8101 100644 --- a/src/components/ui/Input.js +++ b/src/components/ui/Input.js @@ -16,6 +16,7 @@ const messages = defineMessages({ }, }); +// Can this file be merged into the './input/index.tsx' file? class Input extends Component { static propTypes = { field: PropTypes.instanceOf(Field).isRequired, diff --git a/src/components/ui/Link.js b/src/components/ui/Link.js index 565547736..05d511a81 100644 --- a/src/components/ui/Link.js +++ b/src/components/ui/Link.js @@ -8,6 +8,7 @@ import { oneOrManyChildElements } from '../../prop-types'; import { matchRoute } from '../../helpers/routing-helpers'; import { openExternalUrl } from '../../helpers/url-helpers'; +// Should this file be converted into the coding style similar to './toggle/index.tsx'? // TODO: create container component for this component class Link extends Component { onClick(e) { diff --git a/src/components/ui/Loader.tsx b/src/components/ui/Loader.tsx index e889fd054..d4ed0d6bb 100644 --- a/src/components/ui/Loader.tsx +++ b/src/components/ui/Loader.tsx @@ -12,6 +12,7 @@ type Props = { stores: FerdiumStores; }; +// Can this file be merged into the './loader/index.tsx' file? class LoaderComponent extends Component { static defaultProps = { loaded: false, diff --git a/src/components/ui/Radio.tsx b/src/components/ui/Radio.tsx index a033a5fe0..e6cb5d5a5 100644 --- a/src/components/ui/Radio.tsx +++ b/src/components/ui/Radio.tsx @@ -10,6 +10,7 @@ type Props = { showLabel: boolean; }; +// Should this file be converted into the coding style similar to './toggle/index.tsx'? class Radio extends Component { static defaultProps = { focus: false, diff --git a/src/components/ui/SearchInput.tsx b/src/components/ui/SearchInput.tsx index 6a6a1692e..67ca469f0 100644 --- a/src/components/ui/SearchInput.tsx +++ b/src/components/ui/SearchInput.tsx @@ -17,6 +17,7 @@ type Props = { autoFocus: boolean; }; +// Should this file be converted into the coding style similar to './toggle/index.tsx'? class SearchInput extends Component { static defaultProps = { value: '', diff --git a/src/components/ui/Select.js b/src/components/ui/Select.js index 0fc32e573..ca5ec9964 100644 --- a/src/components/ui/Select.js +++ b/src/components/ui/Select.js @@ -4,6 +4,7 @@ import { observer } from 'mobx-react'; import { Field } from 'mobx-react-form'; import classnames from 'classnames'; +// Can this file be merged into the './select/index.tsx' file? class Select extends Component { static propTypes = { field: PropTypes.instanceOf(Field).isRequired, diff --git a/src/components/ui/ServiceIcon.js b/src/components/ui/ServiceIcon.js index d403d107d..b05d791be 100644 --- a/src/components/ui/ServiceIcon.js +++ b/src/components/ui/ServiceIcon.js @@ -24,6 +24,7 @@ const styles = theme => ({ }, }); +// Should this file be converted into the coding style similar to './toggle/index.tsx'? class ServiceIcon extends Component { static propTypes = { classes: PropTypes.object.isRequired, diff --git a/src/components/ui/Slider.js b/src/components/ui/Slider.js index a581e7a65..90f4df1c4 100644 --- a/src/components/ui/Slider.js +++ b/src/components/ui/Slider.js @@ -4,6 +4,7 @@ import { observer } from 'mobx-react'; import classnames from 'classnames'; import { Field } from 'mobx-react-form'; +// Should this file be converted into the coding style similar to './toggle/index.tsx'? class Slider extends Component { static propTypes = { field: PropTypes.instanceOf(Field).isRequired, diff --git a/src/components/ui/StatusBarTargetUrl.js b/src/components/ui/StatusBarTargetUrl.js index d8d329d52..3e0c98c5d 100644 --- a/src/components/ui/StatusBarTargetUrl.js +++ b/src/components/ui/StatusBarTargetUrl.js @@ -5,6 +5,7 @@ import classnames from 'classnames'; import Appear from './effects/Appear'; +// Should this file be converted into the coding style similar to './toggle/index.tsx'? class StatusBarTargetUrl extends Component { static propTypes = { className: PropTypes.string, diff --git a/src/components/ui/Toggle.js b/src/components/ui/Toggle.js index a82be690e..c1d86a7f6 100644 --- a/src/components/ui/Toggle.js +++ b/src/components/ui/Toggle.js @@ -4,6 +4,7 @@ import { observer } from 'mobx-react'; import classnames from 'classnames'; import { Field } from 'mobx-react-form'; +// Can this file be merged into the './toggle/index.tsx' file? class Toggle extends Component { static propTypes = { field: PropTypes.instanceOf(Field).isRequired, -- cgit v1.2.3-54-g00ecf