Skip to content

DS-746 | @ericbakenhus | Additional copy changes #422

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

Merged
merged 3 commits into from
May 30, 2024
Merged

Conversation

ericbakenhus
Copy link
Member

@ericbakenhus ericbakenhus commented May 30, 2024

READY FOR REVIEW

Summary

  • Copy changes
  • A11y changes

Review By (Date)

  • ASAP

Criticality

  • High

Review Tasks

Setup tasks and/or behavior to test

  1. Look at code
  2. Navigate to /citizenship/
  3. Verify it looks fine

Associated Issues and/or People

linkType="url"
link={{
url: 'https://giving.stanford.edu/gift-policy',
<main id="main-content">
Copy link
Member Author

Choose a reason for hiding this comment

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

Added main. Skip nav expects it to exist.

</CenteredContainer>
<div>
<iframe
title="Citizenship form for foreign source reporting"
Copy link
Member Author

Choose a reason for hiding this comment

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

Added title to iframe so we meet 4.1.2.

link={{
url: 'https://giving.stanford.edu/gift-policy',
<main id="main-content">
<CenteredContainer classes="su-max-w-800">
Copy link
Member Author

Choose a reason for hiding this comment

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

I tossed a max width on the text to get closer to meeting 1.4.8. It's technically AAA, but it seemed like a good idea.

Comment on lines +38 to +53
<p>
The federal government requires institutions of higher education,
such as Stanford, to report gifts and other payments from a donor
who is not a U.S. citizen when that donor makes gifts and other
payments to Stanford with an aggregate value of $250,000 or more in
a calendar year. Stanford must also report gifts and other payments
received from a donor who is associated with one of certain specific
countries (the list, which may change over time, currently includes
China (while excluding Hong Kong), Russia, Iran, and North Korea),
when that donor makes gifts and other payments to Stanford having an
aggregate value of $50,000 or more during a reporting period that
runs from July 1 to June 30 of the following year. Because your
generous gift(s) meet one or both of these reporting thresholds, we
ask that you please provide the information requested below. Thank
you.
</p>
Copy link
Member Author

Choose a reason for hiding this comment

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

Copy changes.

Copy link
Member

@sherakama sherakama left a comment

Choose a reason for hiding this comment

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

Ty

Comment on lines +57 to +64
<SbLink
linkType="url"
link={{
url: 'https://giving.stanford.edu/gift-policies/',
}}
>
Gift Policy website
</SbLink>
Copy link
Member Author

@ericbakenhus ericbakenhus May 30, 2024

Choose a reason for hiding this comment

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

Fix link and make text descriptive.

@ericbakenhus ericbakenhus merged commit 756c2dd into dev May 30, 2024
5 checks passed
@ericbakenhus ericbakenhus deleted the task/DS-746 branch May 30, 2024 19:57
@ericbakenhus ericbakenhus mentioned this pull request May 30, 2024
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