-
-
Notifications
You must be signed in to change notification settings - Fork 119
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
When there are too many cigarettes, show 2 full lines #440
Comments
Hey @amaurymartiny I would like to work on this issue. Also, I would like to have a bit more information about what remains constant, the number of maximum cigarettes filling two rows or the width of the cigarettes |
Hey @sarthakpranesh, thanks for stepping up! Go ahead for this issue, I assigned you.
What remains constant:
What is variable:
One more thing: I recently moved this Cigarette component to another repo, so that it can be re-used, it's here now: https://github.com/shootismoke/common/blob/master/packages/ui/src/Cigarettes/Cigarettes.tsx. This may make testing this component a bit harder, let me know if you need help. |
Right now, when the number of cigarettes is >50, we limit to show only 50 cigarettes:
https://github.com/amaurymartiny/shoot-i-smoke/blob/f1c67a4cf04d87d9bc82b3414453f582e8b18c46/App/components/Cigarettes/Cigarettes.tsx#L54-L55
However, it's 40 cigarettes and 180 cigarettes seem to have the same number of cigarettes:
![Screenshot_20200205-183718](https://user-images.githubusercontent.com/1293565/73891844-77698900-4875-11ea-858a-24eca00af8d0.png)
The small challenge is we don't know how many cigarettes to show to fill 2 lines, it depends on the phone's screen size. On my small phone, 50 almost fills 2 lines. On the above one, it seems like we need ~64.
What we can probably do is
The text was updated successfully, but these errors were encountered: