TUM-Dev / campus_flutter

Development of the cross-platform TUM Campus App with Flutter - for and from students at Technical University of Munich.
https://tum.app/
GNU General Public License v3.0
34 stars 4 forks source link

[Enhancement] Nudge Users to Log In #248

Closed CommanderStorm closed 4 months ago

CommanderStorm commented 4 months ago

Is your feature request related to a problem? Please describe. Currently, when a user is not logged in (f.ex. via skipping auth), they get "Token invalid" errors on the screens which require a token. I think it might be more user-friendly to show a login message instead.

Describe the solution you'd like We likely should order another illustration for this. What are you peoples ideas?

Describe alternatives you've considered Just display a plain "this view can only be displayed if you log in"->"Log in" message

Additional context From feedback of users via the feedback chanel ^^

jakobkoerber commented 4 months ago

Like so? 😄

image

CommanderStorm commented 4 months ago

Looks good to me as an inital one. Maybe bit bigger "login needed" with larger bottom padding and adding the text-wrap: balanced equivalent to the second one to make this feel more "centered" (all just a feeling).

Think that ordering another illustration might be beneficial as this is not an error => not 1 picture for everything Do you have any ideas what should be on said illustration?

CommanderStorm commented 4 months ago

Maybe also relevant: How does the dashboard/frontpage/leftmost page look? Can we add a login button there too or would this be too crowded?

jakobkoerber commented 4 months ago

Maybe also relevant: How does the dashboard/frontpage/leftmost page look? Can we add a login button there too or would this be too crowded?

This would be way too crowded in my opinion 😅

CommanderStorm commented 4 months ago

Maybe clicking on where we display the name if logged in could redirect to the login page, but likely the above approach is good enough.