Code review comment for lp:~widelands-dev/widelands/bug-1827786-metaserver-login-box-clean-start

Revision history for this message
kaputtnik (franku) wrote :

I am not convinced by this solution, it is confusing:

1. We have two places (views) which interact with the same thing now.
2. We have two buttons, 'Logout' and 'Back', in the lobby now, which do the same at first sight: Go back to the 'Multiplayer' view. How will a tooltip, if there were any, look to explain the buttons?

Having it like before is better, imho, although the small icon (button) to show the loginbox is visually disturbing. But better a visually disturbing thing than a confusing UI, especially because the Multiplayer view is mostly open for a short time.

Just my personal opinion :)

« Back to merge proposal