WIP: Location sharing text format update #1821

Draft
pastk wants to merge 6 commits from pastk/pastk-loc-share into master
Member

The new format:

  • bookmark / poi / place name if available and except "Unknown Place" and "My Position"
  • address if available
  • https://omaps.app/XYZ/Name link
  • decimal Lat, Lot coords

iOS code was edited naively without any testing, so needs attention of iOS devs (or I can exclude it from the PR).

The new format: * bookmark / poi / place name if available and except "Unknown Place" and "My Position" * address if available * https://omaps.app/XYZ/Name link * decimal Lat, Lot coords iOS code was edited naively without any testing, so needs attention of iOS devs (or I can exclude it from the PR).
biodranik commented 2022-01-09 20:06:54 +00:00 (Migrated from github.com)
/Users/runner/work/organicmaps/organicmaps/iphone/Maps/Classes/Share/MWMShareActivityItem.mm:97:16: error: no visible @interface for 'MWMShareActivityItem' declares the selector 'subjectDefault'
  return [self subjectDefault];
          ~~~~ ^~~~~~~~~~~~~~
/Users/runner/work/organicmaps/organicmaps/iphone/Maps/Classes/Share/MWMShareActivityItem.mm:118:21: error: cannot initialize a variable of type 'NSMutableString *__strong' with an lvalue of type 'const char [1]'
  NSMutableString * result = "";
                    ^        ~~
2 errors generated.
``` /Users/runner/work/organicmaps/organicmaps/iphone/Maps/Classes/Share/MWMShareActivityItem.mm:97:16: error: no visible @interface for 'MWMShareActivityItem' declares the selector 'subjectDefault' return [self subjectDefault]; ~~~~ ^~~~~~~~~~~~~~ /Users/runner/work/organicmaps/organicmaps/iphone/Maps/Classes/Share/MWMShareActivityItem.mm:118:21: error: cannot initialize a variable of type 'NSMutableString *__strong' with an lvalue of type 'const char [1]' NSMutableString * result = ""; ^ ~~ 2 errors generated. ```
Author
Member

I need help with iOS changes (I edit naively, I have no means to build it even).
Otherwise I'll drop iOS changes from this PR.

I need help with iOS changes (I edit naively, I have no means to build it even). Otherwise I'll drop iOS changes from this PR.
biodranik (Migrated from github.com) reviewed 2022-01-09 20:18:19 +00:00
biodranik (Migrated from github.com) left a comment

How does email sharing look now? Which is the subject and the body text?

How does email sharing look now? Which is the subject and the body text?
Author
Member

subject пустой
текст такой же, как и при отправке в мессенджер (см. описание выше)

subject пустой текст такой же, как и при отправке в мессенджер (см. описание выше)
biodranik commented 2022-01-09 21:08:56 +00:00 (Migrated from github.com)

subject пустой текст такой же, как и при отправке в мессенджер (см. описание выше)

Это плохо. Раньше был не пустой. Текст сабжекта можно менять, но убирать его полностью мне кажется очень плохой идеей.

> subject пустой текст такой же, как и при отправке в мессенджер (см. описание выше) Это плохо. Раньше был не пустой. Текст сабжекта можно менять, но убирать его полностью мне кажется очень плохой идеей.
Author
Member

вот так:
Screenshot_20220110-010916
можно и вернуть его, просто он никакой смысловой нагрузки не несёт
а поскольку ещё и по электронке сейчас редко кто шарит, то я решил совсем его убрать

