Jump to content

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


CyMpaK56
 Share

Recommended Posts

<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 зачем?

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

Link to comment
Share on other sites

<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 by CyMpaK56
Link to comment
Share on other sites

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

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

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

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

 

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

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

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

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

 

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

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

Edited by CyMpaK56
Link to comment
Share on other sites

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

Причем тут overflow: hidden; просто назначьте класс clearfix родителю.
Link to comment
Share on other sites

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

 

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

 

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

 

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

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

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

 

и т.д.

Link to comment
Share on other sites

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

 

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

 

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

 

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

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

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

 

и т.д.

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

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

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

Link to comment
Share on other sites

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

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

Link to comment
Share on other sites

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

 

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

 

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

 

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

 

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

 

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

Edited by artem1987
Link to comment
Share on other sites

 

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

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

 

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

Link to comment
Share on other sites

Спасибо за совет сделал на 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 пх. Я открыл, сделал вид, якобы читаю столь важную бумажку. И потом несколько секунд убивался, тыкал в поля вокруг картины, чтобы она закрылась. А нет. Закрывается только крестиком в правом верхнем углу, а это не есть айс. Приходится мотаться обратно вверх.

Link to comment
Share on other sites

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

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

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

 

Спасибо за совет сделал на 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 by CyMpaK56
Link to comment
Share on other sites


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


...но заказчик строго хочет его и нечего слышать не хочет у него все этим шрифтом сделано визтки и тд
[/qoute]
коллега... это хрень... умейте настаивать. разумное обоснование достойно может быть оплачено. 
если у него таким шрифтом на визитках написано то те кто делал визитки, ИМХО, тоже не далеко ушли... да и какое вообще отношение визитки имеют к сайту ?
 
а при клике на крестик удаляю тогда получается ему не надо каждый раз грузить большую картнку
Добавил закрытие
блин... ну вы и спец по изобретению велосипедов :) 
Edited by artem1987
Link to comment
Share on other sites

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

 

блин... ну вы и спец по изобретению велосипедов  :)
Link to comment
Share on other sites

 

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

 

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

 

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

Я не понимаю ваше замечание по поводу велосипедов напишите как вы делаете.

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

 Share

×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue. See more about our Guidelines and Privacy Policy