Оцените работу

CyMpaK56 · September 11, 2014

Оцените работу ссылка на сайт: http://skf-trial.ru/

За ранее спасибо.

Edited September 11, 2014 by CyMpaK56

Sergik+ · September 11, 2014

Очень даже неплохо.

Great Rash · September 11, 2014

Для чего тут неймспейс?

Инлайн стили, фу.

Вы серьёзно? И это 2014 году... И это при наличии .clearfix в стилях...

Абсолютные ссылки, зачем?

<div class="article"><table>...

Таблица нафига тут?

overflow-x: hidden; для body зачем?

За дизайн не знаю надо говорить или нет, но он ужасен. Короче так себе...

CyMpaK56 · September 11, 2014

Можно пожалуйста если находите ошибку, написать правильное решение если не трудна

CyMpaK56 · September 11, 2014

<html xmlns="http://www.w3.org/1999/xhtml">
Для чего тут неймспейс?
<p style="font-weight: bold; color: #000000;">
Инлайн стили, фу.
<div class="clearfix"></div>
Вы серьёзно? И это 2014 году... И это при наличии .clearfix в стилях...
<a href="http://skf-trial.ru/o-kompanii/">
Абсолютные ссылки, зачем?
<div class="article"><table>...
Таблица нафига тут?
overflow-x: hidden; для body зачем?
За дизайн не знаю надо говорить или нет, но он ужасен. Короче так себе...

1)Убрал

2)<p style="font-weight: bold; color: #000000;"> Сайт стоит на движке в редактор текст с стилями закинули.

3)Ссылки сайт на движке, хотя в этом вообще проблемы не наблюдаю если я не прав объясните пожалуйста почему

4)Если не clearfix тогда что?

5)Таблицу убрал

6)overflow-x: hidden; на мобильнике появляется горизонтальная прокрутка это не адаптивная верстка но все же

Спасибо за критику

Edited September 11, 2014 by CyMpaK56

Great Rash · September 11, 2014

Если не clearfix тогда что?

Почиайте как пользоваться clearfix'ом. Там <div> вообще не нужен.

overflow-x: hidden; на мобильнике появляется горизонтальная прокрутка это не адаптивная верстка но все же

И поэтому вы обрезаете всё к чёртовой матери. Т.е. если у меня окно открыто не на весь экран, то шиш мне, а не сайт, так?

CyMpaK56 · September 11, 2014

Если не clearfix тогда что?
Почиайте как пользоваться clearfix'ом. Там <div> вообще не нужен.
overflow-x: hidden; на мобильнике появляется горизонтальная прокрутка это не адаптивная верстка но все же
И поэтому вы обрезаете всё к чёртовой матери. Т.е. если у меня окно открыто не на весь экран, то шиш мне, а не сайт, так?

1) там 2 блока с обтеканием overflow:hidden не подходит поскольку узор с права выходит за пределы блока подскажите как луче будет

2)Об этом не подумал щас просто сделаю стиль для мобильника

Edited September 11, 2014 by CyMpaK56

Great Rash · September 11, 2014

1) там 2 блока с обтеканием overflow:hidden не подходит поскольку узор с права выходит за пределы блока подскажите как луче будет

Причем тут overflow: hidden; просто назначьте класс clearfix родителю.

CyMpaK56 · September 11, 2014

Спасибо видно я что-то недопонял когда про него читал

Вот уже не зря выставил работу

Edited September 11, 2014 by CyMpaK56

artem1987 · September 12, 2014

Приветствую,

Не впечатлило но думаю что клиент в экстазе

Пару вопросов...

1. Ну не понимаю я зачем использовать шрифт с засечками ???

2. Сайт реально тяжелый !!! Зачем пихать скан сертификатов размером 6 мб ???

3. Если делаете плавающее меню то и делайте его плавающим. Ваш вариант это какой-то велосипед

и т.д.

CyMpaK56 · September 12, 2014

Приветствую,

Не впечатлило но думаю что клиент в экстазе

Пару вопросов...

1. Ну не понимаю я зачем использовать шрифт с засечками ???
2. Сайт реально тяжелый !!! Зачем пихать скан сертификатов размером 6 мб ???
3. Если делаете плавающее меню то и делайте его плавающим. Ваш вариант это какой-то велосипед

и т.д.

1) Шрифт не я выбирал если поставлю другой скажут шрифт не тот, как вы поступаете в таких случаях?

2) Заказчику очень важно чтоб скан читабелен был, сейчас полазию в инете поищю как вес уменьшить хотя дизайнера просил

3) Ну это типо фишка такая им понравилось

