-
Notifications
You must be signed in to change notification settings - Fork 6
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
Comments
Hey there! Can I start working on this? |
Absolutely, have fun with it :) Please submit a pull request when you're done. |
Any place in particular that I should put the editText? |
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. |
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. |
Thank you for your suggestion, it seems to work quite fine! |
I'll try solving this problem by wrapping everything up on a Scroll View. What do you think? |
Could you please post a screenshot? |
Sorry, it was actually a mistake on my side. I'll be making the pull request in a couple of minutes :) |
Is this portion of the project complete? if not can I work on this? |
Hello, Is this part complete? Let me know as I would like to work on it if not. |
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. |
Hello.. May i download and take a look at the code? first-timer here.. |
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. |
any link or doc to set this up and run ? I am a beginner and wanted to contribute or atleast have a lookat it. :) |
No description provided.
The text was updated successfully, but these errors were encountered: