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

Fix the organization logo display and positioning #2

Merged
merged 1 commit into from
Mar 20, 2024

Conversation

sandroscosta
Copy link

  • fix the positioning of the org logo display
  • create a guard to only allow the logo to appear if ENABLED_ORG_LOGO is true
  • fix the platform logo on mobile display
  • hide course name on mobile

@sandroscosta sandroscosta self-assigned this Mar 19, 2024
@sandroscosta
Copy link
Author

@sandroscosta sandroscosta force-pushed the sandroscosta/feat/enable-org-logo branch 2 times, most recently from 71ba57b to 90b7705 Compare March 19, 2024 12:43
@sandroscosta sandroscosta requested a review from igobranco March 19, 2024 12:45
* fix: platform logo on mobile display
* fix: hide course name on mobile
@sandroscosta sandroscosta force-pushed the sandroscosta/feat/enable-org-logo branch from 90b7705 to ff837cd Compare March 19, 2024 12:51
Copy link
Member

@igobranco igobranco left a comment

Choose a reason for hiding this comment

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

LGTM ✔️

@sandroscosta sandroscosta merged commit 2399515 into nau/nutmeg.master Mar 20, 2024
4 checks passed
@sandroscosta sandroscosta deleted the sandroscosta/feat/enable-org-logo branch March 20, 2024 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants