Skip to content
This repository has been archived by the owner on Apr 16, 2022. It is now read-only.

On the initial launch if the user press "Cancel" on briefcase storage location selection dialog, user is end up having a disabled main window #87

Closed
rclakmal opened this issue Apr 5, 2017 · 10 comments

Comments

@rclakmal
Copy link
Contributor

rclakmal commented Apr 5, 2017

Software versions

Briefcase v1.4.10, Java 1.8, OS Windows 7

Problem description

On the initial launch if the user press "Cancel" on briefcase storage location selection dialog, user is end up having a totally disabled main window.

Steps to reproduce the problem

  1. Just delete/move the briefcase storage directory and relaunch the app. You will get the storage location selection popup
  2. Press "Cancel"
  3. You are end up on a disabled main window with no option of going back. Only option is to close and relaunch.

b1

b2

Expected behavior

This would probably get fixed over the summer project for unified ui. A quick fix would be to remove the "Cancel" button. Because as of now, it doesn't serve any purpose

@yanokwa
Copy link
Member

yanokwa commented Apr 5, 2017

@rclakmal I'm not sure what to do here. Instead of removing the Cancel button, maybe we should quit the application? Or take the user to the Settings tab (and they can't leave that tab)?

Any other ideas @shivam-tripathi @icemc @nribeka?

@rclakmal
Copy link
Contributor Author

rclakmal commented Apr 5, 2017

@yanokwa Well technically, user shouldn't be able to cancel the selection of a folder since it's a mandatory requirement.

I think your idea of directing to Settings tab is nice. But then, why wouldn't we just redirect the user to Settings page and make them do this setting on the first launch instead of this popup dialog ?

@yanokwa
Copy link
Member

yanokwa commented Apr 5, 2017

Redirecting to Settings seems like it'll be more complicated. You'd then have to prevent them from doing anything until they select a folder.

Another option is instead of Cancel, we replace that with Quit.

@rclakmal
Copy link
Contributor Author

rclakmal commented Apr 5, 2017

My point was not adding redirection. Rather, get rid of the popup and start the main window with the focus on Settings view and not letting user to browse other view until a folder path is set.

However, Changing the button to a "Close/Quit" would be an easy option.

@rclakmal
Copy link
Contributor Author

rclakmal commented Apr 6, 2017

@yanokwa Shall I move forward with "Close" option and submit a PR?

@yanokwa
Copy link
Member

yanokwa commented Apr 6, 2017

So the options are:

  1. Change dialog option to Close/Quit
  2. Get rid of the popup, but lock the user to settings and add some language to settings screen to say that it's required.

I don't like the popup, but I bias towards Close/Quit as a solution because we'll probably be changing Briefcase's UX anyway and Close/Quit is a straightforward fix. Paging @lognaturel and @nribeka for feedback.

@rclakmal
Copy link
Contributor Author

rclakmal commented Apr 7, 2017

@yanokwa @nribeka @lognaturel I created a PR since it was a small change. We can further discuss this through that PR.

@shivam-tripathi
Copy link
Contributor

shivam-tripathi commented Apr 7, 2017

For a quick fix, in my opinion, we can change the "Cancel" button with a "Quit" button.
Settings tab can contain (in future) a lot of irrelevant requirements for a welcome page (presently it has only location option), for example giving options related to form falsification or addition of other features (in future) which may require some form of user settings. So it would be a design decision to whether make a "Welcome page" which shows all ODK storage locations in the system (which will be an empty list if no location has been registered previously) with an option to add a new one, or go with a pop up or use the settings tab.
Arguments in favor of a welcome screen revolves around the following:

  • It would only ask for information needed when the application is launched.
  • Launching briefcase again doesn't necessary mean that I want to use the same location folder. Upon each launch, user can select the location.
  • To avoid repetition of a welcome screen, we can give an option to "Remember location settings" with a message "You can always change your location settings in the 'Settings' tab". Next time, automatically load the forms from the previous location.
    Opinions?

@rclakmal
Copy link
Contributor Author

rclakmal commented Apr 7, 2017

Launching briefcase again doesn't necessary mean that I want to use the same location folder. Upon each launch, user can select the location.

Idea of multiple storage locations could make things little complex from UI perspective and as far as I see it doesn't add any extra value from user's perspective.

@yanokwa
Copy link
Member

yanokwa commented Apr 7, 2017

Thanks for all the ✨great✨ feedback.

I think we'll go with the Cancel/Quit option for now. I'll review the PR shortly. 👨🏿‍💻

lognaturel pushed a commit that referenced this issue Apr 13, 2017
… location selection dialog, user is end up having a disabled main window #87 (#91)

* Changed the 'Cancel' button to a 'Quit' button

* Settings page should still trigger folder change dialog with a "cancel"
rclakmal added a commit to rclakmal/briefcase that referenced this issue Apr 13, 2017
* master:
  On the initial launch if the user press "Cancel" on briefcase storage
location selection dialog, user is end up having a disabled main window
getodk#87 (getodk#91)
  Re-download deleted submission file during next pull from aggregate
server fixes getodk#92 (getodk#96)
  ui/ServerConnectionDialog: Trim spaces in the URL entered (getodk#97)
  Duplication not allowed when pulling forms from Collect
  Add checkstyle support (getodk#90)
  util/ExportToCsv: On export, only create media folder when needed
(getodk#75)
  Cleaner syntax, upgrade buildconfig (getodk#89)
  Fix packaging to use dependent jars (getodk#78)
  Add gradle support for JavaRosa (getodk#80)
  Included a chronological sorting for file instances in csv export
(getodk#66)
  Add additional SDCard mount point for Linux (getodk#74)
  Update CONTRIBUTING.md

# Conflicts:
#	src/org/opendatakit/briefcase/ui/FormTransferTable.java
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants