techmatters / terraso-web-client

A React project that powers the frontend of Terraso platform.
GNU Affero General Public License v3.0
11 stars 3 forks source link

Moodle SSO - Enhance proposed login screen and revise the Terraso consent screen. #1007

Open DerekCaelin opened 1 year ago

DerekCaelin commented 1 year ago

RA has implemented a version of Terraso SSO but we want to improve upon the experience for the user.

Description

Improve Moodle login screen

image

https://learn-tst.landscapes.global/login/index.php

Problems with the current screen:

Terraso SSO screen

image

(This is seen once the user clicks on the Terraso icon)

Assumptions

References

Designs

Process

### Tasks
- [ ] https://github.com/techmatters/terraso-web-client/issues/1041
- [ ] https://github.com/techmatters/terraso-web-client/issues/1042

Acceptance criteria

Product

RA

Consent screen

Design

Hanal-code commented 1 year ago

RELEASED TO RA team: Moodle login screen: https://www.figma.com/file/V2vch484f2clPH0G4hRhRj/Moodle-SSO_V1.0?type=design&node-id=157%3A984&mode=design&t=RyWe8kKOekvCAgZc-1

DerekCaelin commented 1 year ago

@paulschreiber would you mind copy-editing the UI text? Happy to meet if it would be helpful.

UI Copy edit (G-doc)

paulschreiber commented 1 year ago

@DerekCaelin Added comments to the document.

DerekCaelin commented 1 year ago

@paulschreiber thanks for the review!

One comment: this line was meant to be displayed to users who created Terraso accounts as a part of the sign-up process as a signifier that they now have a Terraso account. Unlike ordinary Terraso users, these users will not land on the Terraso home screen but go straight to Learning Landscapes, so there is no visual communication that they have successfully created a Terraso account.

image

I'm tempted to keep it in unless we can think of another signifier.

paulschreiber commented 1 year ago

It'd be good to signify that a new account has been created, but that sentence didn't make sense in the position it was in (in the middle of authentication info). We can add it to the bottom paragraph which explains what the account does.

DerekCaelin commented 1 year ago

I've moved the sentance to be with the explanation of what the account does - @Hanal-code please take a look at the changes.

Hanal-code commented 1 year ago

@DerekCaelin @paulschreiber Thx for the copy edit. I updated the button label and tweak the visual a bit. Here are the updated mockups:

[NEW TO TERRASO & MOODLE]

image

[EXISTING TERRASO USER]

image
paulschreiber commented 1 year ago
  1. There are too many different font size here; we don't need that much hierarchy. Having everything at 16/18px would be fine.
  2. Remove the link to tools — we don't want people getting distracted/leaving the signup flow.
Hanal-code commented 1 year ago

UPDATED MOCKUP

image