taliban 28 ноя 2009 в 20:54

Хабра-редактор комментариев ноль.два

2 мин

696

Habr

+40

Комментарии 95

oe24 28 ноя 2009 в 21:10

Спасибо за улучшение!..
Очень удобно. Обновлюсь…

homm 28 ноя 2009 в 21:24

Кнопки © ® « » ±, а также ™ € ≠ × ° § ≈ и прочие не нужны.
Нужна правильная раскладка, которая работает во всех приложениях, а не на одном сайте.

-5

taliban 28 ноя 2009 в 22:22

а мне хочется самомурасставлять кавычки типа «начало текста «середина» конец текста» а не смотреть как хабр делает «начало текста» середина «конец текста»

-5

homm 28 ноя 2009 в 22:25

Вы вообще о чем? Кого вам расставляет хабр? Уверены, что по ссылке ходили?

taliban 28 ноя 2009 в 22:46

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

-3

homm 28 ноя 2009 в 23:48

Что-то мне кажется, ваше «неожидание» несколько преувеличенно. Знак дюйма (а не «обычная двойная кавычка», как вы её назвали) заменяется на открывающую кавычку, если перед ним стоит пробел и на закрывающую, если после. Знак дюйма не меняется на кавычку, если стоит после цифр, т.е. используется по назначению: 19"

taliban 29 ноя 2009 в 01:52

=) я не знал, спаибо, а кнопочки всеравно люди заменят все под себя

GooRoo 29 ноя 2009 в 21:04

Эм, ну кагбэ, если я не ошибаюсь, то это не знак дюйма, так как для обозначения дюймов (и секунд в градусной мере) все-таки существует отдельный символ (\u2033), как например: 19″ (сравните с английскими кавычками “” и символом, который у всех нас на клавиатуре, — ". В данном шрифте никакой разницы, но в окне набора комментария у меня они все выглядят по-разному).

taliban 28 ноя 2009 в 22:25

там кстати еще будет нюанс с кириллицей, получится на одну клавишу 3 символа, мне кажется это будет неудобно обыным пользователям типа меня

-1

homm 28 ноя 2009 в 22:29

Я опять не понял. Какой нюанс?
Про три символа: дак до четырех, на самом деле. Вам кажется удобнее вводить символы текста с помощью кнопок на экране?

Hanhe 28 ноя 2009 в 22:59

Мне, например, нахрен не сдалась типографская раскладка. Искать на ней символы я буду дольше, чем тыкать в кнопочки на форме.
Учить специально кнопки этой раскладки я, конечно, не буду, а сами они не выучатся — слишком редко нужны(мне).
+ еще наклейки где-то взять надо(на лазать же каждый раз на сайт с описанием и смотреть где что находится).

homm 28 ноя 2009 в 23:44

Вам «нахрен не сдалась» типографская раскладка, тем не менее, вы собираетесь тыкать в кнопочки на экране. Я вполне допускаю, что есть огромный пласт людей, которым «нахрен не сдались» кавычки (так же как другим, знаки припинания, например). Но я так же не понимаю, зачем этим людям кнопочки, в которые они будут тыкать.

Skif300000 28 ноя 2009 в 22:53

Спасибо удобная вещь!

Boomburum 28 ноя 2009 в 23:01

А сразу js-файлик нельзя было вставить? )
Я скопировал код (делал много раз, все было ок), но кнопок не появилось…

taliban 28 ноя 2009 в 23:57

вставил прямую ссылку

Skif300000 28 ноя 2009 в 23:01

А как бы мне вместо "±" смайл добавить? ":)"

taliban 28 ноя 2009 в 23:56

обновил

VEG 28 ноя 2009 в 23:16

Почему не указано для каких браузеров предназначен скрипт? Судя по предыдущей версии — то только для Оперы. Я прав?

taliban 28 ноя 2009 в 23:56

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

Omgovich 28 ноя 2009 в 23:23

Только для Оперы? Можно легко Экстеншн для Хрома сделать.

Omgovich 28 ноя 2009 в 23:28

Собственно, сейчас этим и займусь, если автор не против.

Omgovich 28 ноя 2009 в 23:53

Скрипт оказался слабо совместим с Хромом =) Убрал if для проверки локации, а тулбар все равно не отрисовывается. Проще с нуля сделать наверное, чем искать в чем проблема.

taliban 28 ноя 2009 в 23:55

поидее нужно запустить его ручками: Toolbar.draw()
у меня он автоматом запускается при загрузке странички

SKYnv 28 ноя 2009 в 23:25

шапку надо чуть шире ) а то эти пара пикселей режут глаза )
при наведении на кнопки вылезает подсказка — undefined

taliban 28 ноя 2009 в 23:53

