Skip to content

Comments

Трушков Валерий#58

Open
flafla wants to merge 1 commit intourfu-2016:masterfrom
flafla:master
Open

Трушков Валерий#58
flafla wants to merge 1 commit intourfu-2016:masterfrom
flafla:master

Conversation

@flafla
Copy link

@flafla flafla commented Oct 31, 2016

@honest-hrundel
Copy link

🍏 Пройден линтинг и базовые тесты

@dotokoto
Copy link

dotokoto commented Nov 1, 2016

Привет! Есть следующие замечания по газете:

  1. В firefox совсем не похожа на газету, нет колонок. И картинки огромные, занимают весь экран
    geektimes 2016-11-01 20-11-34
    geektimes 2016-11-01 20-15-31

  2. Не соблюдаются требования задания:

  • не во всем тексте есть мягкие переносы
  • Нет картинок шириной в несколько колонок
  • Мало разных шрифтов. Попробуй подключить больше, не только из googleFonts, но и с помощью @font-face
    В целом, выглядит, как будто сделано на скорую руку :( Попробуй добавить больше разнообразия - статьи с разным количеством колонок, отдельные блоки с интересными фактами, и тд, что еще придумаешь. После этого еще раз посмотрю

@dotokoto
Copy link

dotokoto commented Nov 1, 2016

🍅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants