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

Allow players to set names #5

Open
adriantache opened this issue Dec 11, 2017 · 15 comments
Open

Allow players to set names #5

adriantache opened this issue Dec 11, 2017 · 15 comments

Comments

@adriantache
Copy link
Owner

No description provided.

@Safirah
Copy link

Safirah commented Dec 23, 2017

Hey there! Can I start working on this?

@adriantache
Copy link
Owner Author

Absolutely, have fun with it :) Please submit a pull request when you're done.

@Safirah
Copy link

Safirah commented Jan 1, 2018

Any place in particular that I should put the editText?
I was thinking of changing the TextView with the "Player 1" and "Player 2" to EditViews.
Or maybe create a menu where the players can input their names?

@adriantache
Copy link
Owner Author

Not sure, depends on what would make sense and also look good. I think changing the TextViews to EditText is a good idea, and you could add a drawableRight to show it's editable. Try it out, see if you like the look.

@adriantache
Copy link
Owner Author

Or perhaps it would be nicer to have a Button or ImageView next to the TextView that hides TextView, shows EditText, then changes to a submit thing and then hides EditText and updates and then shows the TextView. Seems more elegant than showing a menu or just keeping it EditText.

@Safirah
Copy link

Safirah commented Jan 2, 2018

Thank you for your suggestion, it seems to work quite fine!
But perhaps because my phone screen is quite small it looks a bit odd.

@Safirah
Copy link

Safirah commented Jan 2, 2018

I'll try solving this problem by wrapping everything up on a Scroll View. What do you think?

@adriantache
Copy link
Owner Author

Could you please post a screenshot?

@Safirah
Copy link

Safirah commented Jan 2, 2018

Sorry, it was actually a mistake on my side. I'll be making the pull request in a couple of minutes :)

@deezy136
Copy link

Is this portion of the project complete? if not can I work on this?

@gamino1996
Copy link

Hello, Is this part complete? Let me know as I would like to work on it if not.

@adriantache
Copy link
Owner Author

Hi there. This part is technically complete, but I just need Safirah to clean up her pull request and resubmit it so I can merge it.

@KostasThan
Copy link

Hello.. May i download and take a look at the code? first-timer here..

@adriantache
Copy link
Owner Author

Of course, you can clone the repo without any permissions. And as long as you don't sell it or pass it off as your own there's no need to ask.

@harshitrjpt
Copy link

any link or doc to set this up and run ? I am a beginner and wanted to contribute or atleast have a lookat it. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants