zeeguu / web

Frontend for the zeeguu web application.
https://www.zeeguu.org
3 stars 5 forks source link

Article Preview Small Redesign #394

Closed tfnribeiro closed 3 weeks ago

tfnribeiro commented 1 month ago

Since users have been confused about the Stats for an Article, I propose a small refactoring:

As we have a project soon to redesign this component of the app, I thought about suggesting these changes, but I understand if we just wait until we get a full redesign.

How it looks:

Desktop Mobile
image image

Current Look:

Desktop Mobile
image image

Different difficulty icons:

In the app:

image

Icons used:

A1 -> A2 -> B1 -> B2 -> C1 -> C2

image

With the Optional Backend change of showing Sources Name:

Desktop Mobile
image image
netlify[bot] commented 1 month ago

Deploy Preview for voluble-nougat-015dd1 ready!

Name Link
Latest commit 52adfac695a0a60d554485febe1335f189fc2f13
Latest deploy log https://app.netlify.com/sites/voluble-nougat-015dd1/deploys/666967aa4906180008efeef4
Deploy Preview https://deploy-preview-394--voluble-nougat-015dd1.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

tfnribeiro commented 3 weeks ago

Something I noticed for the Teacher Dashboard:

Current Look:

image

If the Teacher has too many classes the classes break:

image

This UI does not scale on the phone:

image

New Look:

image

It still doesn't scale to the phone:

image

mircealungu commented 3 weeks ago

Something I noticed for the Teacher Dashboard:

Current Look:

image

If the Teacher has too many classes the classes break:

image

This UI does not scale on the phone:

image

New Look:

image

It still doesn't scale to the phone:

image

Good catch with the teacher dashboard responsiveness... Let's nevertheless ignore it for now: there are too few teachers, and we don't know whether they even use the phone.

Is this ready to be merged?

tfnribeiro commented 3 weeks ago

Should be ready to merge, I refactored the two comments you made into their own components