Improve flow of adding Place to OSM, fix bad state #5727

Closed
opened 2023-08-15 22:22:22 +00:00 by privatemaker · 0 comments
privatemaker commented 2023-08-15 22:22:22 +00:00 (Migrated from github.com)

Describe the issue

While attempting to add a Place when clicking the back button the user is taken to an incorrect state.

Steps to reproduce

The following triggers the bad state.

  1. Select a point on map
  2. Open drawer and click Add Business
  3. Select appropriate category
  4. Fill out fields of data for Place / Business
  5. Click the confirm checkmark in top right (Red Circle 1)

The communicates to user that they should confirm saving the data they just input. It is unclear (to me) if the Place data has been saved somewhere, or if clicking saves state.

Upon clicking the user is taken to Login to Open Street Maps webview.

  1. At the Open Street Map login, click the back button (Red Circle 2)
  2. User is taken to Categories Search view from Step 3.

The categories search field is a bad state. It is confusing to user where they are.
If the user clicks on the appropriate category, the input fields are partially incomplete from data they just entered.

Expected behavior

The confirm button in Step 5. should not take the user directly to OSM login screen.
The button should take the user back to the map view with the newly added Place selected.

Additional context

To submit data to OSM, there should be a toggle switch in the Add Place form.

Conribute Place to OSM   [Off / On]

The toggle should default to Off on each new Place unless it is editable in a global setting.

Upon toggling to On if a user is logged into OSM, then nothing unusual happens, the data is sent.

If a user is not logged into OSM, then it should pop-up the OSM login screen at that moment.

**Describe the issue** While attempting to add a Place when clicking the back button `⭠` the user is taken to an incorrect state. **Steps to reproduce** The following triggers the bad state. 1. Select a point on map 2. Open drawer and click `Add Business` 3. Select appropriate category 4. Fill out fields of data for Place / Business 5. Click the confirm checkmark `✓` in top right (Red Circle 1) The `✓` communicates to user that they should confirm saving the data they just input. It is unclear (to me) if the Place data has been saved somewhere, or if clicking `✓` saves state. Upon clicking `✓` the user is taken to `Login to Open Street Maps` webview. <img src="https://github.com/organicmaps/organicmaps/assets/49612519/e21568b0-f8f3-4c18-859c-1cdd9920597e" width="600"> 1. At the Open Street Map login, click the back `⭠` button (Red Circle 2) 2. User is taken to `Categories Search` view from Step `3.` The categories search field is a bad state. It is confusing to user where they are. If the user clicks on the appropriate category, the input fields are partially incomplete from data they just entered. **Expected behavior** The confirm `✓` button in Step `5.` should not take the user directly to OSM login screen. The button should take the user back to the map view with the newly added Place selected. **Additional context** To submit data to OSM, there should be a toggle switch in the Add Place form. ``` Conribute Place to OSM [Off / On] ``` The toggle should default to `Off` on each new Place unless it is editable in a global setting. Upon toggling to `On` if a user is logged into OSM, then nothing unusual happens, the data is sent. If a user **is not logged into** OSM, then it should pop-up the OSM login screen at that moment.
This repo is archived. You cannot comment on issues.
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
1 participant
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#5727
No description provided.