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

Improvement: Redesign Course Cards #380

Merged
merged 11 commits into from
Feb 12, 2025

Conversation

julian-wls
Copy link
Contributor

@julian-wls julian-wls commented Feb 3, 2025

Problem Description

Compared to the iOS app and the web app the dashboard of the Android app does not really fit. Course cards are designed in a different way on all other platforms.

Changes

This PR changes the design of the dashboard and restructures the logic behind course cards and how there are used in the grid layout:

  • Redesign of the dashboard
    - new icon design for the top bar
    - new design for the course enrollment screen

This PR is related to #350, but doesn't close it yet, because I will create a follow up PR to add the recent section.

Steps for testing

Use Artemis test user 2 on TS to test the following:
- Scroll through the dashboard and check if there are any issues with the new design
- Click on an exercise below next exercise if there is any and check if you're redirected to that exercise
- Navigate to the course enrollment screen and check the design for the preview course items
- Login to TS1 or TS to check the Circular Progress that is visible for courses that have started exercises.


Screenshots

@julian-wls julian-wls self-assigned this Feb 3, 2025
@julian-wls julian-wls added the ready for review This PR can be reviewed label Feb 5, 2025
@julian-wls julian-wls marked this pull request as ready for review February 5, 2025 10:14
Copy link
Collaborator

@FelberMartin FelberMartin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like this change, the feels much more consistent with web and iOS now 👌

I noticed one issue that has been there before, but maybe it makes sense to tackle it as part of this PR as it should not be a huge change:
The contrast on some course cards is quite bad (eg if the course color is yellow, see screenshot). I know this is also this way on the webapp, but I think we can do better. We could use a simple check with android.graphics.Color.luminance() to switch the text color to black for bright surfaces. What do you think?
image

@julian-wls
Copy link
Contributor Author

I noticed one issue that has been there before, but maybe it makes sense to tackle it as part of this PR as it should not be a huge change: The contrast on some course cards is quite bad (eg if the course color is yellow, see screenshot). I know this is also this way on the webapp, but I think we can do better. We could use a simple check with android.graphics.Color.luminance() to switch the text color to black for bright surfaces. What do you think?

I agree with you, however I don't think it's a good idea to deviate from the design of the other platforms. I think all platforms should be consistent regarding this. But I 100% agree with you. My idea would be to maybe discuss this in the meeting. I don't think it's our call to make.

Copy link
Contributor

@eylulnc eylulnc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Its looking really good and agree with its alignment to web and iOS now 😊.
Works as described ✅

@julian-wls julian-wls added ready to merge This PR can be merged and removed ready for review This PR can be reviewed labels Feb 12, 2025
@FelberMartin FelberMartin merged commit 1949fec into develop Feb 12, 2025
5 checks passed
@FelberMartin FelberMartin deleted the improvement/redesign-dashboard branch February 12, 2025 17:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge This PR can be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants