Comments 25
Что бросается в глаза, так это нерациональное использование картинок. Ну зачем там прозрачный PNG-24? Да даже его можно сжать с 16,5 КБ до 2,91 КБ. А вообще рекомендую сделать PNG-8 без прозрачности, и все картинки склеить в спрайт, потому что ваш редактор будет развиваться и обрастать новыми кнопками, и в конце концов при каждой его загрузке придется ждать, пока загрузится сотня маленьких файликов общим весом килобайт 200. Так раньше было TinyMCE и в паре других редакторов.
+11
ок принято к исполнению)))
+2
Еще интересна критика?
0
критика всегда интересна)))
+2
1. Тег пишется tag, паттерн — pattern, линк — link :)
2. У ссылки нет атрибута alt.
3. Вместо лишнего img сделайте лучше картинку фоном ссылки.
4. Не прописывайте в плагине жестко названия и имена файлов, лучше вбейте все это в options по умолчанию (для каждой кнопки массив [тег, имя, атрибуты]).
5. Когда все будет в options, сделайте один общий цикл по кнопкам для генерации кода.
6. Не храните служебную инфу в атрибутах элемента (она уже есть в options), если только не собираетесь подхватывать кнопки из готового html.
7. Используйте +=.
2. У ссылки нет атрибута alt.
3. Вместо лишнего img сделайте лучше картинку фоном ссылки.
4. Не прописывайте в плагине жестко названия и имена файлов, лучше вбейте все это в options по умолчанию (для каждой кнопки массив [тег, имя, атрибуты]).
5. Когда все будет в options, сделайте один общий цикл по кнопкам для генерации кода.
6. Не храните служебную инфу в атрибутах элемента (она уже есть в options), если только не собираетесь подхватывать кнопки из готового html.
7. Используйте +=.
+11
ок спасибо.
+1
в этот раз Вы, видимо, решили внимать советам :)
0
советы, вещь полезная))) и я их всегда учитываю…
Внимать советам…
Учитывать советы,
И слепо следовать всем советам
Это три разных пункта))))) первые два выполняются мною всегда))) третий… Ну ООООЧЕНЬ редко)))
Внимать советам…
Учитывать советы,
И слепо следовать всем советам
Это три разных пункта))))) первые два выполняются мною всегда))) третий… Ну ООООЧЕНЬ редко)))
+1
Вы молодец! Мне понравилось как это выглядит внешне — возьму на вооружение.
Я как-то собирал подобную вещь для себя — недособирал, пропало время и желание.
Я как-то собирал подобную вещь для себя — недособирал, пропало время и желание.
0
Я использую вот этот скрипт hexmode.org/tagger_menu/ и очень доволен. Тоже настроил под себя…
0
teg_linck teg_image
Определитесь, вы пишете транслитом или по-английски: или tag_image или teg_kartinka, а то какой-то взрыв мозга получается. Хотя проблема может быть сложнее — об этом говорит слово «linck», кажется это «link». Про альт у ссылки уже сказали, но я всё никак отойти не могу o_0
0
UFO just landed and posted this here
UFO just landed and posted this here
Нет предпросмотра.
0
я конечно понимаю, что краткость сестра талана, но не все такие талантливые…
какого именно предосмотра нет? если работы плагина то — Посмотреть пример, пощелкать по кнопочкам настройки
какого именно предосмотра нет? если работы плагина то — Посмотреть пример, пощелкать по кнопочкам настройки
0
предпросмотр — это значит нажать на одноименную кнопочку и посмотреть как это будет выглядеть в реальности в браузере.
Можно:
— открыть в новом окне
— разместить div-окно под редактором (вроде лучшее решение)
— заменить редактор на окно предпросмотра и добавить ссылку «Вернуться к редактированию»
— возможно что-то еще…
Можно:
— открыть в новом окне
— разместить div-окно под редактором (вроде лучшее решение)
— заменить редактор на окно предпросмотра и добавить ссылку «Вернуться к редактированию»
— возможно что-то еще…
0
Sign up to leave a comment.
Мой редактор BB кодов