oppia / oppia

A free, online learning platform to make quality education accessible for all.
https://www.oppia.org
Apache License 2.0
5.83k stars 4.08k forks source link

[Feature Request]: Redesign the splash page #19214

Open seanlip opened 11 months ago

seanlip commented 11 months ago

Is your feature request related to a problem? Please describe.

As part of a general project to improve the onboarding flow on the website, we would like to redesign the splash page to make it very clear:

Describe the solution (or solutions) you'd like

The proposed new splash page is detailed in these mocks (second column from left): https://www.figma.com/file/YWe7SqfUVjxlJLKTUn0UZa/Project-2?type=design&node-id=6050-159194&mode=design&t=v3mhfxAkI0l9V53z-0

Original PRD: https://docs.google.com/document/d/1r91ypya5HasdGsuLcuzt_utKik6xpIVe9koYGynl6kg/edit (this refers to the "splash page update" part of it).

Note that the resulting splash page should work well on both desktop and mobile.

Describe alternatives you've considered and rejected

No response

Additional context

This is filed based on the design issue here: https://github.com/oppia/design-team/issues/69

seanlip commented 11 months ago

Also /cc @juliafalarini, the designer for this project, who can answer questions if needed.

AFZL210 commented 11 months ago

@juliafalarini Can you please confirm when we should show the language selector tooltip?

Screenshot from 2023-12-05 21-01-35

AFZL210 commented 10 months ago

Update: Sorry for the delay. Will create a PR this week.

juliafalarini commented 10 months ago

@AFZL210 the details can be viewed on the prototype. Once they access the splash page, after 300ms we will show the tooltip. This tooltip will disappear after 8000ms. You can view all these details here

yashwardhan-jyani commented 9 months ago

Hey @juliafalarini , I had some questions -

  1. Which page do the "Start teaching" and "Start exploring" button lead to in the teachers and parents section?

    Do they go to this page - https://www.oppiatestserver.org/creator-guidelines just like in the old splash page!

Screenshot from 2024-02-16 14-26-18

  1. The text varies between web and tablet/mobile designs in some instances like-

Web -

Screenshot from 2024-02-16 14-42-52

Tablet -

Screenshot from 2024-02-16 14-43-04

Web -

Screenshot from 2024-02-16 14-44-00

Tablet -

Screenshot from 2024-02-16 14-44-12

Also, here is my work so far.. Please do tell what you think -

https://github.com/oppia/oppia/assets/100014271/d566c4e4-80cf-4ca0-9b9d-c8200f30c65c

juliafalarini commented 8 months ago

Hi @yashwardhan-jyani I have fix the issues directly in the file. Thanks!

AkashPaloju commented 7 months ago

hi @juliafalarini ! Could you please let me know if we can change the colors of the below elements for which the lighthouse accessibilty tests are failing because of low-contrast ratio ? These are the only buttons for which the test is failing, if we can address this , we can get 100 lighthouse score in a11y for the splash page. Thanks! image

seanlip commented 7 months ago

@AkashPaloju Go ahead and make the fixes you need. Provide a comment here showing before/after fixes for what you changed, and leave comments on the mocks as well. Then we can merge this and make further changes if needed. Thanks!

AkashPaloju commented 6 months ago

hi @juliafalarini ! Could you please let me know if we can change the colors of the below elements for which the lighthouse accessibilty tests are failing because of low-contrast ratio ? These are the only buttons for which the test is failing, if we can address this , we can get 100 lighthouse score in a11y for the splash page. Thanks! image

I have changed the colors as follows and got lighthouse score of 100.

splashlighthousetest.webm

Before image After image For this, I have used same colors which were used for the 'Classroom card' beside it.

Before image After image For this, I have used the primary-color - #00645c

Thanks!

seanlip commented 6 months ago

@AkashPaloju Thanks, I think the updated colours are fine. Can we get your PR merged?