Add physiotherapist #3844
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#3844
Loading…
Add table
Reference in a new issue
No description provided.
Delete branch "master"
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?
To fix/close #3831 I made the following changes:
I was not able to completely test the changes.
The map I created locally doe not show the new icons neither anything else at the location where I expect a physiotherapist.
Thus I can neither test the icon and if the strings on the place page are correct.
I generated the map with
python -m maps_generator --countries="Germany_Berlin" --skip="Coastline"
.I'll continue to find out what I am doing wrong / which step I am missing.
Any help or hint is appreciated.
@vng can you please take a look?
Add healthcare=physiotherapist to other places in this file, exactly like healthcare=laboratory.
It should also have {text: name} style.
I added the entries in
data/styles/clear/include/Icons.mapcss
and regenerated the styles.However on my locally built map, I still do not see anything.
What I did
tools/unix/generate_symbols.sh
tools/unix/generate_drules.sh
python -m maps_generator --countries="Germany_Berlin" --skip="Coastline"
You didn't edit data/styles/vehicle/include/Basemap.mapcss and data/styles/clear/include/Basemap.mapcss
This is enough. Now you should see therapist icons, right?
Basemap.mapcss can be skipped here ..
physiotherapist should be capitalized
also: please add estonian.
et = Füsioterapeut
@ -4570,0 +4588,4 @@
ja:理学療法士
ko:물리치료사
mr:फिजिओथेरपिस्ट
nb:Fysioterapeut
et:Füsioterapeut
Thanks fixed in organicmaps/organicmaps#3844/commits/c6d60829ca1adb2318cb47df31d48933a2abfede
@ -4570,0 +4588,4 @@
ja:理学療法士
ko:물리치료사
mr:फिजिओथेरपिस्ट
nb:Fysioterapeut
Added in organicmaps/organicmaps#3844/commits/c6d60829ca1adb2318cb47df31d48933a2abfede
Yes I just forgot that I also need to rebuild the app.
@biodranik and @vng what about the Basemaps?
Could you shortly explain what this is for?
If I should add the changes there just let me know.
Right now @biodranik says "yes" and @vng "no" ;)
There is no any specific purpose on different
*.mapcss
files, just some logical code distinguish, made a long time ago. I usually check the final styles indrules_proto_*.txt
If they are ok, no problem here ..We have a lot of bloat mapcss code now (many useless entries), so the basic rule is to achieve needed behaviour with the minimum changes. I suspect that if you make additional therapist changes in Basemap files, you will get the same drules_proto output anyway ..
Please, sign your commits with DCO and make 4 final commits:
The style of symbol in the PR is like the sports symbols

or some other minor things on the map 
.
Shoudn't this symbol be in the style of the hospital symbol
, or the pharmacy symbol
?
I see your point there, as it is still in the medical/doctors domain, it would be good to have a similar coloring and shape.

This is how OsmAnd App shows a normal doctor on the right and a physiotherapist on the left.
I will test my svg skills and create a icon similar to the pharmacy one but with the proposed physiotherapist icon inside.
@dvdmrtnz What do you think about these?
I personally like the rounded one more.
Other opinions and improvement ideas are welcome!
I like the rounded one more, too. Good job! 👍
@vng and / or @biodranik I'd like to have a opinion from a member, regarding the icon decision.
Afterwards I make the changes and push new commits.
The rounded icon looks good, thanks for your help!
Just make sure that it has the same color as other icons/doctor icons in light and dark modes.
I added the new icon and regenerated the symbols.
Here you can see it in light and night mode.
As visible it looks like the doctor icon regarding the coloring and the opacity (for night mode).
@j13m126 I had to remove the estonian translation from the
categories.txt
, as it crashed the android app on startup.It seems like as the estonian translation is not officially added yet and thus the app fails to handle the language code.
Snippet from the android log
Don't worry, we'll enable estonian translations later in a separate commit.
@ -4570,0 +4578,4 @@
el:Φυσικοθεραπευτής
es:Fisioterapeuta
eu:Fisioterapeuta
fa:فیزیوتراپیست
For the future reference: case in categories.txt file is not required, because these strings are used for search only, and are not visible to users.
Capital P
@ -4570,0 +4578,4 @@
el:Φυσικοθεραπευτής
es:Fisioterapeuta
eu:Fisioterapeuta
fa:فیزیوتراپیست
Okay thanks, I just orientated on the other entries.
But is noted for future work 👍