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.

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

Add party #8

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Add party #8

wants to merge 3 commits into from

Conversation

wnex
Copy link

@wnex wnex commented Apr 29, 2019

@Pospelove Pospelove added the workflow:in-review Pull request проходит рассмотрение разработчиками. label Apr 29, 2019
@Pospelove
Copy link
Owner

@neXOnion Благодарю за проделанную работу! Перед тем, как я сделаю merge, необходимо сделать следующие вещи:

  1. Добавить контроль имен членов шайки и лидера с помощью переменной group.
// На данный момент реализовано только обновление маны и здоровья,
// Но лидер, состав и кол-во участников тоже могут меняться в ходе игры
// Нужно, чтобы можно было передать что-то наподобие:
var group = [
  {name: 'Pospelov', heal: 100, mana: 50, isLeader: true},
  {name: 'Sondragon', heal: 40, mana: 20},
  {name: 'Leos', heal: 10, mana: 80},
  {name: 'Somebody', heal: 1, mana: 1},
];
updateParty(group); 
// Все данные должны обновиться
  1. Менять размер страницы в зависимости от размера окна. Это нужно, чтобы не получать следующую картину в игре:
    image
    Как видно на скриншоте, из-за того, что фактический размер браузерного окна меньше, чем размер страницы, появляется горизонтальный скролл. Нужно, чтобы его не было. Ну и фоновое изображение тоже убрать.

@Pospelove Pospelove added workflow:wip Pull request не готов, идёт работа. and removed workflow:in-review Pull request проходит рассмотрение разработчиками. labels Apr 29, 2019
@wnex
Copy link
Author

wnex commented Apr 30, 2019

@Pospelove убрал тестовые данные, доработал скрипт, поправил стили (отступы и полосы скролла).

Пример:

group.update([
	{name: 'Pospelov', heal: 100, mana: 50, isLeader: true},
	{name: 'Sondragon', heal: 40, mana: 20},
	{name: 'Leos', heal: 10, mana: 80},
]);

Звёздочка (*) обязательный параметр.

name* - отображаемое имя;
heal* - процент заполненности полосы жизней от 0-100 включительно;
mana* - процент заполненности полосы маны от 0-100 включительно;
isLeader - bool, отображение иконки лидера группы, если не передан считается как false.

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

@neXOnion
Качество иконки лидерства испортилось после исправлений:
image

@Pospelove Pospelove added workflow:wip Pull request не готов, идёт работа. and removed workflow:in-review Pull request проходит рассмотрение разработчиками. labels Apr 30, 2019
@wnex
Copy link
Author

wnex commented Apr 30, 2019

@Pospelove перепроверил иконку, у меня отображается как и раньше. Возможно причина в способе наложения на изображение.

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

@neXOnion Верно. Проблема была в фоне.
image
Сообщил дизайнеру, чтобы он придумал что-то. Ждём.

@Pospelove Pospelove added workflow:wip-skymp-team Команда SkyMP работает над этим. and removed workflow:in-review Pull request проходит рассмотрение разработчиками. labels Apr 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
workflow:wip-skymp-team Команда SkyMP работает над этим.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Вёрстка: Шайка
2 participants