Как стать автором
Обновить

Комментарии 6

А чем bootstrap вас не устроил?

Спасибо за вопрос. Так как все HTML картинки генерируются у меня в JS, а bootstrap нужно интерпретировать в виде классов, я решил не захламлять свой генерируемый HTML и прописать чистый CSS.
Вижу дизлайки под статьёй. Поэтому буду признателен за комментарии, которые помогут улучшить данный код или укажут на мои ошибки.

Судя по информации из ващего профиля, вы занимаетесь фронтендом минимум 2 года, но ваш код — форматирование, наименование переменных и т.п. — тянет на очень новичка.
Нет никакого кодстайла — где-то ставится точка с запятой, где-то нет; где-то одни отступы, где-то другие.


Куски кода начинаются из ниоткуда и кончаются ни чем:
border: 4px solid #BB70B3; — просто потому что
class ImgComponent и его метод onClick — не закрыты
не, я, конечно, догадаюсь, но есть же правила хорошего тона. Кстати ссылку на демо в каком-нибудь codepen прикрепить тоже не мешало бы.


display: table и float — в век флексбоксов и гридов выглядит странновато, кмк
Почему переменная называется scroll_svg? А если я захочу поменять картинку на пнг, мне надо изменить и название переменной, или оставить как есть? А css класс вообще называется scroll_to.
Что будет, если я домотаю галерею до конца, она исчезнет?


Короче, вопрос много


P.S. Кстати, класс Img из ваших задач, вы так и не реализовали

Огромное спасибо за потраченное на комментарий время, буду работать над собой и кодом.
Зарегистрируйтесь на Хабре, чтобы оставить комментарий

Публикации

Истории