подсказку поправлю, а вот шапку пошире увы, все берется из стандартных хабровских стилей, получается либо больше, либо на пару пикселей меньше, это в todo…

Dlussky 29 ноя 2009 в 01:18

Как поправить в «ответе» пока не нашел, но для основного комментария можно по простому заменить 259 строчку на

panel.style.width = '100%';

textarea.style.width = '100%';

Имхо, 95% ширины — не принципиально, и в опере ничему не противоречит, а аккуратность повышается)
Вообще, я сейчас читаю доки к userJs, думаю можно найти доступ к calculated width и все можно будет поправить одной строчкой)

Dlussky 29 ноя 2009 в 01:19

Хотя нет, вру, принципиально, сейчас обратил внимание, что скроллбар немного обрезается.

taliban 29 ноя 2009 в 01:33

ширину нужно выставлять только в процентах, потому как это поле вставляется в разные места, которые то шире, то уже, и может все вообще некрасиво выглядеть, оптимальные варианты 95% и 96% но визуально мне больеш нравится «недобор» чем «перебор»

Dlussky 29 ноя 2009 в 01:37

Вот рабочий вариант.

textarea.parentNode.style.paddingRight = '4px'

panel.style.width = 'auto';

textarea.style.width = '100%';

panel.style.marginRight = '-4px';

Вставлять вместо

panel.style.width = '95%';

Dlussky 29 ноя 2009 в 01:44

Еще можно добавить аккуратную рамочку:

panel.style.border = '1px solid #c8d2df';

panel.style.borderBottom = 'none';

taliban 29 ноя 2009 в 01:49

благодарю, действительно приятней для глаза :)

sgnppv 29 ноя 2009 в 00:40

Спасибо! Только непонятно, зачем специально сделали «кавычки» отдельно, хабрапарсер все равно подменяет автоматом.

taliban 29 ноя 2009 в 01:23

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

Boomburum 29 ноя 2009 в 00:47

«Наконец-то заработало» — подумал Boomburum и плюсанул пост + карму ;)

-2

GooRoo 29 ноя 2009 в 21:09

Вы добавили себе кнопку /me?

lunatik42 29 ноя 2009 в 01:57

А нельзя воткнуть проверку на многоуровневость комментов, чтобы у одиночных комментариев не отрисовывалась надпись «Свернуть»?
И надо наверно определиться со строчными/заглавными буквами в ответить и свернуть. А заодно чуток побольше их разнести в стороны.

Dlussky 29 ноя 2009 в 01:59

Опередил)

taliban 29 ноя 2009 в 02:30

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

Dlussky 29 ноя 2009 в 03:08

Только одно. Проверить, что

replyElements[q].parentNode.parentNode.childNodes[7]!=null

На этом топике не тормозит)

Dlussky 29 ноя 2009 в 03:11

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

taliban 29 ноя 2009 в 03:19

=) как бэ вставят создатели хабра один пробельчик и все ;) я буду делать через гетэлемент, на этом топике не тормозит так как тут 30 каментов, есть другие по 300-500, а работа моего скрипта может:
1 — повесить бразер
2 — хабр может не откликаться на клик по ссылке ответить
3 — не сможете ставить оценку
4 — итд работающее на JS
а может и не, так что лучше сперва протестить на большом дереве, и потом уже внедрять :) не хочу делать поспешных нововведений

Dlussky 29 ноя 2009 в 03:41

=) как бэ поменяют создатели хабра один айдишник и все)
А как вы будете через гетЭлемент делать? Чекать все чайлдноды у li с каментом на tagName==ul? Нагрузка, конечно, возрастет.
Мне все-таки видится наиболее правильным вынести сворачивалку в отдельный плагин и уповать на стабильность хабровских шаблонов в угоду скорости)
В любом случае, сворачивалка гораздо менее критичный функционал, а пользователи, если увидят, что она не работает, посмотрят, не появилась ли новая версия. А вообще, если по-крутому, можно раз в день проверять последнюю версию ajax-запросом и выдавать запрос на обновление, если что. Хотя, конечно, нужно сделать отдельную версию без автообновлений, и вообще как следует это продумать.

Кстати, а кто-нибудь вообще думал раньше об проверки наличия обновлений в userJs? По-моему в таких контекстно-зависимых штуках было бы мегакруто (например, всякие скачивальщики музыки из контакта периодически ломаются и приходится самому на сайт лезть)

taliban 29 ноя 2009 в 04:03

я хотел через getElementsByTagName но там поддетей возвращает, так что пока циклом обойдусь… кстати чаще меняют структуру чем айдишники и классы :)

мне не нравится концепция userJS, мне лень их запускать :( плагины удобней, они сами выполняются :)

Dlussky 29 ноя 2009 в 04:21

эмм. Ваш плагин написан как раз по технологии userJS. И выполняется сам, при загрузке страницы. Разве нет?)

