Skip to content

Conversation

@rhysyngsun
Copy link
Contributor

What are the relevant tickets?

Closes https://github.com/mitodl/hq/issues/4780

Description (What does it do?)

Tweaks a few of the colors on the departments listing page. The header background looks like it was probably already addressed.

Screenshots (if appropriate):

  • Desktop screenshots
    Screenshot 2024-07-09 at 15-30-47 Departments MIT Open

  • Mobile width screenshots
    Screen Shot 2024-07-09 at 15 49 58

How can this be tested?

Go to the departments listing page and verify the background and item highlight colors match the designs.

@rhysyngsun rhysyngsun added the Needs Review An open Pull Request that is ready for review label Jul 9, 2024
Copy link
Contributor

@ChristopherChudzicki ChristopherChudzicki left a comment

Choose a reason for hiding this comment

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

👍


return (
<Page>
<div>
Copy link
Contributor

Choose a reason for hiding this comment

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

div is fine, but a fragment <>...</> would also work

@ChristopherChudzicki ChristopherChudzicki added Waiting on author and removed Needs Review An open Pull Request that is ready for review labels Jul 9, 2024
@rhysyngsun rhysyngsun force-pushed the nl/dept-listing-ux-fixes branch from 96afee5 to b702937 Compare July 9, 2024 20:44
@rhysyngsun rhysyngsun merged commit 78e61f8 into main Jul 9, 2024
@rhysyngsun rhysyngsun deleted the nl/dept-listing-ux-fixes branch July 9, 2024 21:11
@odlbot odlbot mentioned this pull request Jul 10, 2024
27 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants