Update MUI secondary color

[mateoclarke]

Associated issues

Closes cityofaustin/atd-data-tech#17920

Testing

URL to test:
https://deploy-preview-1437--atd-moped-main.netlify.app/moped/dashboard

Steps to test:

• Open up a page with secondary text, for example, the welcome text on the dashboard.
• Confirm it is the updated shade of grey, #666666

---

Ship list

• Code reviewed
• Product manager approved
• Product manager checked DB view dependencies
• Product manager added to QA test script if applicable

[mddilley]

Looks great! Much easier to read the small text. I checked every view that references this color and every part of the app. I have one small request before merge that I could use some feedback on. Thanks for updating! 🙏

[mddilley]

This is outside the scope of the issue, but I noticed that we use text.header in only three places in the app where we have tabs: the Dashboard view, the main app header, and in the project view. For some reason, the app header tabs respect text.header but the other two do not.

This is making me feel like the header style isn't necessary since it isn't working as expected. What do you think about removing this style and those three places that it is used? All three would then default to the new secondary color that you updated.

[mateoclarke]

Yeah, that makes sense. I was considering updating the text.header value to #666666 along with text.secondary but I was interpreting your issue pretty literally. I figured it would come up in this review either way, and I like your suggestion of removing the header override entirely. I can definitely add that to this PR today. 👍

[mddilley]

sounds great - thanks!

[mddilley]

Looks amazing! Thanks for making this improvement. 🙏 🚢

[roseeichelmann]

looks good, yay for better accessibility :)

[chiaberry]

Easier on the eyes!