taliban 29 ноя 2009 в 04:32

в опере 2 типа userJS первый цепляется на тублар, как во всех других браузерах, а второй грузится автоматом, вот первый мне и не нравится, те что кроссбраузерные

Dlussky 29 ноя 2009 в 09:31

Те, которые на тулбар — это либо button (фича оперы), либо bookmarklet (добавляется в закладки), а те, что грузятся автоматом — это как раз userJS, насколько я понимаю. И они тоже чуть-чуть кроссбраузерные, ибо загружаются в файерфокс с гризманки. Хотя работают, конечно, слегка по-разному)

taliban 29 ноя 2009 в 04:31

пока будет с циклом, если будет тупить, буду думать, вроде бы пока нормально

taliban 29 ноя 2009 в 04:33

воткнул

Dlussky 29 ноя 2009 в 01:59

Еще неплохо было бы убрать ссылку «свернуть» для комментариев, не имеющих ответов.
*да и вообще, имхо было бы логичнее разнести редактор и сворачивалку на два плагина

fatal 29 ноя 2009 в 13:21

У меня вываливается ошибка:

Javascript
User JS compilation
Syntax error C:\Program Files\Opera\profile\userjs\habrButtons.js: line 135 of User JS script :
Expected token: ')'
=\"red\">"") !== -1) ? lc.substr(lc.lastIndexOf("<font color=\"red\">"")) : null;
-------------------------------------------------^

taliban 29 ноя 2009 в 15:03

залейте скрипт по прямой ссылке, я глупость делал что выложил его на сайте исходников, там преобразовали в " (\«red\»>"") и получилась вот такая двойная ковычка, в ней и бок )

прямая ссылка
я ее постепенно обновляю, так что иногда заходите если все будет хорошо, мож обновиться захотите :)

SKYnv 29 ноя 2009 в 15:46

ну еще шапку добавите, чтоб по канонам )

// ==Userscript==

// @name xxx

// @version xxx

// @date xxxx-xx-xx

// @author xxx

// @download xxx

// @include xxx

Guria 29 ноя 2009 в 19:38

автор шапку добавил, но самый главный пункт include — забыл. Да и переименновать в *.user.js как уже ниже сказано

SKYnv 29 ноя 2009 в 20:45

и первую строчку переименовал, хотя не надо ), так и должно быть "// ==Userscript=="

taliban 29 ноя 2009 в 22:00

не переименовал, так как есть небольшие расхожденя между браузерами, и сразу он не заработает в других, если следующая версия будет кроссбраузерная, обязательно переименую

SKYnv 29 ноя 2009 в 22:48

в user.js переименовывать не надо ) в опере все равно как скрипт обзывается ) главное чтобы он лежал в папке со скриптами. А вот шапочка верная желательна(но не обязательна) )

shifttstas 29 ноя 2009 в 13:51

Переименуйте на *.user.js что бы Chrome и FF сами ставили крипт

Skiminok 29 ноя 2009 в 13:59

Он все равно с Хромом пока несовместим.

shifttstas 29 ноя 2009 в 14:00

Тогда я думаю стоит написать для какого браузера скрипт, и почему нельзя сделать кроссплатформенный?

taliban 29 ноя 2009 в 14:57

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

Guria 29 ноя 2009 в 19:36

Чтобы Хром и Лиса стали их понимали надо добавить полей в заголовке. возможно даже сам скрипт даже допиливать не придётся

pxx 30 ноя 2009 в 17:23

Хром и ФФ может и подхватят автоматом, за то Опера перестанет его понимать. Какой-то глюк присутствует при подхватывании скриптов с .user в имени файла.

GooRoo 29 ноя 2009 в 21:38

{
	type: 'image',
	src: 'http://www.picamatic.com/show/2009/11/29/09/36/6125202_20x20.png',
	title: 'Quote',
	text: '<blockquote>?</blockquote>',
},

taliban 29 ноя 2009 в 21:56

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

GooRoo 29 ноя 2009 в 22:02

Ну я ж Вас и не прошу добавлять это в «официальную» версию. Выложил специально для других — вдруг кому пригодится =) Картинку честно спёр с блоггера и перекрасил в хаброцвет ;)

jcdenton_dx 29 ноя 2009 в 22:24

Честное спасибо!

SKYnv 30 ноя 2009 в 12:32

Кстати, если автор против использования картинок для кнопочек с других доменов, почему бы не использовать base64? )

pxx 30 ноя 2009 в 13:48

Подскажите, что я делаю не так?

1. Сохранил C:/userJs/habr.user.js в UTF-8
2. Прописал C:/userJs в опциях оперы. Включил консоль при ошибках.
3. При перезагрузке страницы никакого тулбара и ошибки:

