1. Главная
  2. Форумы
  3. Разработка для Drupal
  4. Выставка сайтов

Интересно мнение друпальцев

Главные вкладки

Аватар пользователя Maddoc Maddoc 10 января 2010 в 20:14

Выставляю на рассмотрение сайт http://semeyka.com.ua

Делал его как подарок жене. Хотелось сделать простенький и легкий дизайн. Поскольку это моя вторая работа в сайтостроении (всего лишь хобби), то буду рад выслушать любую критику. А от кого ее лучше услышать, если не от заядлых друпаловодов:)

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

Комментарии

Аватар пользователя Maddoc Maddoc 10 января 2010 в 21:34

"Ветер" wrote:
Сайт уж очень тормозит.

Странно. Пока такого замечено не было. Не могли бы Вы пояснить, в чем именно тормозит?

Аватар пользователя xxandeadxx xxandeadxx 10 января 2010 в 22:31
  • большинство ссылок не реагируют на наведение
  • заголовки разделов выгладят так же как и заголовки нод http://img509.imageshack.us/img509/5973/20100110220124.jpg
  • область контента и сайдбар слишком близко друг к другу, можно попробовать чуть сдвинуть контент вправо, либо добавить разделитель http://img267.imageshack.us/img267/8754/20100110222049.jpg
  • не выделен текущий раздел и подраздел http://img707.imageshack.us/img707/7532/20100110222235p.jpg
  • горизонтальный скрол при разрешении 1024*768
  • при ширине окна меньше 1024, слева всё впритык, воздух нужен http://img188.imageshack.us/img188/1530/20100110222742.jpg
  • когда находишься в разделе, ссылка на rss наверное должна быть после названия раздела а не в самом низу
  • вам не кажется капча слишком сложной для дам ? Smile может остановится на математической капче?!
  • не объединены скрипты и стили
  • вёрстка не валидная 89 Errors, 7 warning(s)
Аватар пользователя Maddoc Maddoc 11 января 2010 в 9:01

"xxandeadxx" wrote:
* большинство ссылок не реагируют на наведение
* заголовки разделов выгладят так же как и заголовки нод http://img509.imageshack.us/img509/5973/20100110220124.jpg
* область контента и сайдбар слишком близко друг к другу, можно попробовать чуть сдвинуть контент вправо, либо добавить разделитель http://img267.imageshack.us/img267/8754/20100110222049.jpg
* не выделен текущий раздел и подраздел http://img707.imageshack.us/img707/7532/20100110222235p.jpg
* горизонтальный скрол при разрешении 1024*768
* при ширине окна меньше 1024, слева всё впритык, воздух нужен http://img188.imageshack.us/img188/1530/20100110222742.jpg
* когда находишься в разделе, ссылка на rss наверное должна быть после названия раздела а не в самом низу
* вам не кажется капча слишком сложной для дам ? Smile может остановится на математической капче?!
* не объединены скрипты и стили
* вёрстка не валидная 89 Errors, 7 warning(s)

Спасибо большое за такой разбор. Буду оптимизировать:)
Я еще чайник в этом деле и поэтому такая критика очень важна.

Аватар пользователя pselfin pselfin 11 января 2010 в 0:43

а какким модулем (или способом) делается второй ряд ссылок(подразделов) при наведении на "Здоровье", "Он и она", "Семья" и тд?

Аватар пользователя pselfin pselfin 11 января 2010 в 1:13

"Stan.Ezersky" wrote:
А в код слабо посмотреть?

Таки за подсказку спс)))
А вдруг бы они стили и ид по другому назвали? Но нет, не назвали.
Значит сам дурак...)))
Вдруг кому понадобится http://drupal.org/project/dynamic_persistent_menu

Аватар пользователя Serg_M Serg_M 11 января 2010 в 11:55

А мне почему-то не нравится когда сплошной белый фон. Хоть бы колонки или блоки имели рамки?

А можно код к кнопке "Вставить видео" в комментариях?

Аватар пользователя Maddoc Maddoc 11 января 2010 в 13:45

"Serg_M" wrote:
А можно код к кнопке "Вставить видео" в комментариях?

Просто используется модуль Video Filter. А кнопка, соответственно, формирует тег [video:url]. Остальное обрабатывает модуль:

js:
var M = E.getSelection().match(new RegExp('^\\[video(?:=(\\d+)x(\\d+))?](.+)\\[/img]$')) || ['', '', '', ''];
var form = [
{name: 'src', title: 'Ссылка на видео', value: M[3]},
];
eDefTagDialog('video', form, 'Вставить видео', 'OK', 'bbcVideoProcess');

bbcVideoProcess = function(tag, form) {
  var src = form.elements['attr_src'].value;
  var str = '[video:'+ src + ']';
  editor.dialog.close();
  if (src) editor.active.replaceSelection(str);
}

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