вот так: ![Screenshot_20220110-010916](https://user-images.githubusercontent.com/18434508/148701151-a2b878a9-a92a-4106-9ce5-8d089322f05c.jpg) можно и вернуть его, просто он никакой смысловой нагрузки не несёт а поскольку ещё и по электронке сейчас редко кто шарит, то я решил совсем его убрать
biodranik commented 2022-01-09 21:18:06 +00:00 (Migrated from github.com)

можно и вернуть его, просто он никакой смысловой нагрузки не несёт

Туда можно вставлять имя метки и/или адрес, если они есть. А если нет, то заглушку типа Open in Organic Maps

> можно и вернуть его, просто он никакой смысловой нагрузки не несёт Туда можно вставлять имя метки и/или адрес, если они есть. А если нет, то заглушку типа Open in Organic Maps
Author
Member

Хорошая идея с именем! Завтра сделаю

Хорошая идея с именем! Завтра сделаю
biodranik commented 2022-01-09 21:36:32 +00:00 (Migrated from github.com)

Хорошая идея с именем! Завтра сделаю

Если бы была возможность программно различать и подставлять разные тексты в зависимости от того, шарится в емейл или нет, то можно было бы рассмотреть что-то такое:

  1. Для емейлов вставлять в сабж адрес и имя, если они есть, или Open in Organic Maps если нет (текст можно продумать более полезный). При этом в идеале различать случаи шаринга своей текущей позиции и просто любой позиции. Чтобы просто по взгляду на сабж было видно, что это за точка (своя позиция чаще всего будет без имени и адреса). Например, I'm here now, или My position in Organic Maps.
  2. Для емейлов в бади можно вставлять больше текста и ссылок. Начиная с альтернативных geo:// урлов и заканчивая прямой ссылкой на установку OM, как было раньше.
  3. Для не-емейлов оставлять всё максимально кратко, но по возможности тоже различать случай шаринга текущей позиции. Почему это важно? Чтобы при взгляде на сообщение было понятно, что "ага, он(а) сейчас вон там", без дополнительных действий при отправке и при прочтении. I'm here: https://omaps.app/xxx, или I'm at Cafe Numi, Zergstrasse 12, Zumikon https://omaps.app/xxx
> Хорошая идея с именем! Завтра сделаю Если бы была возможность программно различать и подставлять разные тексты в зависимости от того, шарится в емейл или нет, то можно было бы рассмотреть что-то такое: 1. Для емейлов вставлять в сабж адрес и имя, если они есть, или Open in Organic Maps если нет (текст можно продумать более полезный). При этом в идеале различать случаи шаринга своей текущей позиции и просто любой позиции. Чтобы просто по взгляду на сабж было видно, что это за точка (своя позиция чаще всего будет без имени и адреса). Например, I'm here now, или My position in Organic Maps. 2. Для емейлов в бади можно вставлять больше текста и ссылок. Начиная с альтернативных geo:// урлов и заканчивая прямой ссылкой на установку OM, как было раньше. 3. Для не-емейлов оставлять всё максимально кратко, но по возможности тоже различать случай шаринга текущей позиции. Почему это важно? Чтобы при взгляде на сообщение было понятно, что "ага, он(а) сейчас вон там", без дополнительных действий при отправке и при прочтении. I'm here: https://omaps.app/xxx, или I'm at Cafe Numi, Zergstrasse 12, Zumikon https://omaps.app/xxx
Author
Member

Хорошая идея с именем! Завтра сделаю

Различать способ пересылки динамически не нашёл как.
Так что тогда получится дублирование имени точки и в теме, и в теле, что не очень, имхо.

> Хорошая идея с именем! Завтра сделаю Различать способ пересылки динамически не нашёл как. Так что тогда получится дублирование имени точки и в теме, и в теле, что не очень, имхо.
Author
Member

Переделал для андроида, вот примеры:
(subj используется только для емейлов, тело одинаковое и для емейлов, и для мессенджеров)

subj = Chechevitsa

Chechevitsa
Первомайская улица, 148А
https://omaps.app/w5gSA2uMPP/Chechevitsa
56.63404, 47.88158

если названия нет, то используется адрес:

subj = Первомайская улица, 148А

Первомайская улица, 148А
https://omaps.app/w5gSA2uMPP/Первомайская улица%2C_148А
56.63404, 47.88158

если и адреса нет, то дефолтный subj

subj = Hey, check out my pin in Organic Maps!

https://omaps.app/w5gSA2uMPP
56.63404, 47.88158

Unknown Place фильтруется, как непонятный и бесполезный

subj = Hey, check out my pin in Organic Maps!

https://omaps.app/w5gSA2uMPP
56.63404, 47.88158

my position

subj = Hey, check out my current location on the Organic Maps map!

Hey, check out my current location on the Organic Maps map!
https://omaps.app/w5gSA2uMPP/Hey%2C_check_out_my_current_location_on_the_Organic_Maps_map%21
56.63404, 47.88158

можно текст сменить на покороче, например I am here

Переделал для андроида, вот примеры: (subj используется только для емейлов, тело одинаковое и для емейлов, и для мессенджеров) ``` subj = Chechevitsa Chechevitsa Первомайская улица, 148А https://omaps.app/w5gSA2uMPP/Chechevitsa 56.63404, 47.88158 ``` если названия нет, то используется адрес: ``` subj = Первомайская улица, 148А Первомайская улица, 148А https://omaps.app/w5gSA2uMPP/Первомайская улица%2C_148А 56.63404, 47.88158 ``` если и адреса нет, то дефолтный subj ``` subj = Hey, check out my pin in Organic Maps! https://omaps.app/w5gSA2uMPP 56.63404, 47.88158 ``` Unknown Place фильтруется, как непонятный и бесполезный ``` subj = Hey, check out my pin in Organic Maps! https://omaps.app/w5gSA2uMPP 56.63404, 47.88158 ``` my position ``` subj = Hey, check out my current location on the Organic Maps map! Hey, check out my current location on the Organic Maps map! https://omaps.app/w5gSA2uMPP/Hey%2C_check_out_my_current_location_on_the_Organic_Maps_map%21 56.63404, 47.88158 ``` можно текст сменить на покороче, например `I am here`
biodranik commented 2022-01-10 10:24:47 +00:00 (Migrated from github.com)

Так что тогда получится дублирование имени точки и в теме, и в теле, что не очень, имхо.

Дублировать точно не надо. На сколько я понял, при отправке в мессенджеры subject склеивается с телом, так что можно не дублировать. Должно само работать верно, хотя бы в урезанном виде (без if (email) then …)

> Так что тогда получится дублирование имени точки и в теме, и в теле, что не очень, имхо. Дублировать точно не надо. На сколько я понял, при отправке в мессенджеры subject склеивается с телом, так что можно не дублировать. Должно само работать верно, хотя бы в урезанном виде (без if (email) then …)
Author
Member

Так что тогда получится дублирование имени точки и в теме, и в теле, что не очень, имхо.

Дублировать точно не надо. На сколько я понял, при отправке в мессенджеры subject склеивается с телом, так что можно не дублировать. Должно само работать верно, хотя бы в урезанном виде (без if (email) then …)

Нет, не так. Сейчас дублирует мой код.
Андроидом сабжект используется только для емейла, в остальных случаях он просто игнорируется.
Т.е. не получится сделать так, чтобы название/адрес были для емейла только в сабжекте, а для остальных - в теле.

Если не дублировать название/адрес в сабжекте, то вариант только дефолтный текст в нём иметь, например

subj = Hey, check out my pin in Organic Maps!

Chechevitsa
Первомайская улица, 148А
https://omaps.app/w5gSA2uMPP/Chechevitsa
56.63404, 47.88158

для my position логичнее всё же продублировать текст, имхо
(чем делать subj = Hey, check out my pin in Organic Maps!)

subj = Hey, check out my current location on the Organic Maps map!

Hey, check out my current location on the Organic Maps map!
https://omaps.app/w5gSA2uMPP/Hey%2C_check_out_my_current_location_on_the_Organic_Maps_map%21
56.63404, 47.88158
> > Так что тогда получится дублирование имени точки и в теме, и в теле, что не очень, имхо. > > Дублировать точно не надо. На сколько я понял, при отправке в мессенджеры subject склеивается с телом, так что можно не дублировать. Должно само работать верно, хотя бы в урезанном виде (без if (email) then …) Нет, не так. Сейчас дублирует мой код. Андроидом сабжект используется только для емейла, в остальных случаях он просто игнорируется. Т.е. не получится сделать так, чтобы название/адрес были для емейла только в сабжекте, а для остальных - в теле. Если не дублировать название/адрес в сабжекте, то вариант только дефолтный текст в нём иметь, например ``` subj = Hey, check out my pin in Organic Maps! Chechevitsa Первомайская улица, 148А https://omaps.app/w5gSA2uMPP/Chechevitsa 56.63404, 47.88158 ``` для my position логичнее всё же продублировать текст, имхо (чем делать `subj = Hey, check out my pin in Organic Maps!`) ``` subj = Hey, check out my current location on the Organic Maps map! Hey, check out my current location on the Organic Maps map! https://omaps.app/w5gSA2uMPP/Hey%2C_check_out_my_current_location_on_the_Organic_Maps_map%21 56.63404, 47.88158 ```
biodranik commented 2022-01-10 10:55:10 +00:00 (Migrated from github.com)

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

Как тогда в текущем мастере подставляется для емейла сабжект, а для месенджеров адрес?

> Андроидом сабжект используется только для емейла, в остальных случаях он просто игнорируется. Как тогда в текущем мастере подставляется для емейла сабжект, а для месенджеров адрес?
Author
Member

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

Как тогда в текущем мастере подставляется для емейла сабжект, а для месенджеров адрес?

В сабжект ложится дефолтная строка, а в тело адрес.
Андроид для мессенджеров отбрасывает сабжект, остаётся только тело с адресом.
А в емейле отправляется и сабжект со строкой, и тело с адресом.

    final String subject = MapObject.isOfType(MapObject.MY_POSITION, object) ?
                           context.getString(R.string.my_position_share_email_subject) :
                           context.getString(R.string.bookmark_share_email_subject);
    intent.putExtra(Intent.EXTRA_SUBJECT, subject);

...

    final String address = TextUtils.isEmpty(object.getAddress()) ? object.getName() : object.getAddress();
    final String text = context.getString(R.string.my_position_share_email, address, geoUrl, httpUrl);
    intent.putExtra(Intent.EXTRA_TEXT, text);

upd: либо вот так (в мастере три функции сильно дублирующиеся)

    final String subject = context.getString(R.string.bookmark_share_email_subject);
    intent.putExtra(Intent.EXTRA_SUBJECT, subject);

...

    StringBuilder text = new StringBuilder();
    text.append(bookmark.getName());
    if (!TextUtils.isEmpty(bookmark.getAddress()))
    {
      text.append(UiUtils.NEW_STRING_DELIMITER);
      text.append(bookmark.getAddress());
    }
    
...

    intent.putExtra(Intent.EXTRA_TEXT, text.toString());
> > Андроидом сабжект используется только для емейла, в остальных случаях он просто игнорируется. > > Как тогда в текущем мастере подставляется для емейла сабжект, а для месенджеров адрес? В сабжект ложится дефолтная строка, а в тело адрес. Андроид для мессенджеров отбрасывает сабжект, остаётся только тело с адресом. А в емейле отправляется и сабжект со строкой, и тело с адресом. ``` final String subject = MapObject.isOfType(MapObject.MY_POSITION, object) ? context.getString(R.string.my_position_share_email_subject) : context.getString(R.string.bookmark_share_email_subject); intent.putExtra(Intent.EXTRA_SUBJECT, subject); ... final String address = TextUtils.isEmpty(object.getAddress()) ? object.getName() : object.getAddress(); final String text = context.getString(R.string.my_position_share_email, address, geoUrl, httpUrl); intent.putExtra(Intent.EXTRA_TEXT, text); ``` upd: либо вот так (в мастере три функции сильно дублирующиеся) ``` final String subject = context.getString(R.string.bookmark_share_email_subject); intent.putExtra(Intent.EXTRA_SUBJECT, subject); ... StringBuilder text = new StringBuilder(); text.append(bookmark.getName()); if (!TextUtils.isEmpty(bookmark.getAddress())) { text.append(UiUtils.NEW_STRING_DELIMITER); text.append(bookmark.getAddress()); } ... intent.putExtra(Intent.EXTRA_TEXT, text.toString()); ```
biodranik commented 2022-01-10 12:14:23 +00:00 (Migrated from github.com)

Почти идеально:

Переделал для андроида, вот примеры: (subj используется только для емейлов, тело одинаковое и для емейлов, и для мессенджеров)

subj = Chechevitsa

Chechevitsa
Первомайская улица, 148А
https://omaps.app/w5gSA2uMPP/Chechevitsa
56.63404, 47.88158

Поменять сабж и убрать пробел после запятой, чтобы даблкликом сразу выбирались обе координаты.

subj = Chechevitsa на картах Organic Maps

Chechevitsa
Первомайская улица, 148А
https://omaps.app/w5gSA2uMPP/Chechevitsa
56.63404,47.88158

если названия нет, то используется адрес:

subj = Первомайская улица, 148А

Первомайская улица, 148А
https://omaps.app/w5gSA2uMPP/Первомайская улица%2C_148А
56.63404, 47.88158

Выше добавить "on the Organic Maps" в сабж
"Первомайская улица, 148А на картах Organic Maps" на русском

если и адреса нет, то дефолтный subj

subj = Hey, check out my pin in Organic Maps!

https://omaps.app/w5gSA2uMPP
56.63404, 47.88158

Слишком длинно, лучше Shared location on the Organic Maps

Unknown Place фильтруется, как непонятный и бесполезный

Правильно фильтруется, но надо короче сабж (см выше)

my position

subj = Hey, check out my current location on the Organic Maps map!

Hey, check out my current location on the Organic Maps map!
https://omaps.app/w5gSA2uMPP/Hey%2C_check_out_my_current_location_on_the_Organic_Maps_map%21
56.63404, 47.88158

subj: I am here on the Organic Maps

Почти идеально: > Переделал для андроида, вот примеры: (subj используется только для емейлов, тело одинаковое и для емейлов, и для мессенджеров) > > ``` > subj = Chechevitsa > > Chechevitsa > Первомайская улица, 148А > https://omaps.app/w5gSA2uMPP/Chechevitsa > 56.63404, 47.88158 > ``` Поменять сабж и убрать пробел после запятой, чтобы даблкликом сразу выбирались обе координаты. ``` subj = Chechevitsa на картах Organic Maps Chechevitsa Первомайская улица, 148А https://omaps.app/w5gSA2uMPP/Chechevitsa 56.63404,47.88158 ``` > если названия нет, то используется адрес: > > ``` > subj = Первомайская улица, 148А > > Первомайская улица, 148А > https://omaps.app/w5gSA2uMPP/Первомайская улица%2C_148А > 56.63404, 47.88158 > ``` Выше добавить "on the Organic Maps" в сабж "Первомайская улица, 148А на картах Organic Maps" на русском > если и адреса нет, то дефолтный subj > > ``` > subj = Hey, check out my pin in Organic Maps! > > https://omaps.app/w5gSA2uMPP > 56.63404, 47.88158 > ``` Слишком длинно, лучше Shared location on the Organic Maps > Unknown Place фильтруется, как непонятный и бесполезный Правильно фильтруется, но надо короче сабж (см выше) > my position > > ``` > subj = Hey, check out my current location on the Organic Maps map! > > Hey, check out my current location on the Organic Maps map! > https://omaps.app/w5gSA2uMPP/Hey%2C_check_out_my_current_location_on_the_Organic_Maps_map%21 > 56.63404, 47.88158 > ``` subj: I am here on the Organic Maps
Author
Member

Всё сделал (кроме ios).

если и адреса нет, то дефолтный subj

subj = Hey, check out my pin in Organic Maps!

https://omaps.app/w5gSA2uMPP
56.63404, 47.88158

Слишком длинно, лучше Shared location on the Organic Maps

На три символа короче :)
Сделал My pin in Organic Maps.

