search

DevGroup-ru / dotplant2

E-Commerce CMS - Yii Framework 2 (yii2, shop)
http://dotplant.ru/
Other
641 stars 252 forks source link

Slider #286

Closed angoliar closed 8 years ago

angoliar commented 8 years ago

add main currency to price slider

angoliar commented 8 years ago

Нашел еще баг.. Доработаю еще..

angoliar commented 8 years ago

Вроде так...

angoliar commented 8 years ago

а чему будет равно accuracy?

ivansal commented 8 years ago

100,1000,10000 лучше брать из настроек валюты max_fraction_digits

angoliar commented 8 years ago

Добавил инициализацию слайдера после фильтрации..

angoliar commented 8 years ago

"100,1000,10000 лучше брать из настроек валюты max_fraction_digits" Я просто currency.max_fraction_digits написал, а нужно было, наверное, POW(10, currency.max_fraction_digits)... Плохо когда currency.max_fraction_digits = 0. Как мне теперь это исправить? нужно новый PR делать, или сюда можно?

ivansal commented 8 years ago

исправил в коммите f95cc0fc7b6560faaf1402925251a4d1dbf0ac3f

angoliar commented 8 years ago

Спасибо

Philosoft commented 8 years ago

это уже неактуально, но @angoliar пожалуйста, в следующих PR, которые касаются 1 фичи и содержат внутри себя как минимум коммиты вида 'fix'

  1. ставьте метку [WIP], пока PR не принял окончательный вид
  2. перед снятием [WIP] "скваш" коммиты в 1-2