Skip to content
This repository was archived by the owner on Dec 1, 2023. It is now read-only.

create toggle, add bets and multipliers #2

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

SergeyBersenev98
Copy link
Contributor

Time must be fixed. It isn't ready yet

Separate css-style file in modules. Create filter method. Delete not-necessary variables and arrays. Add data-format, add score. Create new component Match.vue
Time must be fixed. It isn't ready yet
<div>
<div class="match__time">
<span v-if="this.time">{{ this.time | moment(`mm:ss`)}}</span>
<span v-else>{{ match.startDate | moment(`D MMM hh:mm`)}}</span>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

убрать дату и время. Указать 0

<!-- /.search__input -->
<label class="search__toggle">
<input type="checkbox" v-model="checkboxClicked">
<span class="search__toggle-switch"></span>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Вёрстка поплыла





Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

что за бездна?)

@rah-emil
Copy link
Contributor

rah-emil commented Feb 1, 2022

  • при разрешении 1368px по 1440px и 1920px плывут имена команд
  • анимация состояний тоглера LIVE должна меняться не по наведению, а по фокусу (это показано в макете)
  • поле поиска не соответствует макету. Присутствует outline + при фокусировке дёргается placeholder
  • дата и время momemt js не локализировано по Мск + нет перевода на русский
  • вся линия матча должна быть ссылкой на матч на ставку, а трансляция должна создавтаь видимость ссылки, но по факту не ссылка
  • по наведению на тогглер не показывается pointer

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants