TrelloWidget icon indicating copy to clipboard operation
TrelloWidget copied to clipboard

Show login fragment as a dialog box

Open MatHazak opened this issue 2 years ago • 6 comments

I think it would make sense to not show the widget's settings until the user logs in. The login fragment can be displayed in a dialog box, and after the user is logged in, give them access to the main screen.

MatHazak avatar Sep 25 '23 20:09 MatHazak

I disagree that a dialog would be an improvement in this case. If the problem is the settings we can maybe hide them while the user is logged out.

oryanm avatar Oct 04 '23 22:10 oryanm

As a mobile user, I found this behavior a bit odd. My experience with other apps is the dialog approach. The LoginFragment doesn't grab the user's attention at first glance and is a bit confusing. From the Material Design guideline for dialogs:

A dialog is a modal window that appears in front of app content to provide critical information or ask for a decision. Dialogs disable all app functionality when they appear, and remain on screen until confirmed, dismissed, or a required action has been taken.

MatHazak avatar Oct 05 '23 20:10 MatHazak

The LoginFragment doesn't grab the user's attention

Again, if this is confusing we can hide the settings while the user is logged out, no? That would be a simpler solution to implement. The design guidelines are great to have but they're not laws we must follow.

oryanm avatar Oct 21 '23 21:10 oryanm

That would be a simpler solution to implement.

Is simplicity your only concern with this change? Implementing a dialog with Material Alert Dialog Builder is simpler and more concise than the current approach. It actually reduces the code and its complexity (we no longer need to login fragment and stuff to change it).

MatHazak avatar Nov 01 '23 22:11 MatHazak

It's more so the UI than how much code it take to write. I just think it's cleaner to avoid a popup in this case. Sorry I was away.. there's a lot going on.

oryanm avatar Nov 20 '23 02:11 oryanm

I just think it's cleaner to avoid a popup in this case.

As a UI POV, I gave my opinion earlier, and as I mentioned above, the Material Design guideline recommends a dialog in this case. It's not a priority at this point. We can leave this issue here until there is more participation from others.

Sorry I was away.. there's a lot going on.

No problem at all ^^ I myself am in the middle of the semester and can't devote enough time to the project.

MatHazak avatar Nov 28 '23 22:11 MatHazak