atom icon indicating copy to clipboard operation
atom copied to clipboard

[Frontend] Переношу свои наработки из проекта

Open devoln opened this issue 8 years ago • 4 comments
trafficstars

  1. Сделал траву как фон canvas вместо серого цвета. Теперь её не нужно делать тайлами.
  2. Заменил текстуру травы на более красивую и качественную.
  3. Изменил конструктор Player так, чтобы игроки создавались с разными спрайтами: george.png, betty.png, betty2.png. Поправил remove, чтобы он удалял из массива players.
  4. Аналогично поправил remove для Fire и Bonus. Исправил конструктор Bonus, чтобы он принимал id и позицию в пикселях, как и другие объекты.
  5. Убрал лишние поля в Bomb, аналогично исправил remove и добавил метод explodeSound. В самом клиенте не используется, но используется у меня при обработке сообщения из реплики о взрыве бомбы.

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

devoln avatar May 19 '17 11:05 devoln

У бомбы spriteSheet.regX и regY менял то ли на -1 то ли на 0. Из-за того что спрайт 28∗28, насколько я полагаю, и если позиционировать его как и остальные в узлах (m∗32; n∗32) бомба смотрится кривовато. У вас такого не было?

Russiancold avatar May 19 '17 17:05 Russiancold

Я на сервере их ставил на четверть тайла правее и ниже, и смотрелось, как в оригинале bombergirl.

devoln avatar May 19 '17 17:05 devoln

@gammaker Хотел написать как раз на эту тему, но забыл. Наверное, лучше если позиционирование на сервере будет одинаковым (если, конечно, не учитывать, что оно у нас и так кривое).

Russiancold avatar May 19 '17 17:05 Russiancold

круто! поревьюим и возьмем

Al-p-i avatar May 23 '17 18:05 Al-p-i