From 45536d82e0c32f80051b4b83ab70f78262a5e709 Mon Sep 17 00:00:00 2001 From: Krocks Date: Tue, 18 Jul 2017 17:34:37 +0300 Subject: [PATCH 1/5] changed hardcode to string resorces and logs --- .../mvp/presenters/CodeInputPresenter.java | 10 ++++++---- .../mvp/presenters/EditProfilePresenter.java | 17 +++++++---------- .../mvp/presenters/InfoPresenter.java | 16 ++++++---------- .../mvp/presenters/LoginPresenter.java | 5 +++-- .../mvp/presenters/MainPresenter.java | 6 +++--- .../communicator/mvp/view/CodeInputView.java | 2 ++ .../communicator/mvp/view/EditProfileView.java | 4 +++- .../communicator/mvp/view/ErrorMvpView.java | 2 -- .../mobile/communicator/mvp/view/InfoView.java | 4 ++++ .../ui/activities/MainActivity.java | 16 +++++++++++++--- .../communicator/ui/base/BaseFragment.java | 12 ++++++++++-- .../fragments/profile/EditProfileFragment.java | 7 +++++-- .../ui/fragments/signin/CodeInputFragment.java | 4 ++-- .../ui/fragments/signin/InfoFragment.java | 10 +++++++--- .../ui/fragments/signin/LoginFragment.java | 5 +---- app/src/main/res/values/strings.xml | 6 ++++++ 16 files changed, 78 insertions(+), 48 deletions(-) diff --git a/app/src/main/java/com/ninja/mobile/communicator/mvp/presenters/CodeInputPresenter.java b/app/src/main/java/com/ninja/mobile/communicator/mvp/presenters/CodeInputPresenter.java index a6af5c3f31..509b92f6ad 100644 --- a/app/src/main/java/com/ninja/mobile/communicator/mvp/presenters/CodeInputPresenter.java +++ b/app/src/main/java/com/ninja/mobile/communicator/mvp/presenters/CodeInputPresenter.java @@ -8,6 +8,8 @@ import com.ninja.mobile.communicator.ui.fragments.signin.InfoFragment; import com.ninja.mobile.communicator.utils.StringUtils; import com.ninja.mobile.communicator.utils.ValidationUtils; +import timber.log.Timber; + /** * Created by dmitro.boiko on 05/07/2017. **/ @@ -44,7 +46,7 @@ public class CodeInputPresenter extends BaseErrorPresenter { } private void errorFlow(Io io) { - showCodeErrorMessage(io.getStatus(), -1); + Timber.e(io.getStatus()); } private void successFlow(Io io) { @@ -70,12 +72,12 @@ public class CodeInputPresenter extends BaseErrorPresenter { mDataManager.confirmSms(StringUtils.deleteSpaces(code)); phoneNumber = phone; } else { - showCodeErrorMessage("Code is too small.", -1); + showCodeTooSmallError(); } } - private void showCodeErrorMessage(String message, int errorCode) { - getViewState().showCodeError(message, errorCode); + private void showCodeTooSmallError() { + getViewState().showCodeTooSmallError(); } private void navigateToInfoFragment() { diff --git a/app/src/main/java/com/ninja/mobile/communicator/mvp/presenters/EditProfilePresenter.java b/app/src/main/java/com/ninja/mobile/communicator/mvp/presenters/EditProfilePresenter.java index d47c05250f..c83710dfb4 100644 --- a/app/src/main/java/com/ninja/mobile/communicator/mvp/presenters/EditProfilePresenter.java +++ b/app/src/main/java/com/ninja/mobile/communicator/mvp/presenters/EditProfilePresenter.java @@ -13,6 +13,8 @@ import com.ninja.mobile.communicator.utils.ValidationUtils; import java.util.ArrayList; +import timber.log.Timber; + /** * Date: 25.06.17 * Time: 04:10 @@ -37,7 +39,7 @@ public class EditProfilePresenter extends BasePresenter { } else { //// FIXME: 13/07/2017 message with error if (response.data instanceof Io) { - getViewState().onErrorMessage(((Io) response.data).getStatus()); + Timber.e(((Io) response.data).getStatus()); } } } @@ -55,10 +57,6 @@ public class EditProfilePresenter extends BasePresenter { getViewState().navigateBack(mProfile); } - private void showErrorMessage(String errorMessage) { - getViewState().onErrorMessage(errorMessage); - } - public void saveClick(Profile profile, String firstName, String lastName) { if (ValidationUtils.isNameValid(firstName)) { updateProfile(profile, firstName, lastName); @@ -69,13 +67,12 @@ public class EditProfilePresenter extends BasePresenter { private void createErrorMessage(String firstName) { //// FIXME: 30/06/2017 add error messages - String errorMessage; - if (TextUtils.isEmpty(firstName)) { - errorMessage = "FirstName cannot be empty."; + + if (firstName.isEmpty()){ + getViewState().showErrorEmptyFirstName(); } else { - errorMessage = "FirstName length should be at least 2 symbols."; + getViewState().showErrorTooSmallFirstName(); } - showErrorMessage(errorMessage); } private void updateProfile(Profile profile, String firstName, String lastName) { diff --git a/app/src/main/java/com/ninja/mobile/communicator/mvp/presenters/InfoPresenter.java b/app/src/main/java/com/ninja/mobile/communicator/mvp/presenters/InfoPresenter.java index 4a44953f1c..e2963afc88 100644 --- a/app/src/main/java/com/ninja/mobile/communicator/mvp/presenters/InfoPresenter.java +++ b/app/src/main/java/com/ninja/mobile/communicator/mvp/presenters/InfoPresenter.java @@ -17,6 +17,8 @@ import com.ninja.mobile.communicator.utils.ValidationUtils; import java.util.ArrayList; +import timber.log.Timber; + @InjectViewState public class InfoPresenter extends BaseErrorPresenter { @@ -37,7 +39,7 @@ public class InfoPresenter extends BaseErrorPresenter { } else { //// FIXME: 13/07/2017 message with error if (response.data instanceof Io) { - showErrorMessage(((Io) response.data).getStatus(), -1); + Timber.e(((Io) response.data).getStatus()); } } } @@ -54,10 +56,6 @@ public class InfoPresenter extends BaseErrorPresenter { } } - private void showErrorMessage(String message, int messageId) { - getViewState().showCodeError(message, messageId); - } - public void onSaveClick(String firstName, String lastName) { if (ValidationUtils.isNameValid(firstName)) { updateRoster(firstName, lastName); @@ -76,13 +74,11 @@ public class InfoPresenter extends BaseErrorPresenter { private void createErrorMessage(String firstName) { //// FIXME: 30/06/2017 add error messages - String errorMessage; - if (TextUtils.isEmpty(firstName)) { - errorMessage = "FirstName cannot be empty."; + if (firstName.isEmpty()){ + getViewState().showErrorEmptyFirstName(); } else { - errorMessage = "FirstName length should be at least 2 symbols."; + getViewState().showErrorTooSmallFirstName(); } - showErrorMessage(errorMessage, -1); } public void getProfile() { diff --git a/app/src/main/java/com/ninja/mobile/communicator/mvp/presenters/LoginPresenter.java b/app/src/main/java/com/ninja/mobile/communicator/mvp/presenters/LoginPresenter.java index 1e452bbe49..1395fc06c3 100644 --- a/app/src/main/java/com/ninja/mobile/communicator/mvp/presenters/LoginPresenter.java +++ b/app/src/main/java/com/ninja/mobile/communicator/mvp/presenters/LoginPresenter.java @@ -14,6 +14,7 @@ import com.ninja.mobile.communicator.utils.Utils; import io.reactivex.Observable; import io.reactivex.disposables.Disposable; +import timber.log.Timber; /** * Date: 25.06.17 @@ -171,7 +172,7 @@ public class LoginPresenter extends BasePresenter { case Io.Responses.MISMATCH_USER_DATA: case Io.Responses.ERROR: default: - getViewState().showCodeError(io.getStatus(), -2); + Timber.e(io.getStatus()); break; } } @@ -186,7 +187,7 @@ public class LoginPresenter extends BasePresenter { break; default: //ShowError - getViewState().showCodeError(io.getStatus(), -2); + Timber.e(io.getStatus()); break; } } diff --git a/app/src/main/java/com/ninja/mobile/communicator/mvp/presenters/MainPresenter.java b/app/src/main/java/com/ninja/mobile/communicator/mvp/presenters/MainPresenter.java index 1c1dfa278a..847b263347 100644 --- a/app/src/main/java/com/ninja/mobile/communicator/mvp/presenters/MainPresenter.java +++ b/app/src/main/java/com/ninja/mobile/communicator/mvp/presenters/MainPresenter.java @@ -35,11 +35,11 @@ public class MainPresenter extends BaseErrorPresenter { manageProfile(response); } else { // FIXME: 12.07.2017 show error - getViewState().showCodeError("Model error", -1); + Timber.e("Model error"); } } else { // FIXME: 12.07.2017 show error - getViewState().showCodeError("error", -1); + Timber.e("error"); } } catch (Exception e) { Timber.e(e); @@ -50,7 +50,7 @@ public class MainPresenter extends BaseErrorPresenter { mDataManager.saveProfile((Profile) response.data); Profile profile = (Profile) response.data; if (profile.status.equals(BaseModelMQQT.Types.GET)) { - getViewState().showCodeError("Profile successfully received", -1); + Timber.e("Profile successfully received"); } else { getViewState().navigateToSplash(); mDataManager.clearPreferences(); diff --git a/app/src/main/java/com/ninja/mobile/communicator/mvp/view/CodeInputView.java b/app/src/main/java/com/ninja/mobile/communicator/mvp/view/CodeInputView.java index 496d39170c..93cd8bf3bb 100644 --- a/app/src/main/java/com/ninja/mobile/communicator/mvp/view/CodeInputView.java +++ b/app/src/main/java/com/ninja/mobile/communicator/mvp/view/CodeInputView.java @@ -20,4 +20,6 @@ public interface CodeInputView extends ErrorMvpView { void onFinishTimer(); void showSuccessCodeSendMessage(); + + void showCodeTooSmallError(); } diff --git a/app/src/main/java/com/ninja/mobile/communicator/mvp/view/EditProfileView.java b/app/src/main/java/com/ninja/mobile/communicator/mvp/view/EditProfileView.java index f4263be8fd..766895f7c2 100644 --- a/app/src/main/java/com/ninja/mobile/communicator/mvp/view/EditProfileView.java +++ b/app/src/main/java/com/ninja/mobile/communicator/mvp/view/EditProfileView.java @@ -9,7 +9,9 @@ import com.ninja.mobile.communicator.data.models.mqtt.Profile; public interface EditProfileView extends MvpView { - void onErrorMessage(String errorMessage); + void showErrorEmptyFirstName(); + + void showErrorTooSmallFirstName(); void navigateBack(Profile userPerson); } diff --git a/app/src/main/java/com/ninja/mobile/communicator/mvp/view/ErrorMvpView.java b/app/src/main/java/com/ninja/mobile/communicator/mvp/view/ErrorMvpView.java index be6da69964..9e03bc95d0 100644 --- a/app/src/main/java/com/ninja/mobile/communicator/mvp/view/ErrorMvpView.java +++ b/app/src/main/java/com/ninja/mobile/communicator/mvp/view/ErrorMvpView.java @@ -13,6 +13,4 @@ public interface ErrorMvpView extends MvpView{ @StateStrategyType(SingleStateStrategy.class) void showInternetError(); - @StateStrategyType(SingleStateStrategy.class) - void showCodeError(String message, int code); } diff --git a/app/src/main/java/com/ninja/mobile/communicator/mvp/view/InfoView.java b/app/src/main/java/com/ninja/mobile/communicator/mvp/view/InfoView.java index 6b11bbd505..4554cf53b5 100644 --- a/app/src/main/java/com/ninja/mobile/communicator/mvp/view/InfoView.java +++ b/app/src/main/java/com/ninja/mobile/communicator/mvp/view/InfoView.java @@ -8,6 +8,10 @@ import com.ninja.mobile.communicator.data.models.mqtt.Profile; public interface InfoView extends ErrorMvpView { + void showErrorEmptyFirstName(); + + void showErrorTooSmallFirstName(); + void navigateForward(Profile roster); void initPerson(Profile profile); diff --git a/app/src/main/java/com/ninja/mobile/communicator/ui/activities/MainActivity.java b/app/src/main/java/com/ninja/mobile/communicator/ui/activities/MainActivity.java index 76889db881..128463cef4 100644 --- a/app/src/main/java/com/ninja/mobile/communicator/ui/activities/MainActivity.java +++ b/app/src/main/java/com/ninja/mobile/communicator/ui/activities/MainActivity.java @@ -51,9 +51,6 @@ public class MainActivity extends BaseActivity implements MainView { showAlert(getString(R.string.error_no_internet_connection), null); } - @Override public void showCodeError(String message, int code) { - showAlert(message, null); - } @Override public void navigateToSplash() { startActivity(WelcomeActivity.getLaunchIntent(this)); @@ -64,7 +61,20 @@ public class MainActivity extends BaseActivity implements MainView { startActivity(ProfileActivity.getLaunchIntent(this, ProfileActivity.ProfileTypes.MyProfile)); } + private void showAlert(int msg){ + showAlert(msg, null); + } + + private void showAlert(int msg, DialogInterface.OnClickListener listener){ + showAlert(getString(msg), listener); + + } + private void showAlert(String message, DialogInterface.OnClickListener listener) { + + if (listener == null){ + listener = (dialog, which) -> dialog.dismiss(); + } //// FIXME: 02.07.17 move to dialog utils/factory AlertDialog alertDialog = new AlertDialog.Builder(this).create(); alertDialog.setTitle(getString(R.string.app_name)); diff --git a/app/src/main/java/com/ninja/mobile/communicator/ui/base/BaseFragment.java b/app/src/main/java/com/ninja/mobile/communicator/ui/base/BaseFragment.java index 754646a2c9..5d11a93b84 100644 --- a/app/src/main/java/com/ninja/mobile/communicator/ui/base/BaseFragment.java +++ b/app/src/main/java/com/ninja/mobile/communicator/ui/base/BaseFragment.java @@ -44,11 +44,19 @@ public abstract class BaseFragment extends MvpAppCompatFragment { return false; } - protected void showAlert(@StringRes int id, DialogInterface.OnClickListener listener) { + protected void showAlert(@StringRes int id){ + showAlert(id, null); + } + + protected void showAlert(@StringRes int id, @Nullable DialogInterface.OnClickListener listener) { showAlert(getString(id), listener); } - protected void showAlert(String message, DialogInterface.OnClickListener listener) { + protected void showAlert(String message, @Nullable DialogInterface.OnClickListener listener) { + + if (listener == null) { + listener = (dialog, which) -> dialog.dismiss(); + } //// FIXME: 02.07.17 move to dialog utils/factory AlertDialog alertDialog = new AlertDialog.Builder(getContext()).create(); alertDialog.setTitle(getString(R.string.app_name)); diff --git a/app/src/main/java/com/ninja/mobile/communicator/ui/fragments/profile/EditProfileFragment.java b/app/src/main/java/com/ninja/mobile/communicator/ui/fragments/profile/EditProfileFragment.java index 4e68a8bfb8..124b5c5d06 100644 --- a/app/src/main/java/com/ninja/mobile/communicator/ui/fragments/profile/EditProfileFragment.java +++ b/app/src/main/java/com/ninja/mobile/communicator/ui/fragments/profile/EditProfileFragment.java @@ -65,9 +65,12 @@ public class EditProfileFragment extends BaseFragment implements EditProfileView mPresenter.saveClick(mProfile, StringUtils.getText(mFirstName), StringUtils.getText(mLastName)); } - @Override - public void onErrorMessage(String errorMessage) { + @Override public void showErrorEmptyFirstName() { + showAlert(R.string.signin_text_fname_empty); + } + @Override public void showErrorTooSmallFirstName() { + showAlert(R.string.signin_text_too_smal); } @Override diff --git a/app/src/main/java/com/ninja/mobile/communicator/ui/fragments/signin/CodeInputFragment.java b/app/src/main/java/com/ninja/mobile/communicator/ui/fragments/signin/CodeInputFragment.java index 5c6d74933c..6fc4016788 100644 --- a/app/src/main/java/com/ninja/mobile/communicator/ui/fragments/signin/CodeInputFragment.java +++ b/app/src/main/java/com/ninja/mobile/communicator/ui/fragments/signin/CodeInputFragment.java @@ -124,7 +124,7 @@ public class CodeInputFragment extends BaseFragment implements CodeInputView { showAlert(getString(R.string.error_no_internet_connection), null); } - @Override public void showCodeError(String message, int code) { - showAlert(message, null); + @Override public void showCodeTooSmallError() { + showAlert(R.string.signin_text_code_small); } } diff --git a/app/src/main/java/com/ninja/mobile/communicator/ui/fragments/signin/InfoFragment.java b/app/src/main/java/com/ninja/mobile/communicator/ui/fragments/signin/InfoFragment.java index 93ba718595..abcb1222bc 100644 --- a/app/src/main/java/com/ninja/mobile/communicator/ui/fragments/signin/InfoFragment.java +++ b/app/src/main/java/com/ninja/mobile/communicator/ui/fragments/signin/InfoFragment.java @@ -79,9 +79,13 @@ public class InfoFragment extends BaseFragment implements InfoView { showAlert(R.string.error_no_internet_connection, null); } - @Override - public void showCodeError(String message, int code) { - showAlert(message, null); + + @Override public void showErrorEmptyFirstName() { + showAlert(R.string.signin_text_fname_empty); + } + + @Override public void showErrorTooSmallFirstName() { + showAlert(R.string.signin_text_too_smal); } @Override diff --git a/app/src/main/java/com/ninja/mobile/communicator/ui/fragments/signin/LoginFragment.java b/app/src/main/java/com/ninja/mobile/communicator/ui/fragments/signin/LoginFragment.java index 1131d5cfe6..ee3a11c129 100644 --- a/app/src/main/java/com/ninja/mobile/communicator/ui/fragments/signin/LoginFragment.java +++ b/app/src/main/java/com/ninja/mobile/communicator/ui/fragments/signin/LoginFragment.java @@ -170,7 +170,7 @@ public class LoginFragment extends BaseFragment implements LoginView { } private void initSpannableTerms() { - View.OnClickListener onClickListener = v -> Toast.makeText(getActivity(), "It is not working now :-(", Toast.LENGTH_SHORT).show(); + View.OnClickListener onClickListener = v -> Toast.makeText(getActivity(), R.string.not_working_now, Toast.LENGTH_SHORT).show(); SpannableUtils.setSpannable(getActivity(), mTermsOfUse, R.string.signin_text_description, R.string.signin_text_description_highlight, onClickListener); } @@ -178,9 +178,6 @@ public class LoginFragment extends BaseFragment implements LoginView { Toast.makeText(getContext(), R.string.error_no_internet_connection, Toast.LENGTH_SHORT).show(); } - @Override public void showCodeError(String message, int code) { - - } @Override protected boolean shouldRetainInstance() { return true; diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml index 3f7180e76c..5e69790f93 100644 --- a/app/src/main/res/values/strings.xml +++ b/app/src/main/res/values/strings.xml @@ -18,6 +18,12 @@ First Name\u002A Last Name --- --- + First name is too small + First name can\'t be empty + Code is too small + + + "It is not working now :-(" Camera -- GitLab From ae5ee1be7579079cba31fb13fce74527430c4d82 Mon Sep 17 00:00:00 2001 From: Krocks Date: Wed, 19 Jul 2017 10:31:34 +0300 Subject: [PATCH 2/5] added next button disability until valid fname --- .../mvp/presenters/InfoPresenter.java | 4 +++ .../communicator/mvp/view/InfoView.java | 2 ++ .../ui/fragments/signin/InfoFragment.java | 27 +++++++++++++++++++ 3 files changed, 33 insertions(+) diff --git a/app/src/main/java/com/ninja/mobile/communicator/mvp/presenters/InfoPresenter.java b/app/src/main/java/com/ninja/mobile/communicator/mvp/presenters/InfoPresenter.java index e2963afc88..5357341ce3 100644 --- a/app/src/main/java/com/ninja/mobile/communicator/mvp/presenters/InfoPresenter.java +++ b/app/src/main/java/com/ninja/mobile/communicator/mvp/presenters/InfoPresenter.java @@ -56,6 +56,10 @@ public class InfoPresenter extends BaseErrorPresenter { } } + public void validateFirstName(String fName){ + getViewState().enableNextButton(ValidationUtils.isNameValid(fName)); + } + public void onSaveClick(String firstName, String lastName) { if (ValidationUtils.isNameValid(firstName)) { updateRoster(firstName, lastName); diff --git a/app/src/main/java/com/ninja/mobile/communicator/mvp/view/InfoView.java b/app/src/main/java/com/ninja/mobile/communicator/mvp/view/InfoView.java index 4554cf53b5..6e266da027 100644 --- a/app/src/main/java/com/ninja/mobile/communicator/mvp/view/InfoView.java +++ b/app/src/main/java/com/ninja/mobile/communicator/mvp/view/InfoView.java @@ -15,4 +15,6 @@ public interface InfoView extends ErrorMvpView { void navigateForward(Profile roster); void initPerson(Profile profile); + + void enableNextButton(boolean enable); } diff --git a/app/src/main/java/com/ninja/mobile/communicator/ui/fragments/signin/InfoFragment.java b/app/src/main/java/com/ninja/mobile/communicator/ui/fragments/signin/InfoFragment.java index abcb1222bc..bcc3d0cfca 100644 --- a/app/src/main/java/com/ninja/mobile/communicator/ui/fragments/signin/InfoFragment.java +++ b/app/src/main/java/com/ninja/mobile/communicator/ui/fragments/signin/InfoFragment.java @@ -3,9 +3,13 @@ package com.ninja.mobile.communicator.ui.fragments.signin; import android.os.Bundle; import android.support.annotation.Nullable; import android.support.design.widget.TextInputEditText; +import android.text.Editable; +import android.text.TextWatcher; import android.view.LayoutInflater; import android.view.View; import android.view.ViewGroup; +import android.widget.Button; +import android.widget.ImageButton; import com.arellomobile.mvp.presenter.InjectPresenter; import com.ninja.mobile.communicator.R; @@ -32,6 +36,7 @@ public class InfoFragment extends BaseFragment implements InfoView { TextInputEditText mFirstName; @BindView(R.id.f_iname_edt_last_name) TextInputEditText mLastName; + @BindView(R.id.f_iname_btn_next) ImageButton next; @InjectPresenter InfoPresenter infoPresenter; @@ -53,6 +58,7 @@ public class InfoFragment extends BaseFragment implements InfoView { @Override public void onViewCreated(View view, @Nullable Bundle savedInstanceState) { super.onViewCreated(view, savedInstanceState); + initFirstNameValidation(); } @Nullable @@ -74,6 +80,23 @@ public class InfoFragment extends BaseFragment implements InfoView { infoPresenter.onSaveClick(StringUtils.getText(mFirstName), StringUtils.getText(mLastName)); } + private void initFirstNameValidation(){ + mFirstName.addTextChangedListener(new TextWatcher() { + @Override + public void beforeTextChanged(CharSequence s, int start, int count, int after) { + + } + + @Override public void onTextChanged(CharSequence s, int start, int before, int count) { + + } + + @Override public void afterTextChanged(Editable s) { + infoPresenter.validateFirstName(s.toString()); + } + }); + } + @Override public void showInternetError() { showAlert(R.string.error_no_internet_connection, null); @@ -97,4 +120,8 @@ public class InfoFragment extends BaseFragment implements InfoView { public void initPerson(Profile profile) { setUserModel(profile); } + + @Override public void enableNextButton(boolean enable) { + next.setEnabled(enable); + } } -- GitLab From 4a14f581392ab2513beaeddc155f3338f6425e6f Mon Sep 17 00:00:00 2001 From: Krocks Date: Wed, 19 Jul 2017 11:20:46 +0300 Subject: [PATCH 3/5] fixed bug #76 changed name to ninja mobile communicator --- app/src/main/res/values/strings.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml index 5e69790f93..64e7e00412 100644 --- a/app/src/main/res/values/strings.xml +++ b/app/src/main/res/values/strings.xml @@ -1,5 +1,5 @@ - NYNJA + NINJA Mobile Communicator Welcome to Get started -- GitLab From d04e566937cfe731c43cadb8bca636a9e2f601f2 Mon Sep 17 00:00:00 2001 From: Krocks Date: Wed, 19 Jul 2017 11:56:16 +0300 Subject: [PATCH 4/5] fixed bug #84 either wheel or keyboard should be always shown --- .../ui/fragments/signin/LoginFragment.java | 3 ++- .../ui/views/HintEditTextAlwaysKeyboard.java | 25 +++++++++++++++++++ .../main/res/layout/fragment_input_phone.xml | 2 +- 3 files changed, 28 insertions(+), 2 deletions(-) create mode 100644 app/src/main/java/com/ninja/mobile/communicator/ui/views/HintEditTextAlwaysKeyboard.java diff --git a/app/src/main/java/com/ninja/mobile/communicator/ui/fragments/signin/LoginFragment.java b/app/src/main/java/com/ninja/mobile/communicator/ui/fragments/signin/LoginFragment.java index ee3a11c129..7bfc464f9a 100644 --- a/app/src/main/java/com/ninja/mobile/communicator/ui/fragments/signin/LoginFragment.java +++ b/app/src/main/java/com/ninja/mobile/communicator/ui/fragments/signin/LoginFragment.java @@ -26,6 +26,7 @@ import com.ninja.mobile.communicator.ui.activities.RegistrationActivity; import com.ninja.mobile.communicator.ui.adapters.CountryWheelAdapter; import com.ninja.mobile.communicator.ui.base.BaseFragment; import com.ninja.mobile.communicator.ui.views.HintEditText; +import com.ninja.mobile.communicator.ui.views.HintEditTextAlwaysKeyboard; import com.ninja.mobile.communicator.ui.views.wheel.WheelView; import com.ninja.mobile.communicator.utils.PhoneWatcher; import com.ninja.mobile.communicator.utils.SpannableUtils; @@ -49,7 +50,7 @@ public class LoginFragment extends BaseFragment implements LoginView { @BindView(R.id.f_iphone_tv_country) TextView mCountry; @BindView(R.id.f_iphone_edt_country_code) EditText mCountryCode; - @BindView(R.id.f_iphone_edt_number) HintEditText mPhoneEditText; + @BindView(R.id.f_iphone_edt_number) HintEditTextAlwaysKeyboard mPhoneEditText; @BindView(R.id.f_iphone_tv_description) TextView mTermsOfUse; @BindView(R.id.f_iphone_btn_next) ImageButton nextButton; diff --git a/app/src/main/java/com/ninja/mobile/communicator/ui/views/HintEditTextAlwaysKeyboard.java b/app/src/main/java/com/ninja/mobile/communicator/ui/views/HintEditTextAlwaysKeyboard.java new file mode 100644 index 0000000000..50db6e3738 --- /dev/null +++ b/app/src/main/java/com/ninja/mobile/communicator/ui/views/HintEditTextAlwaysKeyboard.java @@ -0,0 +1,25 @@ +package com.ninja.mobile.communicator.ui.views; + +import android.content.Context; +import android.util.AttributeSet; +import android.view.KeyEvent; + +public class HintEditTextAlwaysKeyboard extends HintEditText { + public HintEditTextAlwaysKeyboard(Context context) { + super(context); + } + + public HintEditTextAlwaysKeyboard(Context context, AttributeSet attrs) { + super(context, attrs); + } + + public HintEditTextAlwaysKeyboard(Context context, AttributeSet attrs, int defStyleAttr) { + super(context, attrs, defStyleAttr); + } + + + @Override public boolean onKeyPreIme(int keyCode, KeyEvent event) { +// if (keyCode == KeyEvent.KEYCODE_BACK) + return true; + } +} diff --git a/app/src/main/res/layout/fragment_input_phone.xml b/app/src/main/res/layout/fragment_input_phone.xml index 79b965c869..862988e430 100644 --- a/app/src/main/res/layout/fragment_input_phone.xml +++ b/app/src/main/res/layout/fragment_input_phone.xml @@ -119,7 +119,7 @@ app:layout_constraintVertical_bias="0.5" tools:text="1237"/> - Date: Wed, 19 Jul 2017 16:40:44 +0300 Subject: [PATCH 5/5] small fix to app name --- app/src/main/res/values/strings.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/src/main/res/values/strings.xml b/app/src/main/res/values/strings.xml index 64e7e00412..e188237189 100644 --- a/app/src/main/res/values/strings.xml +++ b/app/src/main/res/values/strings.xml @@ -1,5 +1,5 @@ - NINJA Mobile Communicator + NYNJA Mobile Communicator Welcome to Get started -- GitLab