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

49 time periods selector #52

Merged
merged 5 commits into from
Jan 23, 2016
Merged

Conversation

michaelkl
Copy link
Contributor

Переход по периодам кнопками со стрелочками. Корректная обработка периода "месяц"

@pihel
Copy link
Owner

pihel commented Jan 22, 2016

Спасибо, протестировать смогу только в выходные, наверно.

pihel added a commit that referenced this pull request Jan 23, 2016
@pihel pihel merged commit bc5fc9d into pihel:master Jan 23, 2016
@pihel
Copy link
Owner

pihel commented Jan 23, 2016

Михаил, спасибо за активность :)

Проверил, все хорошо.
Внес только небольшие изменения label кнопок, и по смещению дат: 4636d88

Задачу #49 можно считать выполненной в упрощенном варианте?

@michaelkl
Copy link
Contributor Author

Хорошо.
А почему поменяли на var ddiff = d2 - d1 + 1;?

По поводу закрытия задачи пока предлагаю не спешить. Я хочу еще доработать, чтобы переход вперёд/назад понимал не только один полный месяц, но и N полных месяцев. Таким макаром мы покроем варианты "квартал" и "год", и бонусом экзотику типа "два месяца", "шесть месяцев" и пр. :-)

@michaelkl
Copy link
Contributor Author

Ещё бы придумать, как быстро и удобно выставлять интервал и тогда совсем хорошо будет. :)

@pihel
Copy link
Owner

pihel commented Jan 23, 2016

А почему поменяли на var ddiff = d2 - d1 + 1;?

Без +1 переход захватывал 1 день из предыдущего интервала.
Сейчас если выбрать неделю и нажать на следующий интервал, то мы сместимся ровно на другую неделю. Другой пример: если выбран 1 день, то смещения на 1 день без +1 вообще не происходило.
Можно в демке посмотреть:
http://demo.homebuh.pro/#act=list&from=2016-01-11&to=2016-01-17

@pihel
Copy link
Owner

pihel commented Jan 23, 2016

Я хочу еще доработать, чтобы переход вперёд/назад понимал не только один полный месяц, но и N полных месяцев.

Не очень ясно, зачем такой функционал? Когда в гриде уже данные за неделю, анализ провести почти не возможно.
Если хочется сравнивать периоды между собой, то возможно стоит сделать/доработать отчет в аналитике.

@michaelkl
Copy link
Contributor Author

Не очень ясно, зачем такой функционал? Когда в гриде уже данные за неделю, анализ провести почти не возможно.

Ну, это я пытаюсь получить аналог окна операций в Personal Finances. :-)
Если бы удалось группировать операции по категориям товаров с подсчётом подитога, это было бы удобно и в длинных периодах был бы смысл. Сразу видны расходы и доходы за месяц и не надо переходить в отчёты. Вот, посмотрите пример:

image

@pihel
Copy link
Owner

pihel commented Jan 23, 2016

Да, я понял откуда эта идея.
По группам товаров: проще, мне кажется, расширить функционал отчета по группам товара, чем усложнять линейный список операций.

@michaelkl
Copy link
Contributor Author

По группам товаров: проще, мне кажется, расширить функционал отчета по группам товара, чем усложнять линейный список операций.

Да, согласен. Мне в PF это удобно, потому что другими отчётами я не пользуюсь, а этот отчёт доступен сразу в списке. Давайте тогда пока что действительно не будем продолжать усложнять, посмотрим, как пойдёт. Может, позже вернёмся в этому вопросу.

Ошибку нашел. Шаг вперёд неправильно работает. Выбираем неделю 2015-12-14 - 2015-12-20, жмём "Вперёд", получаем 2015-12-20 - 2015-12-26.

@michaelkl michaelkl deleted the 49-time-periods-selector branch January 23, 2016 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants