dmitrynjg / project-taxi

0 stars 0 forks source link

По проекту Taxi #1

Open ivklgn opened 4 years ago

ivklgn commented 4 years ago

Добавьте npm-automation-env

ivklgn commented 4 years ago

вынесите в отдельные файлы.

ivklgn commented 4 years ago

html код который вставляете через innerHTML - вынесите в отдельный файл:

        this.innerHTML =
          `<div class ='content-info-item-btns'> <button class='content-info-item-btn content-info-item-btn-order'>Закзать</button>` +
          `<button class='content-info-item-btn content-info-item-btn-cancel'>Отмена</button>` +
          `</div> `;
ivklgn commented 4 years ago

коллекцию drivers вынесите в отдельный файл:

    var drivers = [{
        driverName: 'Игорь',
        driverCar: 'Wolksvagen Polo',
        driverCarColor: 'Красный',
        driverCarNumberplate: '12ru В228ИД',
        driverPhone: '+79872115577'
      },
      {
        driverName: 'Сергей',
        driverCar: 'Hyundai Solaris',
        driverCarColor: 'Серый',
        driverCarNumberplate: '21rus Б109ИС',
        driverPhone: '+79348714555'
      },
      {
        driverName: 'Андрей',
        driverCar: 'Toyota Camry',
        driverCarColor: 'Черный',
        driverCarNumberplate: '12rus С447ИД',
        driverPhone: '+79371177384'
      },
      {
        driverName: 'Константин',
        driverCar: 'Skoda Octavia',
        driverCarColor: 'Белая',
        driverCarNumberplate: '12rus Б991ЭМ',
        driverPhone: '+79841455281'
      },
      {
        driverName: 'Данил',
        driverCar: 'Lada Granta',
        driverCarColor: 'Серая',
        driverCarNumberplate: '12rus П321СТ',
        driverPhone: '+79993315690'
      }
    ];
ivklgn commented 4 years ago

Объявления компонент у customElement должны быть в файлах с компонентами

https://github.com/dmitrynjg/project-taxi/blob/e55a6e6364cf30fa5198bff230547115e4aa615f/src/app.js#L9

ivklgn commented 4 years ago

https://github.com/dmitrynjg/project-taxi/blob/e55a6e6364cf30fa5198bff230547115e4aa615f/src/app.js#L14

ivklgn commented 4 years ago

Типы машин которые есть в системе как и drivers вынесите в отдельный файл

https://github.com/dmitrynjg/project-taxi/blob/e55a6e6364cf30fa5198bff230547115e4aa615f/src/app.js#L80

ivklgn commented 4 years ago

вместо var -> const

https://github.com/dmitrynjg/project-taxi/blob/e55a6e6364cf30fa5198bff230547115e4aa615f/src/app.js#L126

ivklgn commented 4 years ago

не нужно создавать лишние переменные

https://github.com/dmitrynjg/project-taxi/blob/e55a6e6364cf30fa5198bff230547115e4aa615f/src/webcomponents/button.js#L7

просто поместите в if:

this.getAttribute('preview') === 'true'
ivklgn commented 4 years ago

в данном случае:

https://github.com/dmitrynjg/project-taxi/blob/e55a6e6364cf30fa5198bff230547115e4aa615f/src/webcomponents/preview-order.js#L2

лучше просто объявить класс:

class PreviewOrder extends HTMLElement {
...

экспорт модуля лучше делайте в конце класса компонента:

module.exports = PreviewOrder;
ivklgn commented 4 years ago

В консоли хрома куча ошибок и замечаний

ivklgn commented 4 years ago

Папка build должна быть заигнорирована в git