subj = My pin in Organic Maps

https://omaps.app/w5gSA2uMPP
56.63404,47.88158

my position

subj = I am here on Organic Maps

I am here on Organic Maps
https://omaps.app/w5gSA2uMPP/I_am_here_on_Organic Maps
56.63404,47.88158
subj = Chechevitsa on Organic Maps

Chechevitsa
Первомайская улица, 148А
https://omaps.app/w5gSA2uMPP/Chechevitsa
56.63404,47.88158
Всё сделал (кроме ios). > > если и адреса нет, то дефолтный subj > > ``` > > subj = Hey, check out my pin in Organic Maps! > > > > https://omaps.app/w5gSA2uMPP > > 56.63404, 47.88158 > > ``` > > Слишком длинно, лучше Shared location on the Organic Maps На три символа короче :) Сделал `My pin in Organic Maps`. ``` subj = My pin in Organic Maps https://omaps.app/w5gSA2uMPP 56.63404,47.88158 ``` my position ``` subj = I am here on Organic Maps I am here on Organic Maps https://omaps.app/w5gSA2uMPP/I_am_here_on_Organic Maps 56.63404,47.88158 ``` ``` subj = Chechevitsa on Organic Maps Chechevitsa Первомайская улица, 148А https://omaps.app/w5gSA2uMPP/Chechevitsa 56.63404,47.88158 ```
Author
Member

