Перейти к содержимому

Фотография
- - - - -

Очередной HTML+JS


  • Вы не можете создать новую тему
  • Авторизуйтесь для ответа в теме
Сообщений в теме: 7

#1 eelijah

eelijah

    Прохожий

    Топикстартер
  • Участник
  • 11
    60 сообщ.

Отправлено 25 Июнь 2013 в 12:00

Всем доброго времени суток!
Вот очередная работа http://препод18.рф
  • 0

Спонсор

#2 lekzd

lekzd

    parse error: parse error, unexpected T_STRING...

  • V.I.P.
  • 428
    1 125 сообщ.

Отправлено 25 Июнь 2013 в 16:41

1. Лет 5 назад все сайты были такими
http://screenshot.su...ae3a303d304.jpg
2. везде разный интерлиньяж, не очень удобно читать
3. ширина основного текстового блока более 600 пикселей = тяжелее читать
4. Заголовки имеют слишком маленький нижний отступ, использование padding для отступов как-то странно
5. удобнее, когда схема проезда является ссылкой на карту яндекс или гугл, существующая картинка на сайте слишком мелкая
6. зачем на сайте часы? (не, ну, в 2007-ом у всех на сайтах были часы и снежинки, но время-то идет)
7. отступы снизу у списков слишком мелкие
8. фото учителя некачественное http://xn--18-jlcd9b...img/uchitel.jpg, к сожалению, оно от этого не выигрывает
9. e-mail учителя - ссылкой
10. адрес учителя - ссылкой на карту
11. ссылку на сайт про бассейны оторвать от меню
  • 1

#3 Volter9

Volter9

    defined('SURRENDER') or die(); // StarWars

  • V.I.P.
  • 158
    1 145 сообщ.

Отправлено 25 Июнь 2013 в 18:34

12. На маленьких экранах, (в моем случае 1024х768) горзонтальный скроллбар появляется.
13. Картинки у вас подгружаются странно подгружаются. Нужно их сначало загрузить с помощью:
var img = new Image();
img.src = "путь/к/картинке.jpg";
img.onload = function () {
// загрузилась картинка
}
а потом на DOMContetnLoaded вставить на страницу.
14. jQuery должен быть в сжатой версии а не в версии разработчика, скачайте ее тут. (так будет быстрее загружатся)

Сообщение отредактировал Volter9: 25 Июнь 2013 в 18:35

  • 1

#4 CamaroSS

CamaroSS

    Частый гость

  • Участник
  • 52
    176 сообщ.

Отправлено 26 Июнь 2013 в 10:11

14. jQuery должен быть в сжатой версии а не в версии разработчика, скачайте ее тут. (так будет быстрее загружатся)


А ещё лучше такие вещи сразу тянуть с CDN гугла,
а свою копию иметь только для случаев, когда CDN недоступен.
  • 0

#5 eelijah

eelijah

    Прохожий

    Топикстартер
  • Участник
  • 11
    60 сообщ.

Отправлено 26 Июнь 2013 в 13:44

1. Интерлиньяж поправил.
2. Ширина в 1200px была задана заказчиком, я взял по 200px каждой колонке. Попробую добавить по 100px к боковым колонкам.
3. Отступы отредактировал. Padding изменил значениями margin.
4. Схему проезда и адрес сделал ссылками на яндекс карту.
5. Часы тоже пожелание заказчика.
6. Сделал подгрузку картинок (не уверен на 100% что верно, но работает) и заменил jQuery на сжатую версию (имя файла оставил без изменений)
  • 0

#6 lekzd

lekzd

    parse error: parse error, unexpected T_STRING...

  • V.I.P.
  • 428
    1 125 сообщ.

Отправлено 26 Июнь 2013 в 16:00

2. Ширина в 1200px была задана заказчиком, я взял по 200px каждой колонке. Попробую добавить по 100px к боковым колонкам.

можно добавить отступов и там и там
  • 1

#7 Jackito

Jackito

    Designer

  • Участник
  • 103
    289 сообщ.

Отправлено 27 Июнь 2013 в 09:43

Можно я тоже добавлю?

1. На главной странице ссылка на логотипе не нужна, так же как и ссылка на пункте меню Главная.
2. А почему бы не сделать выделения пункта меня для соответствующей страницы?
3. Внутренние отступы в трех колонках разные. В последнем его почти нет. Ну по крайней мере смотрится так.
4. А еще, мне кажется, что текст который написан под логотипом курсивом про Русский язык, мало кто будет читать. Не понятно, что написано.
5. Ну и ссылки подчеркнуть не мешало бы.

А ты этот сайт делал для удовольствия или за материальное вознаграждение? И если не секрет, за какую стоимость?
  • 1

#8 eelijah

eelijah

    Прохожий

    Топикстартер
  • Участник
  • 11
    60 сообщ.

Отправлено 28 Июнь 2013 в 08:37

Текст под логотипом служит чисто разделителем
  • 0



Похожие темы Свернуть

  Название темы Форум Автор Статистика Последнее сообщение
Статистика

Количество пользователей, читающих эту тему: 0

0 пользователей, 0 гостей, 0 скрытых пользователей