search

overhangio / tutor-indigo

An elegant, customizable theme for Open edX
GNU Affero General Public License v3.0
73 stars 275 forks source link

Tutor Indigo Theme Updates #53

Closed hinakhadim closed 8 months ago

hinakhadim commented 8 months ago

Regis, We've removed the static files which we haven't overridden. Also, Not a lot of changes we've made but comments has been added by Tanveer where he made changes in the html files. I've also added the ENV_PATCHES for the learning-mfe. This will install the header, footer and brand-openedx in learning-mfe image.

regisb commented 8 months ago

This is awesome work. I'm eager to see this live.

landing page

When unauthenticated, the "explore courses" menu in the top left corner looks off:

Screenshot from 2023-12-12 10-43-48

favicon

I think we should not use the "" logo for the favicon: it's not neutral enough. Instead, we should use the blue "<>" logo:

Screenshot from 2023-12-12 10-48-53

course about page

In the course "about" page, when I hover on the social sharing links, the "Share with friends and family" modal looks off. The background should not be black, the font should not be serif and not italic:

Screenshot from 2023-12-12 10-49-44

regisb commented 8 months ago

Closed in favour of #54