Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Chat sky mp #4

Merged
merged 34 commits into from
Apr 25, 2019
Merged

Chat sky mp #4

merged 34 commits into from
Apr 25, 2019

Conversation

rrov
Copy link
Contributor

@rrov rrov commented Apr 23, 2019 •
edited by Pospelove
Loading

resolve #1
Доделываю:

Исчезновение при скроле.
Одноразовая анимация появления сообщения

Exhorts added 11 commits April 23, 2019 11:13
chat
3f757b3 
Доделать:
1. Исчезновение при скроле
2. Цвета авторам
chat
ae88d64
chat
ca5e75a
chat
b6f44e1
chat
6003516
chat
2f07158 
Добавлены рандомные цвета ников игроков, которые пишут в чат
chat
bf59409
chat
0b1eb79 
Переход после удаления на общий чат
@Pospelove
Add Chat
5ec22d9
chat
3d2e48f
add
7e9c459
@Pospelove Pospelove added the workflow:wip Pull request не готов, идёт работа. label Apr 23, 2019
Exhorts added 9 commits April 23, 2019 20:39
chat
262a2fd 
Сделал одиночное появление сообщения
chat
d03dd43 
Сделал отдаление записей у верха вьюпорта чата.
chat
d03752e 
убрал лишнее
Update chat.js
316de9f
Update chat.js
b093df0
Update chat.js
73d60f3
Update chat.css
7436c3f
new svg
5334c74 
Добавил отредактированный в Vectr svg файл, увеличил его так, чтобы еще входило смещение при появлении второй полосы менюхи.
Update chat.css
82e607c
@Pospelove Pospelove added workflow:in-review Pull request проходит рассмотрение разработчиками. and removed workflow:wip Pull request не готов, идёт работа. labels Apr 24, 2019
@Pospelove
Copy link
Owner

@Exhorts Благодарю за проделанную работу. Перед тем, как я сделаю merge, пожалуйста, исправьте следующие проблемы:

  1. Анимация отдаления сообщений должна выглядеть, как показано здесь.
  2. Если это возможно, исправить выпирающие чёрные углы.
  3. Выделенные вкладки чата не должны заходить за линию.
  4. Необходимо скроллить чат вниз при появлении нового сообщения в любой из вкладок, кроме Общий чат.
  5. Необходимо сделать поле ввода менее прозрачным, чем весь остальной чат. См. здесь.
  6. Табуляция в chat.js должна быть корректной. Исправить табуляцию быстро можно тут. Обратите внимание, что нужно 2 пробела вместо 1 таба (опция Ident with 2 spaces).

@Pospelove Pospelove added workflow:wip Pull request не готов, идёт работа. and removed workflow:in-review Pull request проходит рассмотрение разработчиками. labels Apr 24, 2019
Exhorts added 2 commits April 24, 2019 20:53
Chat
b3b5a71 
1. Сделано.
2. Надо менять svg-фон, либо делать собственный фон с svg углами (заплатки в виде полос при разных мониторах могут выдавать либо пробелы (белы линии) между основным фоном и полосой, либо наоборот наложение прозрачных объектов, что = более темной полосе.
3. Сделано.
4. Сделано.
5. Сделано.
6. Сделано.
chat
d2b4b2d
@Pospelove Pospelove added workflow:in-review Pull request проходит рассмотрение разработчиками. and removed workflow:wip Pull request не готов, идёт работа. labels Apr 24, 2019
chat
af9a1b2 
Лучше видно анимацию на однострочных сообщениях
@Pospelove
Copy link
Owner

@Exhorts
При появлении нового сообщения чат всё ещё не скроллится вниз автоматически.
Анимация отдаления сообщений должна начинаться со 2 сообщения сверху (подчеркнул)
image

Также, пожалуйста, удалите front/js/chat-oldcode.js.

@Pospelove Pospelove added workflow:wip Pull request не готов, идёт работа. and removed workflow:in-review Pull request проходит рассмотрение разработчиками. labels Apr 24, 2019
Exhorts added 9 commits April 24, 2019 21:37
Delete chat-oldcode.js
32b3949
Update chat.js
9ddda95 
Скрол починил
Update chat.js
062ee19
Update chat.js
87bbf8f
Scroll animation
b972f5c 
Переделал анимацию исчезновения сообщений
chat
2512111
chat
f472d25
Update chat.css
7eecc4a
chat
83120c7 
захватывает 3 сообщения
@Pospelove Pospelove added workflow:in-review Pull request проходит рассмотрение разработчиками. and removed workflow:wip Pull request не готов, идёт работа. labels Apr 25, 2019
@Pospelove
Copy link
Owner

@Exhorts
Всё отлично работает. Единственное, что необходимо - сделать, чтобы анимация отдаления не проигрывалась, если все сообщения влезают на страницу чата. Иначе получается такое:
image

@Pospelove Pospelove added workflow:wip Pull request не готов, идёт работа. and removed workflow:in-review Pull request проходит рассмотрение разработчиками. labels Apr 25, 2019
@Pospelove Pospelove added workflow:in-review Pull request проходит рассмотрение разработчиками. and removed workflow:wip Pull request не готов, идёт работа. labels Apr 25, 2019
@Pospelove Pospelove merged commit e8798e0 into Pospelove:master Apr 25, 2019
@Pospelove Pospelove removed the workflow:in-review Pull request проходит рассмотрение разработчиками. label Apr 25, 2019
@Pospelove
Copy link
Owner

@Exhorts Благодарю за выполненную работу!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Вёрстка: Чат
2 participants
@rrov @Pospelove