Немного причесал iOS версию, может даже будет работать :) Логика там всё равно не совсем как в андроиде сейчас.

Немного причесал iOS версию, может даже будет работать :) Логика там всё равно не совсем как в андроиде сейчас.
biodranik commented 2022-01-10 14:29:20 +00:00 (Migrated from github.com)

Сделал My pin in Organic Maps.

Когда я получаю емейл с таким текстом, выглядит не очень понятно. Плюс люди же могут шарить не только букмарки, но и что угодно. Это "Объект на карте Organic Maps". Но можно оставить и my pin пока. Хотя my чуть смущает, близко к my location.

> Сделал `My pin in Organic Maps`. Когда я получаю емейл с таким текстом, выглядит не очень понятно. Плюс люди же могут шарить не только букмарки, но и что угодно. Это "Объект на карте Organic Maps". Но можно оставить и my pin пока. Хотя my чуть смущает, близко к my location.
biodranik commented 2022-01-10 22:03:36 +00:00 (Migrated from github.com)

Почему в Android 33 файла с локализациями, а в iOS — 38?

Почему в Android 33 файла с локализациями, а в iOS — 38?
Author
Member

Почему в Android 33 файла с локализациями, а в iOS — 38?

Я в андроиде 39 папок насчитал, а в айос 38 - потому что fr-CA нет.

