Fixes #433, Fixed disappearing ProgressDialog on touch

[DravitLochan]

Fix:433

Summary of changes

While connecting to server, refreshing, deleting a db or at the time of logout, Progress dialog box won't be dismissed by taping outside the box.

• [x] Code is formatted.

• [x] Run the lint tests with ./gradlew lintDebug and make sure BUILD is SUCCESSFUL

• [x] If new feature is implemeted then it is compatible with Night theme too.

• [x] Commit messages are well-written.

[smarx]

Automated message from Dropbox CLA bot

@DravitLochan, it looks like you've already signed the Dropbox CLA. Thanks!

[kunall17]

Again you are calling setCancelable before showWithMessage!

[DravitLochan]

Ya. It's working fine.

[DravitLochan]

@saketkumar95 is there anything to change in this pr?

[DravitLochan]

@kunall17 ???

[brainwane]

@DravitLochan Hi and thanks for your pull request. I am not an Android developer so I can't really review your code, but I can help you with the style of your PR to make it easier for others to review!

Here's a guide to help you write commit messages that are more consistent with our style, which makes it easier for us in the future -- when someone's trying to solve a new problem, if commit messages are all in the same format, it's easier to skim them in logs and quickly understand what's happening.

Here are some tips on tidying your commit history, including changing your commit messages. And when you update your code or your commit message, you can update your pull request and the changes will appear here. :)

[DravitLochan]

Thanks @brainwane I have once again gone through the link you've attached. But I think I have followed them. please correct me if I am wrong... thanks once again!

[brainwane]

@DravitLochan Please take another look at https://zulip.readthedocs.io/en/latest/version-control.html#commit-messages . Zulip’s preferred style for this is to have the final paragraph of the commit message have the issue number with "Fixes:" before it, as in:

Fixes: #nnn.

Also, please use complete sentences, with proper punctuation, spacing, capitalization, etc. We have free English tutoring sessions in the next few weeks and days that can help you with grammar more generally! Here your commit message could be something like:

Fix disappearing ProgressDialog on touch.

When the user selects "refresh" from the overflow menu, the progress dialog should only appear if the user is connected to the network.

Fixes: #433.

If anything in my suggested commit message is inaccurate then of course please fix the inaccuracy in your revised message. :) Thanks.

[DravitLochan]

@brainwane I think you caught something wrong. The purpose ain't that. {When the user selects "refresh" from the overflow menu, the progress dialog should only appear if the user is connected to the network.}

@kunall17 can you please make the changes so that I can get to know where I am doing wrong?

[zulipbot]

@DravitLochan, your pull request has developed a merge conflict! Please review the most recent commit (f791888) for conflicting changes and resolve your pull request's merge conflicts.

[brainwane]

@DravitLochan if I've misunderstood the purpose of your commit, then please explain the purpose so someone can clearly understand it. Thanks! :)

[DravitLochan]

I think functionality has been added properly. It's just the commit messages. That would be great help if someone from mentors can make the changes and guide me. :)