Замечания по упр. 3

ivklgn commented 4 years ago

  "scripts": {
    "server": "live-server src",
    "validate-html": "html-validate src",
    "stylelint": "stylelint src/**/*.css",
    "jshint": "jshint src/",
    "watch": "npm-watch",
    "dev-mode": "npm-run-all --parallel server watch",
    "postinstall": "npm i -g live-server html-validate jshint",
    "run-webpack": "webpack --mode development",
    "build": "npm-run-all validate-html stylelint run-webpack"
  },

свалка команд. server - возможно уже неактуален, dev-mode не работает как нужно. возможно стоит выпилить. и кстати в build не фигурирует jshint.

нужно сократить скрипты до самых необходимых:

dev- можно реализовать через webpack-dev-server
build - сборка вебпак с флагом -p (для релиза)
валидации (stylelint, jshint)

при сборке build - есть ошибки валидаторов. данное замечание актуально и для 4го упражнения

ivklgn commented 4 years ago

вынесите WeatherWidget в папку components и все ресурсы положите рядом с ним. Точку входа приложения переименуйте в index.js и импортируйте виджет для рендеринга


import WeatherWidget from './components/WeatherWidget';

function App() {
  return <WeatherWidget />;
}

ReactDOM.render(
  <React.StrictMode>
    <App />
  </React.StrictMode>,
  document.querySelector('#root')
);

ivklgn commented 4 years ago

  <div id="form"></div>

используйте лучше обобщенное имя для точки биндинга приложения - например root или app

tlswp / exercises

Замечания по упр. 3 #4