> Почему в Android 33 файла с локализациями, а в iOS — 38? Я в андроиде 39 папок насчитал, а в айос 38 - потому что fr-CA нет.
biodranik (Migrated from github.com) reviewed 2022-01-11 11:52:47 +00:00
biodranik (Migrated from github.com) commented 2022-01-10 22:05:46 +00:00

Коротко же, можно и в одну строку.

Коротко же, можно и в одну строку.
biodranik (Migrated from github.com) commented 2022-01-10 22:13:14 +00:00

Конечно, хорошо бы пробовать получать адрес здания, где сейчас находится my position. Можно хотя бы в TODO вставить.

Конечно, хорошо бы пробовать получать адрес здания, где сейчас находится my position. Можно хотя бы в TODO вставить.
biodranik (Migrated from github.com) commented 2022-01-10 22:11:06 +00:00

А где ещё используется формат с пробелом? Может в ядре поменять лучше?

А где ещё используется формат с пробелом? Может в ядре поменять лучше?
@ -3044,95 +3008,70 @@
ja = Organic Mapsでピン情報を確認
ko = Organic Maps 지도에서 내 핀 보기
nb = Hei, se merket mitt på Organic Maps-kartet
biodranik (Migrated from github.com) commented 2022-01-10 22:17:53 +00:00

Точка использовалась только при построении маршрута. И звучит не понятно для простых людей. И что значит "моя"? Текущая позиция человека? Надо менять тексты.

Точка использовалась только при построении маршрута. И звучит не понятно для простых людей. И что значит "моя"? Текущая позиция человека? Надо менять тексты.
biodranik commented 2022-01-11 11:53:51 +00:00 (Migrated from github.com)

Удалить коммит с генерацией, отребейзить, перегенерить без af и hi

Удалить коммит с генерацией, отребейзить, перегенерить без af и hi
pastk reviewed 2022-01-11 11:57:51 +00:00
@ -3044,95 +3008,70 @@
ja = Organic Mapsでピン情報を確認
ko = Organic Maps 지도에서 내 핀 보기
nb = Hei, se merket mitt på Organic Maps-kartet
Author
Member

Предложения?

Предложения?
biodranik (Migrated from github.com) reviewed 2022-01-11 11:58:07 +00:00
biodranik (Migrated from github.com) commented 2022-01-11 11:58:06 +00:00

Здесь возвращалась строчка с типами фичи, то, что под названием пишется в PP (restaurant * italian). Полезная инфа для шаринга, если она есть. Для ведра тоже можно добавить, пускай после имени пишется в скобках, или через запятую.

Здесь возвращалась строчка с типами фичи, то, что под названием пишется в PP (restaurant * italian). Полезная инфа для шаринга, если она есть. Для ведра тоже можно добавить, пускай после имени пишется в скобках, или через запятую.
biodranik (Migrated from github.com) reviewed 2022-01-11 11:59:28 +00:00
@ -58,7 +58,7 @@ NSString * httpGe0Url(NSString * shortUrl)
auto const title = ^NSString *(PlacePageData *data)
{
if (!data || data.isMyPosition)
biodranik (Migrated from github.com) commented 2022-01-11 11:59:28 +00:00

Почему не share_my_position? Если выбрана текущая позиция на карте, то этот кейз вернёт пустую строку. А внизу share_my_position

Почему не share_my_position? Если выбрана текущая позиция на карте, то этот кейз вернёт пустую строку. А внизу share_my_position
pastk reviewed 2022-01-11 11:59:56 +00:00
Author
Member

думаю в PP точно используется для вывода координат, мож и ещё где
страшновато в ядре менять

думаю в PP точно используется для вывода координат, мож и ещё где страшновато в ядре менять
pastk reviewed 2022-01-11 12:00:25 +00:00
Author
Member

туду вставлю

туду вставлю
pastk reviewed 2022-01-11 12:03:15 +00:00
Author
Member

Я думал над этим. Перегруза не получится? Там многие в чате вообще без текста хотели бы.

Я думал над этим. Перегруза не получится? Там многие в чате вообще без текста хотели бы.
pastk reviewed 2022-01-11 12:04:36 +00:00
@ -58,7 +58,7 @@ NSString * httpGe0Url(NSString * shortUrl)
auto const title = ^NSString *(PlacePageData *data)
{
if (!data || data.isMyPosition)
Author
Member

Я не знаю, как оно реально в айфоне отображается, правлю вслепую :)

Я не знаю, как оно реально в айфоне отображается, правлю вслепую :)
pastk reviewed 2022-01-11 12:05:41 +00:00
@ -58,7 +58,7 @@ NSString * httpGe0Url(NSString * shortUrl)
auto const title = ^NSString *(PlacePageData *data)
{
if (!data || data.isMyPosition)
Author
Member

Лучше чтоб кто-нибудь другой за айфон взялся, мои правки разве что как заготовку использовать можно.

Лучше чтоб кто-нибудь другой за айфон взялся, мои правки разве что как заготовку использовать можно.
pastk reviewed 2022-01-11 13:17:09 +00:00
Author
Member

ещё в андроиде в RoutingController для чего-то используется, в ядро и айфон даже не заглядывал
scope расползается, хочется ограничить уже

ещё в андроиде в RoutingController для чего-то используется, в ядро и айфон даже не заглядывал scope расползается, хочется ограничить уже
Author
Member

Сделал. Пока генерить локализации не стал.

Сделал. Пока генерить локализации не стал.
biodranik (Migrated from github.com) reviewed 2022-01-11 13:39:38 +00:00
@ -113,3 +112,4 @@
self.isMyPosition ? L(@"share_my_position")
: self.data.previewData.title];
}
biodranik (Migrated from github.com) commented 2022-01-11 12:05:25 +00:00

