← Timeline
Avatar placeholder
lamed
Пре-релиз 0.9.0 RC1

Почти три месяца работы - и у нас новая версия Moera. И теперь в ней есть комменты.

Официальный анонс будет, как всегда, после обновления документации.

To react or comment  View in Web Client
Comments (75)
Avatar placeholder

В коде есть. Но, видимо, что-то не совпадает в создании подписи и в проверке.

Avatar placeholder

Я починил всплывашки на Chrome. Проверь, как на мобильнике.

Avatar placeholder
Avatar placeholder

Ещё две вещи:

  1. Надо сказать юзеру @toshick обновить ноду, а то с его нынешней ожидаемо не заводится новый клиент.
  2. Стоит отредактировать твой пост, чтобы @-=Barshay=- наконец распарсился как белый человек
Avatar placeholder

кстати, если открыть ссылку на @-=Barshay=- в новой вкладке, то сервер возвращает 404

Avatar placeholder

поправка, вне зависимости от юзера генерируется ссылка вида https://client.moera.org/moera/gotoname?name=eliduvid_0 которая возвращает 404 если её открыть в новой вкладке

Avatar placeholder

Фичу "Reply as post" я бы тоже оставил. Это ответ для всех подписчиков + в какой-то степени это фича "репост".

Avatar placeholder

Посмотрел - она и осталась, в меню поста - но в этом случае не работает "выделить и ответить, отквотив выделенное". Возможно, стоит в настройках клиента регулировать, размещать ли кнопку "Reply as post" под постами: Always, Only if no comments allowed, Never.

Avatar placeholder

ну что, окропим снежок.... проверим, как работают комментарии. Надеюсь, это станет началом большого флуда, а может даже и холливара )))

👍1
Avatar placeholder

пятидесятый комментарий не может быть началом флуда, только продолжением

Avatar placeholder

но в этом случае не работает "выделить и ответить, отквотив выделенное".

Да, я знаю и думаю, что с этим делать. Зато в комментах это работает 😉

👍1
Avatar placeholder
Avatar placeholder

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

Avatar placeholder

Да, заменяется, когда клиент получает ответ от нейминга. Но почему непрямые ссылки у тебя с client.moera.org? Они должны были быть с хостом твоей домашней ноды.

Avatar placeholder
Avatar placeholder

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

Avatar placeholder

Я и по логам вижу. Но очень странно :-O

Avatar placeholder

@lamed, а что с выплывашками было? цсс слегка плохой?

Avatar placeholder

Я их вставлял через ReactDOM.createPortal() в отдельный div, который у меня для всяких диалогов предназначен. Но оказалось, что если они там, то в Chrome позиционируются неправильно.

🤦1
Avatar placeholder

я не претендую на первенство )) тут олимпийский принцип, главное - участие

👍3
Avatar placeholder

Попробуй еще раз с ответами и редактированием ответов. Я одну вещь исправил, но, может, еще что-то есть.

Avatar placeholder

работает. а что поломалось?

Avatar placeholder

Он брал repliedToId из переданного в запросе, а при редактировании надо было брать из самого коммента. В контроллере это учтено, а в вычислении подписи я забыл.

👍1
Avatar placeholder

Мне непонятно, как работает пейджинг комментариев. То есть вроде как работает (фейсбуко-подобное "Load earlier comments"/ "Load more comments"), но вот что означают при этом цифры и кнопки "промотать вперёд/назад" - неясно.

Avatar placeholder

к первому комментарию и к последнему комментарию соответственно. очень удобно, кмк

👍1
Avatar placeholder

И в обоих случаях пишется "20 of 65". Непонятно, изменилось ли что-то при нажатии на кнопку перемотки. Мне было бы понятнее "1..20 of 65" и "46-65 of 65".

Когда показаны последние 20, то вверху обе кнопки перемотки, а внизу одна (и этот нижний вариант правильный, как я думаю). Или неактивную кнопку надо показывать в начале/в конце.

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

Avatar placeholder

И в обоих случаях пишется "20 of 65". Непонятно, изменилось ли что-то при нажатии на кнопку перемотки. Мне было бы понятнее "1..20 of 65" и "46-65 of 65".

Хорошая мысль.

Когда показаны последние 20, то вверху обе кнопки перемотки, а внизу одна (и этот нижний вариант правильный, как я думаю). Или неактивную кнопку надо показывать в начале/в конце.

Оба варианта правильные. Если показываются комменты 40..60 из 60, то внизу кнопка "в конец" не нужна, мы и так в конце. Но в начале она нужна. При это в начале нужна кнопка "в начало", ведь начало - это 40-й комментарий, а не первый.

И ещё непонятно, как показать часть комментов в середине, но не с всю кучу с начала.

Если открыть ссылку на какой-то комментарий из середины, увидишь комменты в середине.

👍1
Avatar placeholder

Если показываются комменты 40..60 из 60, то внизу кнопка "в конец" не нужна, мы и так в конце. Но в начале она нужна. При это в начале нужна кнопка "в начало", ведь начало - это 40-й комментарий, а не первый.

А. Понял. "Конец" и "начало" не в терминах "страниц", а вообще. Как по мне, навигация по страницам - это одна задача, а прыжки в рамках страницы - другая и элементы управления должны быть отдельные: по страницам - влево-вправо, в рамках одной страницы - вверх-вниз (но и это смущать будет). Но это, возможно, вкусовщина.

Avatar placeholder

Если открыть ссылку на какой-то комментарий из середины, увидишь комменты в середине.

Ссылкой на комментарий показывать те, что рядом с ним - как по мне, неплохая идея.

Я имел в виду юз кейс "листать комменты последовательно, когда их тысячи".

Avatar placeholder

Я имел в виду юз кейс "листать комменты последовательно, когда их тысячи".

Убирать те, которые вверху, по мере того, как продвигаешься вниз? Это можно, но иногда пользователю нужно противоположное. И непонятно, как понять, как именно нужно.

Avatar placeholder

Ещё пожелание - сделать поле ввода комментария более явно выделяющимся. Например, фоном.

Avatar placeholder
Avatar placeholder

Да, листать страницы, убирая верхние. Я согласен, что это нужно не всегда. И в общем, с этой проблемой реально мы скорее всего столкнёмся не скоро.

Avatar placeholder

Не яркий. Примерно такой интенсивности, как любая из сот вокруг центральной белой соты здесь: https://www.w3schools.com/colors/colors_picker.asp - или ещё чуть бледнее.
По-моему, и фоны такие довольно бросаются в глаза, и контраст с чёрным текстом достаточно сильный. Ну или как альтернатива более заметная рамка, в т.ч. когда не в фокусе.

Avatar placeholder

И в обоих случаях пишется "20 of 65". Непонятно, изменилось ли что-то при нажатии на кнопку перемотки. Мне было бы понятнее "1..20 of 65" и "46-65 of 65".

Вот, я сделал.

👍2
To react or comment  View in Web Client