Jump to content

Просьба прокомментировать


Den
 Share

Recommended Posts

Ну так конкретней объясните в чем странность слов?

А что с датой реги? Да, я зарегистрировался здесь одним из первых. Да, я посещал еще старый форум, до создания этого и что тут странного?

Link to comment
Share on other sites

Den

Во-первых если ты выставляешь на обсуждение свою работу и хочешь услышать критику, то нехрен тогда спорить, когда тебе её выговаривают.

Во-вторых вёрстка мягко сказать - полное говно. Рассказываю почему:

1) <div class="divider_top"></div> - Вот этот див нахрен нужен? Что мешало повесить полоску фоном на body например?

2)

<div><img height="16" width="16" alt="" src="http://****.ru/images/news_ico.png">
<a href="news.html">Новости</a>
</div>

Вот это что за бред? Меню не на списках, у картинок прописана одинаковая ширина и высота, почему нельзя было вынести их в КСС? У последнего дива стоит класс, тут вообще можно было прописать классы у всех пунктов, а картинки повесить фоном у ссылок.

3)

<span onclick="window.openCenteredWindow('http://****.ru/city.html');">Выберите населённый пункт</span>

Это что, ссылка? Тогда почему я не вижу элемента <a></a>?

4)

<img height="106" width="284" alt="Умная доска объявлений, объявления о товарах и услугах" src="http://****.ru/images/img/logo.gif">

А если в хеаде появятся ещё картинки, то что будет с лого? Неграмотно, плохо, необдуманно.

5)

<div class="searchform">

Зачем нужен этот лишний див? Почему нельзя туже операцию проделать с тегам <form>, чем не угодил? Сами элементы формы можно было бы поместить в таблицу и сделать всё намного удобнее.

6) Далее меню. Везде стоят отрицательные отступы, релативы, отрицательные top, УЖАС. <ins class="icon_menu"> Зачем нужен тег <ins>, знаешь? Нахера его тут использовать? Дале на всех стоят классы icon_menu, зачем? Разве нельзя просто обратиться к ним через КСС без этого класса?

Меню сделано бездарно совершенно!

7) Центр сделан хреново очень, неудобно, и неправильно. В будущем если понадобится, будет неудобно менять структуру, да и вообще весь код тоже.

Можно было обойтись тремя дивами максимум с более гибкой формой.

Итог:

Сложилось такое впечатление, что автор вообще не понимает и не знает, как правильно применять элементы на странице, как с ними обращаться и использовать. Но зато он хорошо знает дивы, что они есть и причём только ОНИ. Других элементов для него не существует видимо.

В целом вёрстка ужасная, некультурная с неменее ужасным кодом и организацией. Тут нужно всё перевёрстывать однозначно.

Жаль того программера и других людей, которые в последствии будут иметь с сайтом дело)

Кстати я рассказал только поверхностный взгляд, если вдоваться в подробности, то не хватит времени описать всю жесть этой вёрстки.

Это мой тон высокомерен?? Мда... вопросов больше не имею.

Ты мне Вердера напоминаешь, тот тоже выставляя свою работу опровергает любую критику в свой адрес. Не смотря на то, что сам просит прокомментировать косяки.

Edited by s0rr0w
Link to comment
Share on other sites

Окей, спасибо всем за мнения. psywalker, на некоторую критику уже был дан ответ выше.

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

Одним словом обсуждение можно закрыть, я получил достаточно информации :)

Link to comment
Share on other sites

Окей, спасибо всем за мнения. psywalker, на некоторую критику уже был дан ответ выше.

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

Одним словом обсуждение можно закрыть, я получил достаточно информации :)

А что ты ожидал услышать, как тебя будут хвалить за эту жесть?

Кстати я уверен, что это твои отмазки. Покажи нам любую свою работу и уверен - она окажется не чуть не лучше)

Link to comment
Share on other sites

Прочтите еще раз первый пост пожалуйста, в частности это:

Где что не верно, глупо и не правильно, не профессионально, косяки разных браузеров и всё такое.

Разве так напрашиваются на похвалу?

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

Я не говорю, что у меня супер код и с большей частью критики я согласен, а вы думали, что я буду просто читать никак не комментируя чтоли? Просто, если начать здесь спорить и выяснять что-то это зафлудится на 10 страниц, а это не тот раздел.

Засим откланиваюсь.

Link to comment
Share on other sites

Извини, но я просто логики твоей не понимаю. Если ты знал, что код у тебя говно, то что хотел услышать, непонятно. Информацию ты получил, а что ты с ней будешь делать, если исправить то как ты говоришь, ничего нельзя (хотя я в этом сомневаюсь).

Link to comment
Share on other sites

Если вы приведете более простой способ нарисовать закругления углов, я вам спасибо скажу

css3pie.com

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

Кто вам это сказал? Чем же вам ссылка на главную с главной не угодила?

По поводу ins

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

Не все, что есть на сайте Лебедева - хорошо. Сделать картинки с подписями по центру совершенно не проблема. А вот вкладывать блочный элемент в строчный - проблема.

г-ну. Great Rash, хочу дать совет со стороны. Будьте менее импульсивны и предполагайте хоть иногда, что если так сделано, то может по-другому в этом месте и нельзя было? Старайтесь так же не употреблять в общении грубостей, таких, как пипец, отстой и т.д. Вы все таки модератор - лицо ресурса

Грубости в ваш адрес не было никакой. Если ваш код и правда отстой, то тут уж ничего не поделаешь... если б я сказал, что ваш код не слишком хорош, то это было бы неправдой, так же как неправдой было бы если б я сказал, что код ужасен. Ваш код какой-то никакой... лучше чем "отстой" слова и не подобрать :)

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

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

Link to comment
Share on other sites

Ваш код какой-то никакой...лучше чем "отстой" слова и не подобрать

Нет, неправда, я подобрал более точнее - "Полное говно" :)

Кстати, да, подтверждаю, Great Rash не злой, а справедливый)

Link to comment
Share on other sites

Мне показалось что он нормально на критику реагирует. А вот критики какие то агрессивные.

Это мы то агрессивные? Мы что-то не по делу сказали разве? Как бы вы назвали этот код если не отстойным?

Вот если бы Verder глянул на это УГ с точки зрения дизайна, то ТС тогда понял бы, что такое агрессия :)

Link to comment
Share on other sites

Как бы вы назвали этот код если не отстойным?

Дружище, я же вверху уже написал, как. Смотри пост 35

Вот если бы Verder глянул на это УГ с точки зрения дизайна, то ТС тогда понял бы, что такое агрессия :)

Это точно)

Link to comment
Share on other sites

width="16" height="16"

Кстати говоря, некоторым капризным библиотекам (вроде jScrollPane) необходимо указывать ширину и высоту картинкам находящимся в этом блоке. Так, что здесь в зависимости от ситуации надо оценивать.

Link to comment
Share on other sites

Guest
This topic is now closed to further replies.
 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