search

eshcherbin / save-the-moment

SPbAU educational Android project
0 stars 0 forks source link

Implemented choosing location in MomentEditor #18

Closed eshcherbin closed 7 years ago

eshcherbin commented 7 years ago

При редактировании момента можно выбирать локацию, как на карте, так и запросив текущую.

krinkinmu commented 7 years ago

2016-12-03 20:34 GMT+03:00 Egor Shcherbin notifications@github.com:

@eshcherbin commented on this pull request.

In app/src/main/java/ru/spbau/savethemoment/ui/MomentEditorActivity.java https://github.com/eshcherbin/save-the-moment/pull/18:

@@ -163,11 +244,56 @@ private void initLocation() { editLocation.setOnClickListener(new View.OnClickListener() { @Override public void onClick(View v) {

  • //TODO: handle location changes
  • final String currentLocationMessage = getString(R.string.moment_editor_current_location);

Хорошо, тогда правильно ли я понимаю, что предлагается делать следующим образом: открыть ChooseLocation, если у момента не было локации, достать текущую, и отобразить её на экране? Даже если сама карта не подгрузится?

нет, если никакой локации не было, не пытаться ее хоть как-то показывать и задавать, пока пользователь сам не попросит, если локация была отображаем ее и разрешаем редактировать.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/eshcherbin/save-the-moment/pull/18, or mute the thread https://github.com/notifications/unsubscribe-auth/ABH4c0X3mBBWaT-1Q6M2RJzxYcc2rA9zks5rEag0gaJpZM4LDTFJ .

eshcherbin commented 7 years ago

Такое чувство, что мы сейчас разговариваем о разных вещах. То, как мне кажется сейчас, что надо сделать:

krinkinmu commented 7 years ago

2016-12-03 20:45 GMT+03:00 Egor Shcherbin notifications@github.com:

Такое чувство, что мы сейчас разговариваем о разных вещах. То, как мне кажется сейчас, что надо сделать:

  • При просмотре момента никак не отображать локацию, если её нет.
  • При редактировании момента по нажатии на кнопку "Change location" открывать карту, на которой пользователь ставит маркер для выбора позиции, причём:
    • Можно менять/задавать локацию всегда
    • Если локация уже есть, то на этой карте изначально маркер стоит в этой локации
    • Если её изначально не было, то на этой карте изначально маркер стоит в текущей локации

да, только последняя известная локация и текущая локация это две разные вещи, но суть примерно в этом.

-

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/eshcherbin/save-the-moment/pull/18#issuecomment-264653936, or mute the thread https://github.com/notifications/unsubscribe-auth/ABH4c19qGt0mRefPp-RAEudkCLrGi_fUks5rEarLgaJpZM4LDTFJ .

krinkinmu commented 7 years ago

ок