[android] Reworked descriptions in the editor #6173
No reviewers
Labels
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
2 participants
Due date
No due date set.
Dependencies
No dependencies set.
Reference: organicmaps/organicmaps-tmp#6173
Loading…
Add table
Reference in a new issue
No description provided.
Delete branch "editor"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Reworked descriptions in the editor:
Todo:
Is this text replaced by another from strings.txt?
I now added
editor_note_description
to strings.txtThanks! The next step would be to make it simpler (if possible).
Now the note has 3 texts:
Can we squash them and make a better wording text, and use only edit text hint, without title and description? OM is about simplicity and efficiency ;-) Maybe in a separate PR?
@ -476,7 +477,8 @@ public class EditorFragment extends BaseMwmFragment implements View.OnClickListe
mOpeningHours.setOnClickListener(this);
Is cardMore used somewhere? Can it be deleted?
?
What do you think about this?
I think, with the description below,
OpenStreetMap.org
would be rather redundant@ -476,7 +477,8 @@ public class EditorFragment extends BaseMwmFragment implements View.OnClickListe
mOpeningHours.setOnClickListener(this);
It's used in line 468 for finding the mDescription
I think we can remove
public
in the sentenceNo, it's critically important, based on our experience.
I check all OM edits with a boundary box intersecting munich and there are regularly people who upload personal notes
I updated the description
Some improvements for layout
Same for other TextView
Thanks! Will the centered top warning look better than the left-aligned one?
@ -16589,0 +16588,4 @@
[editor_note_hint]
comment = Hint of the input field in the OSM note section of the editor
tags = android
(optional)
from here and writeOptional note to OpenStreetMap volunteers
in the title?@ -16589,0 +16592,4 @@
en = Describe errors on the map or things that cannot be edited with Organic Maps
af = Beskryf foute in die kaart of wat nie met Organiese Kaarte geredigeer kan word nie
ar = وصف الأخطاء على الخريطة أو الأشياء التي لا يمكن تحريرها باستخدام الخرائط العضوية
az = Xəritədəki səhvləri və ya Üzvi Xəritələrdən istifadə edərək redaktə edilə bilməyənləri təsvir edin
@ -16589,0 +16588,4 @@
[editor_note_hint]
comment = Hint of the input field in the OSM note section of the editor
tags = android
The shorter version is good. For the 'optional' I prefer the version after the hint in brackets, it stands out more that way.
Let's check how it looks with new strings (also check the longest translations language).
@ -16589,0 +16588,4 @@
[editor_note_hint]
comment = Hint of the input field in the OSM note section of the editor
tags = android
The third option "Note to OpenStreetMap volunteers (optional)" is also good, so let's proceed with that one
The longest string, stretching over two lines, works as well:

@ -16589,0 +16586,4 @@
zh-Hans = OpenStreetMap 志愿者须知(可选)
zh-Hant = OpenStreetMap 志工注意事項(可選)
[editor_note_hint]
@map-per adding
comment =
with a bit more context would also be helpful.@organicmaps/translations please help with reviewing these translations. Here is the image with the example of their use:
@ -16589,0 +16647,4 @@
el = Οι επεξεργασίες σας μεταφορτώνονται στη δημόσια βάση δεδομένων <a href="https://wiki.openstreetmap.org/wiki/El:About_OpenStreetMap">OpenStreetMap</a>. Παρακαλούμε μην προσθέτετε προσωπικές πληροφορίες ή πληροφορίες που προστατεύονται από πνευματικά δικαιώματα.
es = Sus ediciones se cargan en la base de datos pública <a href="https://wiki.openstreetmap.org/wiki/ES:Acerca_de_OpenStreetMap">OpenStreetMap</a>. Por favor, no añada información personal o protegida por derechos de autor.
et = Teie muudatused laaditakse üles avalikku <a href="https://wiki.openstreetmap.org/wiki/About_OpenStreetMap">OpenStreetMap</a> andmebaasi. Palun ärge lisage isiklikku või autoriõigusega kaitstud teavet.
eu = Zure aldaketak <a href="https://wiki.openstreetmap.org/wiki/About_OpenStreetMap">OpenStreetMap</a> datu-base publikora kargatzen dira. Mesedez, ez gehitu informazio pertsonalik edo copyrightdun informaziorik.
Other auto-translations are good
Hu LGTM
nl is OK
Thanks! @kirylkaveryn can you please check if it's easy to make similar changes on iOS, and if the link in the text is displayed properly/is clickable (or the whole text label is clickable)?
Is the text/url clickable now?
Yes, the blue text is clickable. The removed code is form the old separate 'About OpenStreetMap' textview.