Great Rash · September 12, 2014

2) Заказчику очень важно чтоб скан читабелен был, сейчас полазию в инете поищю как вес уменьшить хотя дизайнера просил

Можно на сайт повесить пожатые сканы, а по клику увеличенные показывать. Кому не надо не будет скачивать, а кому важно почитать, то тыкнет и почитает. Хотя 99% посетителей сайта эти документы вообще не упёрлись. Это всё тараканы в голове у заказчика бесятся.

artem1987 · September 12, 2014

1) Шрифт не я выбирал если поставлю другой скажут шрифт не тот, как вы поступаете в таких случаях?

Элементарно - не читабельно таким шрифтом

2) Заказчику очень важно чтоб скан читабелен был, сейчас полазию в инете поищю как вес уменьшить хотя дизайнера просил

Ну проще простого. Берете InfantView и при сохранении уменьшаете параметр quality. Также уменьшите размер самого фото. У вас простыни такие что ого-го... 1800 на 2539... кто это будет смотреть ???

3) Ну это типо фишка такая им понравилось

Это велосипед

Edited September 12, 2014 by artem1987

Get to heaven · September 12, 2014

норм

Edited September 12, 2014 by Get to heaven

CyMpaK56 · September 12, 2014

2) Заказчику очень важно чтоб скан читабелен был, сейчас полазию в инете поищю как вес уменьшить хотя дизайнера просил
Можно на сайт повесить пожатые сканы, а по клику увеличенные показывать. Кому не надо не будет скачивать, а кому важно почитать, то тыкнет и почитает. Хотя 99% посетителей сайта эти документы вообще не упёрлись. Это всё тараканы в голове у заказчика бесятся.

Спасибо за совет сделал на jquery подгрузку большой картинки

Make Uper · September 14, 2014

Спасибо за совет сделал на jquery подгрузку большой картинки

Судя по всему, это fancybox. Только почему то он неправильно работает. Ни переключения нажатием на текущую увеличенную картину, ни кнопок навигации.

И почему такой код,

<img class="certificate2" width="226" height="332" alt="Свидетельство о допуске" src="/images/smallSrtificat2.png">

откуда он? А где путь к увеличенной картине? И вообще, я помню работал над одним сайтом, и большие картины и превюшки

Попробуйте почитать, Как настроить fancybox.

Я когда-то пользовался, но мой код при просмотре через файрбаг выглядел вот как:

(код развернутого изображения)<div id="fancybox-content" style="border-width: 10px; width: 402px; height: 557px; opacity: 1;"><img id="fancybox-img" src="http://blablabla/some_files/File/bla3.jpg" alt="Бла бла в современной культуре">( код превьюшки)<img width="252" height="349" style="margin-left: 20px; float: right;" alt="Роль бла бла в современной культуре" src="/some_files/Image/mods3.jpg"></a>

Заметьте, увеличенные картины и превью берутся из разных папок.

А вставляется в текст вот так -

<a class="fancy-img" href="/some_files/File/bla3.jpg" title="Роль бла бла в современной культуре" rel="group"><img width="252" height="349" style="margin-left: 20px; float: right;" alt="Роль бла бла в современной культуре" src="/some_files/Image/bla3.jpg"></a>

А у вас a-href вообще никуда не ведет. Вобщем, посмотрите, сравните.

П.с., И к тому же, у вас увеличенная картина высотой в 800 пх. Я открыл, сделал вид, якобы читаю столь важную бумажку. И потом несколько секунд убивался, тыкал в поля вокруг картины, чтобы она закрылась. А нет. Закрывается только крестиком в правом верхнем углу, а это не есть айс. Приходится мотаться обратно вверх.

PaAndrew · September 14, 2014

Все очень хорошо,

только один вопрос - Вас самих текущий шрифт устраивает? не лучше ли его поменять?

CyMpaK56 · September 15, 2014

Все очень хорошо,
только один вопрос - Вас самих текущий шрифт устраивает? не лучше ли его поменять?

Шрифт лично меня бесит но заказчик строго хочет его и нечего слышать не хочет у него все этим шрифтом сделано визтки и тд

Спасибо за совет сделал на jquery подгрузку большой картинки

Судя по всему, это fancybox. Только почему то он неправильно работает. Ни переключения нажатием на текущую увеличенную картину, ни кнопок навигации.

И почему такой код,
<img class="certificate2" width="226" height="332" alt="Свидетельство о допуске" src="/images/smallSrtificat2.png">

откуда он? А где путь к увеличенной картине? И вообще, я помню работал над одним сайтом, и большие картины и превюшки

