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

Added feature to prevent users from sending empty messages #1

Closed
wants to merge 1 commit into from
Closed

Added feature to prevent users from sending empty messages #1

wants to merge 1 commit into from

Conversation

seshagiriprabhu
Copy link

As mentioned in #60, GUI freezes upon pressing either Enter or Return key for a long time. This feature prevents user from sending blank messages. Well the .pro file which was added automatically along with this commit is awful. It was automatically added. I cloned again and made changes and pushed again and it was again automatically added to my pull request.

@nurupo
Copy link

nurupo commented Mar 1, 2014

@seshagiriprabhu you made a PR to the wrong repo and if you want to commit a specific file, just do git commit -m "message" ./file/to/commit1 ./file/to/commit2 ./etc
Also, the freeze of GUI is caused not by pressing Enter directly, but by messages that get created in the message display widget. We would like to prevent users from sending empty space messages, so your PR is fine. Just remove the .pro file from the commit and submit it to the right repo : )

@Schlumpf
Copy link
Owner

Schlumpf commented Mar 5, 2014

Merged to nurupo/ProjectTox-Qt-GUI

@Schlumpf Schlumpf closed this Mar 5, 2014
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.

3 participants