Add new warning to transfer page of onboarding #683
Conversation
|
[diff-counting] Significant lines: 24. |
|
Visit the preview URL for this PR (updated for commit cbca44c): https://cornelldti-courseplan-dev--pr683-transfer-credit-warn-ghclm19b.web.app (expires Tue, 24 May 2022 21:30:19 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 |
| @@ -52,6 +52,10 @@ | |||
| >here</a | |||
| >. | |||
| </div> | |||
| <div class="onboarding-transferCreditDescription"> | |||
| *Some transfer credits may incorrectly apply to requirements. We are working to fix these | |||
There was a problem hiding this comment.
I'm not a designer, but I'm not a big fan of the * before the warning (and before the warning above). The text is already small and gray, so I already recognize it as "extra information". Having two *s is a little intrusive and uneven IMO.
|
I'm also not a designer, but I feel like the message should at least be the first message and maybe bolded since it's a pretty important thing for students to know and definitely more important than the second message. |
Summary
This pull request adds a new warning to the transfer page of onboarding. The purpose of this warning is to notify users about AP/IB bugs before merging the next release (#680).
Test Plan
👀
Notes
As a follow-up to this PR, the gatekeep to hide AP/IB fulfillment will be removed.