Тут не про массив. Через квадратные скобки в ObjC вызываются методы (в Java и C++ через точку). Должно быть [@"" mutableCopy] или [NSMutableString stringWithCapacity:500] или [[NSMutableString alloc] init].

Тут не про массив. Через квадратные скобки в ObjC вызываются методы (в Java и C++ через точку). Должно быть `[@"" mutableCopy]` или `[NSMutableString stringWithCapacity:500]` или `[[NSMutableString alloc] init]`.
biodranik (Migrated from github.com) commented 2022-01-11 13:39:33 +00:00
  std::string result;
  PlacePagePreviewData * pd = self.data.previewData;
  for (auto const & str : {pd.title, pd.subtitle, pd.address, self.data.infoData.phone, url})
  {
    if (str.length)
      result += "\n" + str;
  }
  return @(result.c_str());
``` std::string result; PlacePagePreviewData * pd = self.data.previewData; for (auto const & str : {pd.title, pd.subtitle, pd.address, self.data.infoData.phone, url}) { if (str.length) result += "\n" + str; } return @(result.c_str()); ```
biodranik (Migrated from github.com) reviewed 2022-01-11 13:42:04 +00:00
biodranik (Migrated from github.com) commented 2022-01-11 13:42:04 +00:00

По названию не всегда понятно, что за фича. Если текст будет компактный, и неплохо смотреться как одна строка/предложение, то нужно вставлять. Например:
Siniola (cafe * vegetarian * WiFi), Bahnhof strasse 12, Zurich https://omaps.app/XXXXX/Siniola

По названию не всегда понятно, что за фича. Если текст будет компактный, и неплохо смотреться как одна строка/предложение, то нужно вставлять. Например: Siniola (cafe * vegetarian * WiFi), Bahnhof strasse 12, Zurich https://omaps.app/XXXXX/Siniola
biodranik (Migrated from github.com) reviewed 2022-01-11 13:44:16 +00:00
biodranik (Migrated from github.com) commented 2022-01-11 13:44:16 +00:00

Для емейлов, конечно же, переносы строк лучше. Они автоматом в пробелы превратятся в обычных мессенджерах?

Для емейлов, конечно же, переносы строк лучше. Они автоматом в пробелы превратятся в обычных мессенджерах?
pastk reviewed 2022-01-11 13:44:30 +00:00
@ -113,3 +112,4 @@
self.isMyPosition ? L(@"share_my_position")
: self.data.previewData.title];
}
Author
Member

спасибо за знания!
в последней версии я даже с [@"" mutableCopy] угадал :)

спасибо за знания! в последней версии я даже с `[@"" mutableCopy]` угадал :)
pastk reviewed 2022-01-11 13:47:35 +00:00
Author
Member

тут только лишний перевод строки в начале будет, ничего?

тут только лишний перевод строки в начале будет, ничего?
pastk reviewed 2022-01-11 13:51:15 +00:00
Author
Member

в андроиде думаю не превратятся
это если только сам мессенджер полученную строку захочет переформатировать

в андроиде думаю не превратятся это если только сам мессенджер полученную строку захочет переформатировать
pastk reviewed 2022-01-11 13:54:28 +00:00
Author
Member

если текст на непонятном языке из какой-нибудь страны, то даже можно спутать, где кончается название, а где начинается адрес :)
мне кажется с переносом строк попонятнее структура

если текст на непонятном языке из какой-нибудь страны, то даже можно спутать, где кончается название, а где начинается адрес :) мне кажется с переносом строк попонятнее структура
pastk reviewed 2022-01-11 13:56:53 +00:00
Author
Member

ещё в субт могут попасть эмодзи, типа wheelchair
мессенджеры наверное съедят, а вот если шарить в файл или текстовый редактор простой, то может могут какие-нибудь коды вылезти вместо эмодзи

ещё в субт могут попасть эмодзи, типа wheelchair мессенджеры наверное съедят, а вот если шарить в файл или текстовый редактор простой, то может могут какие-нибудь коды вылезти вместо эмодзи
pastk reviewed 2022-01-11 14:01:55 +00:00
Author
Member

а если шарить метку, то в субте название списка не вылезет?
вобщем мне кажется тут надо будет хорошо так тестить и исключения делать (например, для unknown place в субте координаты убирать)
можно отдельным pr потом, если желание появится

а если шарить метку, то в субте название списка не вылезет? вобщем мне кажется тут надо будет хорошо так тестить и исключения делать (например, для unknown place в субте координаты убирать) можно отдельным pr потом, если желание появится
pastk reviewed 2022-01-11 14:04:46 +00:00
Author
Member

добавил этот код

добавил этот код
pastk reviewed 2022-01-11 16:06:49 +00:00
Author
Member
error: invalid operands to binary expression ('const char [2]' and 'NSString *__strong const')
      result += "\n" + str;
                ~~~~ ^ ~~~
