Модераторы: Sardar, Aliance
  

Поиск:

Ответ в темуСоздание новой темы Создание опроса
> Jquery Bbcode редактор, что улучшить? 
:(
    Опции темы
kamaikin
Дата 16.10.2009, 08:45 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Новичок



Профиль
Группа: Участник
Сообщений: 4
Регистрация: 5.9.2007

Репутация: нет
Всего: нет



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

Прочитать описание плагина, порядок его использования и настройки
Посмотреть пример, пощелкать по кнопочкам настройки
И конечно, скачать плагин для использования или ознакомления

Это сообщение отредактировал(а) kamaikin - 16.10.2009, 08:47
PM MAIL WWW   Вверх
icewind
Дата 16.10.2009, 09:09 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 393
Регистрация: 11.4.2009
Где: Ростов-на-Дону

Репутация: нет
Всего: 20



На вскидку... Не плохо было бы запрашивать урл при вставке линка в диалоге. Не нужно будет отдельно курсор возвращать и вписывать


--------------------
user posted image
PM MAIL Skype   Вверх
kamaikin
Дата 16.10.2009, 09:29 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Новичок



Профиль
Группа: Участник
Сообщений: 4
Регистрация: 5.9.2007

Репутация: нет
Всего: нет



Цитата(icewind @ 16.10.2009,  09:09)
На вскидку... Не плохо было бы запрашивать урл при вставке линка в диалоге. Не нужно будет отдельно курсор возвращать и вписывать

как на этом сайте?
PM MAIL WWW   Вверх
IDVsbruck
Дата 16.10.2009, 10:41 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 778
Регистрация: 12.11.2006

Репутация: 4
Всего: 22



Да а что там улучшать? Код-то коротенький ...

Правда, а я не понимаю зачем, к примеру, такой изврат:
Код
var id = '#' + $(this).attr("id");
var e = $(id).get(0);

Шутка?
Код
var e = $(this).get(0);


Потом: в плагине определяется типа глобальная переменная е ("типа" - потому что не глобальная), а потом используется в обработчике клика. Думаю, работает, иначе бы что-то делал ... но так не правильно. Код может быть пользователем значительно расширен и тогда появится "более глобальная" переменная е и обработчик клика будет делать уже совсем другое. Таки вещи надо делать сразу и следить за ними:
Код
$('#bbcode_bb_bar a').bind("click", {e:e}, function(event) {/* вместо е теперь используем event.data.e */});


Логику с начальными if then else пересмотрел бы. Для начального уровня это нормально, но для серьезного плагина мелковато.

Не буду утверждать, но по-моему, в последней функции реализация с createRange не кроссбраузерная. Редко использую, но по-моему, чего-то не хватает.
PM MAIL   Вверх
kamaikin
Дата 16.10.2009, 10:48 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Новичок



Профиль
Группа: Участник
Сообщений: 4
Регистрация: 5.9.2007

Репутация: нет
Всего: нет



Цитата(IDVsbruck @ 16.10.2009,  10:41)
Шутка?
Код
var e = $(this).get(0);


нет не шутка...
избавлялся от
Код

document.getElementById('user_text');

может быть не очень удачно....
от if - else я подумаю как избавится ))
PM MAIL WWW   Вверх
brother79
Дата 28.10.2009, 09:26 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 471
Регистрация: 18.8.2009

Репутация: 1
Всего: 6



Цитата(kamaikin @  16.10.2009,  10:48 Найти цитируемый пост)
Цитата(IDVsbruck @ 16.10.2009,  10:41)
Шутка?
Выделить всёкод Javascript
1:
    
var e = $(this).get(0);


нет не шутка...
избавлялся от



Зачем тебе вообще get(0) ? Ты же jQuery юзаешь, я что-то не представлю ситуацию, которую имело бы смысл делать через стандартный ДОМ.


--------------------
PM MAIL WWW   Вверх
  
Ответ в темуСоздание новой темы Создание опроса
0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема | JavaScript: Применение библиотек | Следующая тема »


 




[ Время генерации скрипта: 0.0439 ]   [ Использовано запросов: 22 ]   [ GZIP включён ]


Реклама на сайте     Информационное спонсорство

 
По вопросам размещения рекламы пишите на vladimir(sobaka)vingrad.ru
Отказ от ответственности     Powered by Invision Power Board(R) 1.3 © 2003  IPS, Inc.