Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Log-in UI finetuning #203

Open
soxofaan opened this issue Sep 13, 2021 · 3 comments
Open

Log-in UI finetuning #203

soxofaan opened this issue Sep 13, 2021 · 3 comments
Labels
enhancement New feature or request
Milestone

Comments

@soxofaan
Copy link
Member

A couple of remarks/ideas regarding current login UI on https://editor.openeo.cloud/

Screenshot from 2021-09-13 11-10-41
(FYI: I used Firefox 92 for screenshot)

This "landing view" is a bit confusing: you have buttons at the top and a button at the bottom. The ones at the top look like buttons (and even have a "cursor: pointer" look), but don't really do something, except acting as as tab switchers. The one at the bottom looks less like a button that does something (e.g.less rounded corners than the top ones, doesn't have "cursor: pointer"), and looks less important because it is smaller.

  • change the top button to a real tab representation
  • make bottom button look more like a real button that triggers an action (e.g. cursor: pointer)
  • above OIDC login button: add text to explain that a popup will be opened to set expectations for the user.
  • bottom buttons: make bit larger (e.g. same height as text fields of "Internal")
  • "No credentials": call it "guest mode"?
  • as "Internal" is something we want to avoid being used: make it least favorable option: put it last, grayed out or smaller text ?
@m-mohr m-mohr self-assigned this Sep 28, 2021
@m-mohr
Copy link
Member

m-mohr commented Oct 8, 2021

This "landing view" is a bit confusing: you have buttons at the top and a button at the bottom. The ones at the top look like buttons (and even have a "cursor: pointer" look), but don't really do something, except acting as as tab switchers. The one at the bottom looks less like a button that does something (e.g.less rounded corners than the top ones, doesn't have "cursor: pointer"), and looks less important because it is smaller. [...]

  • make bottom button look more like a real button that triggers an action (e.g. cursor: pointer)

The button at the bottom is the default (real) button style of the browsers, but these may differ a lot between vendors. So styling them (e.g. an active style that you need for tabs) is very hard as you may mess up with the default styling. At the same time, you want to give users what they expect. Here's an alternative with the native button look in my environment:

image

I don't think this makes it better? So keeping it as it is for now until a better solution comes to mind...
Also, if I want to get right of the default styling (which I should?), I'd need to do it throughout the whole editor, which will take a bit more effort.

  • change the top button to a real tab representation

I've tried that, but it doesn't look good at all. Also, see example above, some providers need multiple rows to show all options, which also is an issue with real tabs.

  • above OIDC login button: add text to explain that a popup will be opened to set expectations for the user.
  • bottom buttons: make bit larger (e.g. same height as text fields of "Internal")

Both done:
image

  • "No credentials": call it "guest mode"?

Hmm... yeah, that needs a rename, but I'm not sure yet how to call it. I think previously we had "Discovery mode", too.

  • as "Internal" is something we want to avoid being used: make it least favorable option: put it last, grayed out or smaller text ?

With regards to openEO Platform, I'd think that we should even remove it from production, see https://github.com/openEOPlatform/architecture-docs/issues/107
For all other back-ends, it's not sure what is the most favorable option (e.g. for GEE it's the most favorable option).

@soxofaan
Copy link
Member Author

soxofaan commented Oct 8, 2021

maybe switching to vertical tabs addresses a couple of issues raised here
rough sketch of how that might look:

Screenshot from 2021-10-08 22-33-04
(I didn't include icons in this mockup to not loose too much time on it)

@m-mohr
Copy link
Member

m-mohr commented Oct 11, 2021

Thanks for the mock. I'll keep this in the loop for a future iteration although there are likely changes required (e.g. description won'
t fit into the tabs as longer descriptions provided by the back-ends would break it...)

@m-mohr m-mohr added this to the v1.0 milestone Oct 11, 2021
@m-mohr m-mohr removed their assignment Oct 11, 2021
@m-mohr m-mohr added the enhancement New feature or request label Oct 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants