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.

Sign up for GitHub

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

Jump to bottom

Roll multiple dices at the same time #6

Open
n-apps opened this issue Oct 2, 2018 · 5 comments
Open

Roll multiple dices at the same time #6

n-apps opened this issue Oct 2, 2018 · 5 comments
Labels
enhancement New feature or request

Comments

@n-apps
Copy link
Owner

n-apps commented Oct 2, 2018

User can choose the amount of dice that they want to roll with possibility to choose their type (d6,d8,d20 see settings tab). App shows the numbers on each dice and also the total.
@n-apps n-apps added enhancement New feature or request design required Design required before development labels Oct 2, 2018
@n-apps
Copy link
Owner Author

n-apps commented Oct 9, 2018

Maybe it's good opportunity to try new Motion Layout 😸

@n-apps
Copy link
Owner Author

n-apps commented Sep 13, 2020 •
edited
Loading

User can choose the amount of dice that they want to roll

Done in 6.1.6

@vabene1111
Copy link
Contributor

vabene1111 commented Sep 25, 2020 •
edited
Loading

2020-09-25_13-25_Android_Emulator_-_2 7_QVGA_API_295556

added the feature to show each single dice value, if you like it i can submit a PR next week, if not feel free to ask for chages, does not look super aweomse when rolling tons of dice at the same time but thats not a very common thing to do anyway.

do you think a setting is needed to enable/disable this behavior ?

@n-apps
Copy link
Owner Author

n-apps commented Sep 28, 2020

no, settings is not needed. make sure to hide this label for 1 die 😄

@vabene1111
Copy link
Contributor

haha can you read minds ? i forgot exactly that in the first commit and then noticed during testing 😂

@n-apps n-apps removed the design required Design required before development label Sep 29, 2020
@vabene1111 vabene1111 mentioned this issue Sep 30, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@vabene1111 @n-apps