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

Mobile poll design #453

Merged
merged 4 commits into from
Mar 19, 2013
Merged

Mobile poll design #453

merged 4 commits into from
Mar 19, 2013

Conversation

hizel
Copy link
Contributor

@hizel hizel commented Mar 19, 2013

Видиво: http://www.youtube.com/watch?v=1PvXUFITktk

Для мобильного размера экрана вместо пингвинов, просто полосочки. Прасти @andrux0id !

maxcom added a commit that referenced this pull request Mar 19, 2013
@maxcom maxcom merged commit e29a329 into maxcom:master Mar 19, 2013
@hizel
Copy link
Contributor Author

hizel commented Mar 19, 2013

До меня дошло, зачем использовался maxVote. Что-бы длинна варианта который набрал максимальное кол-во голосов была на всю ширину опроса. Чтобы график был более выразителен чтоли. Но тогода 50% в одном опросе отрисуются по разному в разных опросах. Хорошо ли это.

@maxcom
Copy link
Owner

maxcom commented Mar 19, 2013

выкатил на сайт. А нельзя сделать чтобы в десктопном варианте пингвины были на той же строке что и текст варианта?

@maxcom
Copy link
Owner

maxcom commented Mar 19, 2013

при просмотре неподтвержденных:

java.lang.ArithmeticException: / by zero
        at ru.org.linux.poll.PreparedPoll.<init>(PreparedPoll.java:50)
        at ru.org.linux.poll.PollPrepareService.preparePoll(PollPrepareService.java:45)
        at ru.org.linux.topic.TopicPrepareService.prepareMessage(TopicPrepareService.java:176)
        at ru.org.linux.topic.TopicPrepareService.prepareMessagesForUser(TopicPrepareService.java:323)
        at ru.org.linux.topic.TopicListController.viewAll(TopicListController.java:631)

@maxcom
Copy link
Owner

maxcom commented Mar 19, 2013

пофиксил деление на ноль

@hizel
Copy link
Contributor Author

hizel commented Mar 19, 2013

Надо будет тест написать (

@hizel hizel deleted the mobile-poll-design branch March 19, 2013 19:39
@hizel
Copy link
Contributor Author

hizel commented Mar 20, 2013

Если делать в одну строчку, получается примерно так: http://youtu.be/SacGebIelME

@maxcom
Copy link
Owner

maxcom commented Mar 20, 2013

может синие полоски с планшетной ширины включать?

@hizel
Copy link
Contributor Author

hizel commented Mar 20, 2013

ну да, можно и так, вот #455

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.

None yet

2 participants