taliban 30 ноя 2009 в 14:59

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

SKYnv 30 ноя 2009 в 16:41

консоль отключи и попробуй без нее, тулбара нет? Файл сам нормально сохранён?

pxx 30 ноя 2009 в 16:46

Так я ж чего и включил консольку — потому что тулбара не было.
Что значит нормально? Ctrl+C+V в Notepad++, Convert to UTF-8 without BOM, Save. Символы при этой процедуре не пострадали.

SKYnv 30 ноя 2009 в 16:47

ошибок в консоли точно не было? это те что с красным крестиком )

pxx 30 ноя 2009 в 16:49

Нет. Только ворнинги, которые на скрине.
И Опера ж зараза никаких комментариев к ворнингам не пишет. Где? Почему?

SKYnv 30 ноя 2009 в 16:54

бр )) проблема с именем ) habr.user.js не корректное ) habruser.js будет корректно. Видимо опера немного некорректно определяет расширение, никому не лень багрепорт сделать?

pxx 30 ноя 2009 в 16:57

Это капец. Воистину спасибо за наводку. Я был свято уверен, что должно работать. Тем более учитывая то, что я раньше уже подключал юзерскрипты к более старой версии оперы с именами файлов .user.js

SKYnv 30 ноя 2009 в 17:00

тогда с вас багрепрот ) bugs.opera.com

pxx 30 ноя 2009 в 17:11

Done. Только я через визард зарепортил. Некогда регистрироваться.

pxx 30 ноя 2009 в 17:15

И кстати вся магия только в .user.
Скрипт с именем habr.comment.js подхватился.

taliban 30 ноя 2009 в 17:46

ахренеть, спасибо, будем знать :)

SKYnv 30 ноя 2009 в 19:09

будем тогда ждать ответа на ваш репорт, может резервируется где это имя…

pxx 30 ноя 2009 в 19:17

Возникла идея прикрутить типограф от Spearance. Но там есть несколько нюансов для рассмотрения… Есть мысль, что должен получиться следующий маневр:
— у себя на сайте находится прокси, который может общаться с веб-сервисом типографа посредством POST и формировать ответ для тулбара.
— тулбар делает JSONP запрос на прокси и получает оттипографированный текст.

Тут только несколько вопросов:
1. Разрешено ли будет userJS-скрипту произвести JSONP запрос?
2. Насколько это будет тормозить?
3. Не накроет ли проксю или веб-сервис типографа хабраэффектом? :)

Второй и третий вопросы риторические и больше относятся к проверяемым опытным путем. А вот на первый вопрос не знает ли кто ответ?

taliban 30 ноя 2009 в 20:25

=) я просто хотел сделать кнопочки для удобства написания каментов, и сворачивание ненужных веток каментов, типограф более нужный при создании топиков, но для этих целей есть хабра-редактор, хотя он и подглючивает с подсветкой кода, еще проблемма в кросбраузерном аяксе, в принципе на данный момент я знаю только один способ — вставка скрипта в тело документа, но проблемма в том что результат должен быть валиндым JS кодом, иначе возникает ошибка

pxx 1 дек 2009 в 10:37

еще проблемма в кросбраузерном аяксе

Как-то так:

/* Создание нового объекта XMLHttpRequest для общения с Web-сервером */

var xmlHttp = false;

/*@cc_on @*/

/*@if (@_jscript_version >= 5)

try {

  xmlHttp = new ActiveXObject("Msxml2.XMLHTT");

} catch (e) {

  try {

    xmlHttp = new ActiveXObject("Microsoft.XMLHTT");

  } catch (e2) {

    xmlHttp = false;

  }

}

@end @*/



if (!xmlHttp && typeof XMLHttpRequest != 'undefined') {

  xmlHttp = new XMLHttpRequest();

}

SKYnv 1 дек 2009 в 11:09

объявление переменной раскрашено как комент, бажек?

pxx 1 дек 2009 в 11:14

Колорер косит что-то

taliban 3 дек 2009 в 02:22

уже не косит :)

taliban 3 дек 2009 в 02:22

fixed, updated. буду благодарен за очередные багрепоты =)

taliban 1 дек 2009 в 17:24

да, похоже баг, будем разбираться =)
дело в том что обьекту XMLhttpRequest запрещается делать кросдоменные запросы, поэтому не все так просто

SKYnv 30 ноя 2009 в 21:43

ну виджетам разрешено к примеру, нужно проверять опытный путём

falone 2 мар 2010 в 20:11

То ли скачивается криво, то ли просто не работает в релизе Оперы 10.50

taliban 2 мар 2010 в 21:21

в скрипте в самом низу вместо
document.attachEvent('onload', Toolbar.draw);
ставим
document.addEventListener('DOMContentLoaded', Toolbar.draw)
должно все заработать