v1.0

Author: qianyu liu

.editorconfig

@@ -0,0 +1,16 @@
+root = true
+
+[*]
+indent_style = space
+end_of_line = lf
+charset = utf-8
+trim_trailing_whitespace = true
+insert_final_newline = true
+indent_size = 2
+
+[package.json]
+indent_style = space
+indent_size = 2
+
+[*.md]
+trim_trailing_whitespace = false

.eslintignore

@@ -0,0 +1,7 @@
+node_modules/*
+static/*
+turbo/*
+lib/echarts/*
+client/vendor/*
+server/datasource/thrift/*
+*.json

.eslintrc

@@ -0,0 +1,117 @@
+{
+    "parser": "babel-eslint",
+    "env": {
+      "browser": true,
+      "es6": true,
+      "node": true
+    },
+    "ecmaFeatures": {
+      "modules": true
+    },
+    "rules": {
+      "brace-style": [1, "1tbs", {"allowSingleLine": true}],
+      "camelcase": 1,
+      //"comma-dangle": [1, "never"],
+      "comma-spacing": [0, {"before": false, "after": true}],
+      "comma-style": [1, "last"],
+      "consistent-this": [1, "self"],
+      "curly": [2, "multi-line"],
+      "eqeqeq": [2, "allow-null"],
+      "eol-last": 1,
+      "guard-for-in": 0,
+      "indent": [1, 2],
+      "key-spacing": 0,
+      "new-cap": [1, {"capIsNew": false, "newIsCap": true}],
+      "new-parens": 0,
+      "no-bitwise": 0,
+      "no-cond-assign": 1,
+      "no-console": 0,
+      "no-constant-condition": 1,
+      "no-delete-var": 2,
+      "no-debugger": 1,
+      "no-dupe-keys": 1,
+      "no-duplicate-case": 1,
+      "no-empty-character-class": 1,
+      "no-eval": 1,
+      "no-ex-assign": 1,
+      "no-extend-native": 1,
+      "no-floating-decimal": 1,
+      "no-func-assign": 1,
+      "no-implied-eval": 1,
+      "no-inner-declarations": 1,
+      "no-invalid-regexp": 1,
+      "no-irregular-whitespace": 1,
+      "no-iterator": 1,
+      "no-labels": 1,
+      "no-lonely-if": 1,
+      "no-loop-func": 1,
+      "no-mixed-spaces-and-tabs": 1,
+      "no-native-reassign": 2,
+      "no-negated-in-lhs": 1,
+      "no-new-func": 1,
+      "no-new-object": 1,
+      "no-new-wrappers": 2,
+      "no-obj-calls": 1,
+      "no-octal": 1,
+      "no-plusplus": 0,
+      "no-proto": 2,
+      "no-redeclare": 1,
+      "no-return-assign": 1,
+      "no-self-compare": 1,
+      "no-sequences": 1,
+      "no-script-url": 0,
+      "no-shadow": 0,
+      "no-spaced-func": 1,
+      "no-sparse-arrays": 1,
+      "no-throw-literal": 1,
+      "no-trailing-spaces": 1,
+      "no-unused-vars": [1, {"vars": "all", "args": "none"}],
+      "no-undef": 1,
+      "no-undefined": 2,
+      "no-undef-init": 1,
+      "no-underscore-dangle": 0,
+      "no-unreachable": 1,
+      "no-use-before-define": [1, "nofunc"],
+      "no-warning-comments": [0, {"terms": ["todo", "fixme"]}],
+      //"no-extra-parens": 1,
+      "quotes": [1, "single"],
+      "quote-props": [0, "as-needed"],
+      "radix": 1,
+      "semi": 0,
+      //"keyword-spacing": [1, "always"],
+      "space-before-blocks": [1, "always"],
+      "space-infix-ops": 1,
+      //"space-in-brackets": ["error", "always"],
+      "space-in-parens": [1, "never"],
+      //"space-return-throw-case": 1,
+      "space-unary-ops": [1, {"words": true, "nonwords": false}],
+      "use-isnan": 1,
+      "valid-typeof": 1,
+      "vars-on-top": 0,  // disabled for now, see github.com/eslint/eslint/issues/2099
+      "wrap-iife": [1, "inside"],
+      "yoda": 0,
+      // Node.js specific
+      "handle-callback-err": [1, "^(err|error)$"],
+      "no-mixed-requires": 1,
+      "no-new-require": 1,
+      "no-path-concat": 1,
+      // React
+      "react/jsx-boolean-value": 1,
+      "react/jsx-no-undef": 1,
+      //"react/jsx-quotes": 1,
+      "react/jsx-sort-props": 0,
+      "react/jsx-uses-react": 1,
+      "react/jsx-uses-vars": 1,
+      //"react/no-did-mount-set-state": 1,
+      "react/no-did-update-set-state": 1,
+      "react/no-multi-comp": 0,
+      "react/no-unknown-property": 1,
+      //"react/prop-types": 1,
+      "react/react-in-jsx-scope": 1,
+      //"react/self-closing-comp": 1,
+      "react/jsx-wrap-multilines": 1
+    },
+    "plugins": [
+      "react"
+    ]
+  }

.gitignore

@@ -0,0 +1,21 @@
+# See https://help.github.com/ignore-files/ for more about ignoring files.
+
+# dependencies
+/node_modules
+
+# testing
+/coverage
+
+# production
+/build
+
+# misc
+.DS_Store
+.env.local
+.env.development.local
+.env.test.local
+.env.production.local
+
+npm-debug.log*
+yarn-debug.log*
+yarn-error.log*

README.md

@@ -0,0 +1,28 @@
+### 记录
+- npm全局安装的模块放在 /usr/local/lib/node_modules/ 目录下
+- 不要直接对state进行修改，例如数组的一些方法会直接修改原数组
+- react-router4.*版本使用browerHistory的方法：[https://github.com/brickspert/blog/issues/3](https://github.com/brickspert/blog/issues/3)
+- 安装history[https://github.com/ReactTraining/history](https://github.com/ReactTraining/history)
+```
+  const history = createHistory({
+  forceRefresh: true,
+})
+```
+这样可以方便使用history，但页面跳转时是自动刷新的；
+- 事件函数的参数传递onClick={this.handleClick.bind(this,传递的参数)}
+
+### 问题
+- 划分组件时，由于各个组件之前是有联系的，即 可能相互传数据，如果把其纳入一个组件会减少组件间通信，但是又会让组件显得在UI上划分不合理。如何做一个平衡？
+
+### 项目逻辑
+1. 组件划分
+2. 状态state考虑(涉及到页面数据展示和页面变化)  
+1）页面展示的数据需要state来维护，即todo数组，数组每个对象会维护多个属性；   
+2）页面task，completed，active图标和相关todolist的展示，可以用一个状态来维护，即classify，这样可以同时控制图标的展示和相应todolist的展示；      
+3) 状态isAdd，表示是否新增todo，控制css类名变化，即控制页面的显示；  
+
+### 踩坑
+1. 描述：history.push之后页面路径会变化，但没有跳转。所以加了forceRefresh:true的初始化参数，强制每次路径变化刷新页面。这样页面就才会有刷新。   
+解决方法：主要我之前采用的import { BrowserRouter as Router } 加 ```<Router></Router>```的方式，现在采用```<Router history={history}></Router>```这种方式，就可以默认跳转了。
+
+