``` error: invalid operands to binary expression ('const char [2]' and 'NSString *__strong const') result += "\n" + str; ~~~~ ^ ~~~ ```
biodranik (Migrated from github.com) reviewed 2022-01-11 20:44:09 +00:00
biodranik (Migrated from github.com) commented 2022-01-11 20:43:09 +00:00
      return url;
```suggestion return url; ```
biodranik (Migrated from github.com) commented 2022-01-11 20:43:34 +00:00
      return url;
```suggestion return url; ```
biodranik (Migrated from github.com) reviewed 2022-01-11 20:45:03 +00:00
biodranik (Migrated from github.com) commented 2022-01-11 20:45:03 +00:00

Надо тестировать )

Надо тестировать )
biodranik (Migrated from github.com) reviewed 2022-01-11 20:46:05 +00:00
biodranik (Migrated from github.com) commented 2022-01-11 20:46:04 +00:00

мне кажется с переносом строк попонятнее структура

Да, верно, особенно для RTL языков.

> мне кажется с переносом строк попонятнее структура Да, верно, особенно для RTL языков.
biodranik (Migrated from github.com) reviewed 2022-01-11 20:57:07 +00:00
biodranik (Migrated from github.com) commented 2022-01-11 20:57:06 +00:00

тут только лишний перевод строки в начале будет, ничего?

Он был и в изначальном коде. Надо поменять на result += result.empty() ? str : "\n" + str;

> тут только лишний перевод строки в начале будет, ничего? Он был и в изначальном коде. Надо поменять на `result += result.empty() ? str : "\n" + str;`
biodranik (Migrated from github.com) reviewed 2022-01-11 23:15:27 +00:00
biodranik (Migrated from github.com) commented 2022-01-11 23:05:51 +00:00
    // TODO: try to obtain the address of the current position.
```suggestion // TODO: try to obtain the address of the current position. ```
biodranik (Migrated from github.com) commented 2022-01-11 23:07:24 +00:00
    en = A place on Organic Maps
```suggestion en = A place on Organic Maps ```
biodranik (Migrated from github.com) commented 2022-01-11 23:09:08 +00:00
    be = Каардынаты ў Organic Maps
```suggestion be = Каардынаты ў Organic Maps ```
@ -3044,95 +3008,70 @@
ja = Organic Mapsでピン情報を確認
ko = Organic Maps 지도에서 내 핀 보기
nb = Hei, se merket mitt på Organic Maps-kartet
biodranik (Migrated from github.com) commented 2022-01-11 23:07:58 +00:00
    ru = Координаты в Organic Maps
```suggestion ru = Координаты в Organic Maps ```
biodranik (Migrated from github.com) commented 2022-01-11 23:09:49 +00:00
    uk = Координати у Organic Maps
```suggestion uk = Координати у Organic Maps ```
biodranik (Migrated from github.com) commented 2022-01-11 23:14:29 +00:00
    be = Я зараз тут на мапе Organic Maps
```suggestion be = Я зараз тут на мапе Organic Maps ```
biodranik (Migrated from github.com) commented 2022-01-11 23:11:20 +00:00
    en = %1$@ on Organic Maps
    be = %1$@ ў Organic Maps
```suggestion en = %1$@ on Organic Maps be = %1$@ ў Organic Maps ```
vng (Migrated from github.com) requested changes 2022-01-12 18:00:25 +00:00
vng (Migrated from github.com) left a comment
  • iOS crashes now
  • No need to append share_my_position to the sharing url
- iOS crashes now - No need to append share_my_position to the sharing url
rtsisyk requested changes 2022-02-05 09:37:58 +00:00
rtsisyk left a comment
Owner

Folks, could you please illustrate proposed changes? For example,

Before this patch:

Hey, check out my current location on the Organic Maps map!
https://omaps.app/w5gSA2uMPP/Hey%2C_check_out_my_current_location_on_the_Organic_Maps_map%21
56.63404, 47.88158

After this patch:

https://omaps.app/w5gSA2uMPP/%21
56.63404, 47.88158
Folks, could you please illustrate proposed changes? For example, Before this patch: ``` Hey, check out my current location on the Organic Maps map! https://omaps.app/w5gSA2uMPP/Hey%2C_check_out_my_current_location_on_the_Organic_Maps_map%21 56.63404, 47.88158 ``` After this patch: ``` https://omaps.app/w5gSA2uMPP/%21 56.63404, 47.88158 ```
Author
Member

The current state of this PR is
Android change is done
iOS change is half done (its being worked on pastk-loc-share branch in the main repo) - waiting for someone to finish it @biodranik or @vng ?
after iOS is done we'll merge them together

The current state of this PR is Android change is done iOS change is half done (its being worked on `pastk-loc-share` branch in the main repo) - waiting for someone to finish it @biodranik or @vng ? after iOS is done we'll merge them together
biodranik commented 2022-02-10 13:50:06 +00:00 (Migrated from github.com)

Don't hurry. Let's do it properly.

Don't hurry. Let's do it properly.
LaoshuBaby (Migrated from github.com) reviewed 2022-10-22 20:31:11 +00:00
LaoshuBaby (Migrated from github.com) commented 2022-10-22 20:31:01 +00:00

zh-TW have same problem with zh-CN version

zh-TW have same problem with zh-CN version
LaoshuBaby (Migrated from github.com) commented 2022-10-22 20:29:02 +00:00

Your current Chinese translation is for deleted bookmark_share_email_subject, not for new share_coords_subject_default. Is this excepted? I don't know.

