Code Monkey home page Code Monkey logo

sluchik.github.io's People

Watchers

 avatar  avatar  avatar

sluchik.github.io's Issues

Feedback lesson 4

HTML:

  • язык документа не совпадает с языком основного контенте
  • смешанное название классов (someclass, some-class, some_class, someClass). Выбери какой-то один стиль
  • лишний div
  • неправильный адрес для ссылки на имейл

CSS:

  • фон не совпадает с макетом (смещен по вертикали, странный отступ сверху)
  • меню с флагами лучше добавить очистку float
  • картинка в шапке, её лучше сделать фоном вместо тега img
  • Проблема с разметкой основной контентной части: если добавить текст в средней колонке, то макет ламается
    screen shot 2015-11-11 at 23 47 21
  • нет стилей для текста по-умолчанию

Feedback lesson 6

HTML:

  • Язык документа не соотв. контенту
  • «регистрация», «забыл пароль» должны быть ссылкой, а не кнопкой
  • тег article нужен для обозначения статьи, список товаров — не статья
  • тег p нужен для обозначения абзаца, но не для одного слова
  • болчные элементы не должны быть внутри строчных
  • есть незакрытые теги
  • очевидно же что абзац в абзаце не может быть, <p> <p> </p> </p> — такого тоже не должно быть

CSS:

  • иконки не такие как в макете:
    screen shot 2015-12-08 at 2 12 13
  • Стили боковых блоков отличаются от макета:
    screen shot 2015-12-08 at 2 13 29

Feedback JS lesson 7-8

Зачет

Доп задание

  • Добавить функцию debounce для обработчиков событий hover, так чтобы они срабатывали с задержкой если пользователь многократно проводить курсором над полями ввода

Feedback JS lesson 3-4

В целом хорошо. Понравилось что ты придумал вынести создание DOM-элементов в отдельную функцию. Но не было смысла функции создания и генерации выносить в объект, возможно ты хотел создать класс из которого наследовать новые объекты, но по факту это объект с методами.
Это зачет, но можно улучшить:

  • здесь зарезервированное слово можно обернуть в кавычки, то есть вместо params.forType сделать params['for']. А так получается что часть параметров идентична параметрам DOM-элемента, а часть нет.
  • убрать объект taskJenerator или переписать его на класс.

Feedback JS EXAM

1)http://joxi.ru/nAy8vNqFXM8M82 тут бордер постоянный а не только по ховеру
2) тут http://joxi.ru/eAO1zP7i4j0l3m три одинаковых слайдера как говорил Олег на вебинаре.
3) тут http://joxi.ru/Q2K7zPps9Jnoz2 есть заголовок + нужно сделать чтоб картинки открывались в попапе.
4) это все что я вижу в ИЕ8
image
image
image

  1. структура файлов не совсем правильная. Хороший тон использовать такую как в этой статье
  2. Все внешние скрипты конкатынируються минифицируються и подключаются отдельно например файл vendor.min.js( это не касается скриптов IE), твои скрипты подключаются отдельно
  3. при каждой загрузке страницы в плитке рамдомные картинки пункт 15 в ТЗ

Feedback lesson 7-8-9

нужно добавить кроссбраузерность. так как для зачета нужны все нужно сделать 7 8 и 9 ДЗ/

Feedback JS lesson 2

JS:

  • Непонятно зачем используется преобразование имени юзера в upperCase
  • Оптимальнее было бы прервать выполнение цикла сразу же, когда найдено совпадение искомого имени в списке пользователей
  • Переменная user пусть будет булевой (true/false) и назваться соответственно isUserExist или в таком духе

Dz Rewiev lessons 5

НЕ ЗАЧЕТ

Зачем выносить одну логическую форму в три отдельные?

screenshot 2015-11-13 13 52 43

- [x] сделать все в одной форме; - [x] сделать все кнопки кликабельны и при наведение на них должен быть cursor - poiner; для чего выносить каждый инпут в отдельный див?

screenshot 2015-11-13 14 11 09

- [x] внести все инпуты в один див

Feedback JS lesson 1

JS:

  • функция pow должна возвращать результат только число. Выводить этот результат в консоль или диалоговое окно нужно отдельно
  • Убрать лишнее объявление переменной result

Feedback lesson 3

HTML:

  • Язык документа
  • Названия классов с ошибками, неудобно читать
  • Лишние названия классов (не используются в стилях)

CSS:

  • много лишних вендорных префиксов
  • много лишних свойств, которые повторяются (шрифт, бордер), их можно выделить общие для всех и задать, допустим для body
  • блок с картинкой имеет бордер, но картика внутри не занимает всю ширину родителя, может быть ситуация, когда картинка имеет свой бордер и тогда получится две границы.

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.