Мой редактор BB кодов

[ipetropolsky]

Что бросается в глаза, так это нерациональное использование картинок. Ну зачем там прозрачный PNG-24? Да даже его можно сжать с 16,5 КБ до 2,91 КБ. А вообще рекомендую сделать PNG-8 без прозрачности, и все картинки склеить в спрайт, потому что ваш редактор будет развиваться и обрастать новыми кнопками, и в конце концов при каждой его загрузке придется ждать, пока загрузится сотня маленьких файликов общим весом килобайт 200. Так раньше было TinyMCE и в паре других редакторов.

[kamaikin]

ок принято к исполнению)))

[ipetropolsky]

Еще интересна критика?

[kamaikin]

критика всегда интересна)))

[ipetropolsky]

1. Тег пишется tag, паттерн — pattern, линк — link :)
2. У ссылки нет атрибута alt.
3. Вместо лишнего img сделайте лучше картинку фоном ссылки.
4. Не прописывайте в плагине жестко названия и имена файлов, лучше вбейте все это в options по умолчанию (для каждой кнопки массив [тег, имя, атрибуты]).
5. Когда все будет в options, сделайте один общий цикл по кнопкам для генерации кода.
6. Не храните служебную инфу в атрибутах элемента (она уже есть в options), если только не собираетесь подхватывать кнопки из готового html.
7. Используйте +=.

[kamaikin]

ок спасибо.

[aubt]

в этот раз Вы, видимо, решили внимать советам :)

[kamaikin]

советы, вещь полезная))) и я их всегда учитываю…
Внимать советам…
Учитывать советы,
И слепо следовать всем советам
Это три разных пункта))))) первые два выполняются мною всегда))) третий… Ну ООООЧЕНЬ редко)))

[Kakysha]

Ну хотябы корявые слова поправьте скорее, этож страшно читать, а вам должно быть стыдно писать teg и linck, как web-разработчику :(

[hexes]

Судя по тому что ничего не исправлено проект заброшен? Жаль…

[kamaikin]

угу, хотя сейчас идет процесс активного востановления… может быть и до этого плагина руки дойдут…

[wolandino]

Вы молодец! Мне понравилось как это выглядит внешне — возьму на вооружение.
Я как-то собирал подобную вещь для себя — недособирал, пропало время и желание.

[kamaikin]

Спасибо на добром слове))) Тут мне замечаний набросали))) будет немного свободного времени, допишу с учетом замечаний)))

[und]

Я использую вот этот скрипт hexmode.org/tagger_menu/ и очень доволен. Тоже настроил под себя…

[pepelsbey]

teg_linck
teg_image

Определитесь, вы пишете транслитом или по-английски: или tag_image или teg_kartinka, а то какой-то взрыв мозга получается. Хотя проблема может быть сложнее — об этом говорит слово «linck», кажется это «link». Про альт у ссылки уже сказали, но я всё никак отойти не могу o_0

[kamaikin]

да проблема сложнее… в свое время учил японский (через 20 лет уже мало что помню но....) английский не учил никогда… так что приношу извинения спасибо за комментарий.

[r1e]

жаль, что свои комменты нельзя удалять :)

[ChemAli]

Нет предпросмотра.

[kamaikin]

я конечно понимаю, что краткость сестра талана, но не все такие талантливые…

какого именно предосмотра нет? если работы плагина то — Посмотреть пример, пощелкать по кнопочкам настройки

[13i]

предпросмотр — это значит нажать на одноименную кнопочку и посмотреть как это будет выглядеть в реальности в браузере.
Можно:
— открыть в новом окне
— разместить div-окно под редактором (вроде лучшее решение)
— заменить редактор на окно предпросмотра и добавить ссылку «Вернуться к редактированию»
— возможно что-то еще…

[kamaikin]

ок принято.