Your current Chinese translation is for deleted `bookmark_share_email_subject`, not for new `share_coords_subject_default`. Is this excepted? I don't know.
LaoshuBaby (Migrated from github.com) commented 2022-10-22 20:27:05 +00:00

Sorry, do you already changed share_my_position's English string to I am here on Organic Maps?

share_my_position's new zh translation is for old English string Hi, check out my current location in Organic Maps!, which in my_position_share_sms.

Sorry, do you already changed `share_my_position`'s English string to `I am here on Organic Maps`? `share_my_position`'s new zh translation is for old English string `Hi, check out my current location in Organic Maps!`, which in `my_position_share_sms`.
matheusgomesms (Migrated from github.com) approved these changes 2022-10-22 22:04:05 +00:00
matheusgomesms (Migrated from github.com) left a comment

I made small fixes into PT strings

I made small fixes into PT strings
jimcarst (Migrated from github.com) approved these changes 2022-10-23 15:37:24 +00:00
Zack-83 (Migrated from github.com) reviewed 2023-07-22 07:09:42 +00:00
Zack-83 (Migrated from github.com) commented 2023-07-22 07:09:42 +00:00
	<string name="share_coords_subject_default">Guarda il mio luogo salvato sulla mappa Organic Maps!</string>
	<!-- Text preceding shared "my position" coordinates -->

Being an Italian mother tongue, I replaced "You can see" with "watch" and "my favorite place" with "my place saved on"

```suggestion <string name="share_coords_subject_default">Guarda il mio luogo salvato sulla mappa Organic Maps!</string> <!-- Text preceding shared "my position" coordinates --> ``` Being an Italian mother tongue, I replaced "You can see" with "watch" and "my favorite place" with "my place saved on"
Zack-83 (Migrated from github.com) reviewed 2024-04-16 07:45:56 +00:00
@ -168,3 +168,2 @@
/* Subject for emailed position */
"my_position_share_email_subject" = "Guarda dove mi trovo attualmente sulla mappa Organic Maps!";
/* Subject for emailed map point / poi / bookmark coordinates, %1$@ is point's name or address */
Zack-83 (Migrated from github.com) commented 2024-04-16 07:45:55 +00:00
"share_coords_subject_default" = "Guarda il mio luogo salvato sulla mappa Organic Maps!";
```suggestion "share_coords_subject_default" = "Guarda il mio luogo salvato sulla mappa Organic Maps!"; ```
Zack-83 (Migrated from github.com) requested changes 2024-04-16 07:48:08 +00:00
Zack-83 (Migrated from github.com) left a comment

I find it good, but it does not solve #2261 . Please adjust the Italian language as from suggestions

I find it good, but it does not solve #2261 . Please adjust the Italian language as from suggestions
Zack-83 (Migrated from github.com) reviewed 2024-06-21 09:31:50 +00:00
Zack-83 (Migrated from github.com) commented 2024-06-21 09:31:50 +00:00

I would appreciate if someone with writing privileges implemented my language correction :)

I would appreciate if someone with writing privileges implemented my language correction :)
This repo is archived. You cannot comment on pull requests.
No labels
Accessibility
Accessibility
Address
Address
Android
Android
Android Auto
Android Auto
Android Automotive (AAOS)
Android Automotive (AAOS)
API
API
AppGallery
AppGallery
AppStore
AppStore
Battery and Performance
Battery and Performance
Blocker
Blocker
Bookmarks and Tracks
Bookmarks and Tracks
Borders
Borders
Bug
Bug
Build
Build
CarPlay
CarPlay
Classificator
Classificator
Community
Community
Core
Core
CrashReports
CrashReports
Cycling
Cycling
Desktop
Desktop
DevEx
DevEx
DevOps
DevOps
dev_sandbox
dev_sandbox
Directions
Directions
Documentation
Documentation
Downloader
Downloader
Drape
Drape
Driving
Driving
Duplicate
Duplicate
Editor
Editor
Elevation
Elevation
Enhancement
Enhancement
Epic
Epic
External Map Datasets
External Map Datasets
F-Droid
F-Droid
Fonts
Fonts
Frequently User Reported
Frequently User Reported
Fund
Fund
Generator
Generator
Good first issue
Good first issue
Google Play
Google Play
GPS
GPS
GSoC
GSoC
iCloud
iCloud
Icons
Icons
iOS
iOS
Legal
Legal
Linux Desktop
Linux Desktop
Linux packaging
Linux packaging
Linux Phone
Linux Phone
Mac OS
Mac OS
Map Data
Map Data
Metro
Metro
Navigation
Navigation
Need Feedback
Need Feedback
Night Mode
Night Mode
NLnet 2024-06-281
NLnet 2024-06-281
No Feature Parity
No Feature Parity
Opening Hours
Opening Hours
Outdoors
Outdoors
POI Info
POI Info
Privacy
Privacy
Public Transport
Public Transport
Raw Idea
Raw Idea
Refactoring
Refactoring
Regional
Regional
Regression
Regression
Releases
Releases
RoboTest
RoboTest
Route Planning
Route Planning
Routing
Routing
Ruler
Ruler
Search
Search
Security
Security
Styles
Styles
Tests
Tests
Track Recording
Track Recording
Translations
Translations
TTS
TTS
UI
UI
UX
UX
Walk Navigation
Walk Navigation
Watches
Watches
Web
Web
Wikipedia
Wikipedia
Windows
Windows
Won't fix
Won't fix
World Map
World Map
No milestone
No project
No assignees
3 participants
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: organicmaps/organicmaps-tmp#1821
No description provided.