Попробуйте почитать, Как настроить fancybox.

Я когда-то пользовался, но мой код при просмотре через файрбаг выглядел вот как:
(код развернутого изображения)<div id="fancybox-content" style="border-width: 10px; width: 402px; height: 557px; opacity: 1;"><img id="fancybox-img" src="http://blablabla/some_files/File/bla3.jpg" alt="Бла бла в современной культуре">( код превьюшки)<img width="252" height="349" style="margin-left: 20px; float: right;" alt="Роль бла бла в современной культуре" src="/some_files/Image/mods3.jpg"></a> 
Заметьте, увеличенные картины и превью берутся из разных папок.
А вставляется в текст вот так -
<a class="fancy-img" href="/some_files/File/bla3.jpg" title="Роль бла бла в современной культуре" rel="group"><img width="252" height="349" style="margin-left: 20px; float: right;" alt="Роль бла бла в современной культуре" src="/some_files/Image/bla3.jpg"></a>
А у вас a-href вообще никуда не ведет. Вобщем, посмотрите, сравните.
П.с., И к тому же, у вас увеличенная картина высотой в 800 пх. Я открыл, сделал вид, якобы читаю столь важную бумажку. И потом несколько секунд убивался, тыкал в поля вокруг картины, чтобы она закрылась. А нет. Закрывается только крестиком в правом верхнем углу, а это не есть айс. Приходится мотаться обратно вверх.

Там нету ссылк я не через фансибокс сделал а через .append

<div class="bigImg"><a href="#" class="certificate-close"></a><img src="/images/certificate1.png" alt="Свидетельство о допуске"></div>

а при клике на крестик удаляю тогда получается ему не надо каждый раз грузить большую картнку

Добавил закрытие

Edited September 15, 2014 by CyMpaK56

artem1987 · September 15, 2014

...но заказчик строго хочет его и нечего слышать не хочет у него все этим шрифтом сделано визтки и тд
[/qoute]
коллега... это хрень... умейте настаивать. разумное обоснование достойно может быть оплачено.
если у него таким шрифтом на визитках написано то те кто делал визитки, ИМХО, тоже не далеко ушли... да и какое вообще отношение визитки имеют к сайту ?

а при клике на крестик удаляю тогда получается ему не надо каждый раз грузить большую картнку
Добавил закрытие
блин... ну вы и спец по изобретению велосипедов

...но заказчик строго хочет его и нечего слышать не хочет у него все этим шрифтом сделано визтки и тд
[/qoute]
коллега... это хрень... умейте настаивать. разумное обоснование достойно может быть оплачено.
если у него таким шрифтом на визитках написано то те кто делал визитки, ИМХО, тоже не далеко ушли... да и какое вообще отношение визитки имеют к сайту ?

а при клике на крестик удаляю тогда получается ему не надо каждый раз грузить большую картнку
Добавил закрытие
блин... ну вы и спец по изобретению велосипедов

Edited September 15, 2014 by artem1987

artem1987 · September 15, 2014

...но заказчик строго хочет его и нечего слышать не хочет у него все этим шрифтом сделано визтки и тд
[/qoute]
коллега... это хрень... умейте настаивать. разумное обоснование достойно может быть оплачено.
если у него таким шрифтом на визитках написано то те кто делал визитки, ИМХО, тоже не далеко ушли... да и какое вообще отношение визитки имеют к сайту ?

а при клике на крестик удаляю тогда получается ему не надо каждый раз грузить большую картнку
Добавил закрытие

блин... ну вы и спец по изобретению велосипедов

CyMpaK56 · September 15, 2014

...но заказчик строго хочет его и нечего слышать не хочет у него все этим шрифтом сделано визтки и тд
[/qoute]
коллега... это хрень... умейте настаивать. разумное обоснование достойно может быть оплачено.
если у него таким шрифтом на визитках написано то те кто делал визитки, ИМХО, тоже не далеко ушли... да и какое вообще отношение визитки имеют к сайту ?

а при клике на крестик удаляю тогда получается ему не надо каждый раз грузить большую картнку
Добавил закрытие

блин... ну вы и спец по изобретению велосипедов

Ну мне кажется любой б настаивал а ругаться с клиентом из-за того что он не хочет другой шрифт по моему это глупо просто найдут человека который поставит именно этот шрифт и этот шрифт не ограничивается одними визитками вся наружная реклама идет этим шрифтом.
Я не понимаю ваше замечание по поводу велосипедов напишите как вы делаете.

Sign In

Оцените работу

Recommended Posts

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Join the conversation

Обсуждения

Important Information