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

RENO-3681: NewsletterSignup Component #1422

Merged
merged 43 commits into from
Oct 12, 2023
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
43 commits
Select commit Hold shift + click to select a range
cde951b
NewsletterSignup as Refactored FeedbackBox
aarnold101 Sep 8, 2023
6e6b373
NewsletterSignup as exposed Form
aarnold101 Sep 8, 2023
d7427a7
React EmailSubscription as DS Component
aarnold101 Sep 8, 2023
8710deb
NewsLetterSignup add getSectionColors function, and other advances
aarnold101 Sep 14, 2023
0b2b4a9
Merge branch 'development' into RENO-3681/newslettersignup-component
aarnold101 Sep 14, 2023
735e4c7
NewsletterSignup Functional Layout
aarnold101 Sep 20, 2023
00f217b
NewsletterSignup w/o onSubmit funtionality
aarnold101 Sep 22, 2023
706bf00
NewsletterSignup Near Complete Functionality
aarnold101 Sep 22, 2023
e7952c1
NewsletterSignup Updates & Fixes
aarnold101 Sep 25, 2023
5983419
NewsletterSignup Tests Round 1
aarnold101 Sep 26, 2023
3bd8d9d
NewsletterSignup w/o 3-second Timer
aarnold101 Sep 26, 2023
ac17c39
code cleanup
nypl-wluisi Sep 26, 2023
747b451
more code cleanup
nypl-wluisi Sep 27, 2023
4b33145
NewsletterSignup Update w/ Testing
aarnold101 Sep 28, 2023
8d53b28
Merge branch 'development' into RENO-3681/newslettersignup-component
aarnold101 Sep 28, 2023
f9dc57d
NewsletterSignup Snapshot Update
aarnold101 Sep 28, 2023
bcbb3b4
NewsletterSignup Minor Updates
aarnold101 Sep 28, 2023
117de84
NewsletterSignup Snapshot Update
aarnold101 Sep 28, 2023
8c603a9
code/storybook clean up
isastettler Oct 4, 2023
39670cc
fix darkmode, update storybook
isastettler Oct 4, 2023
e10e3de
add onChange and onSubmit test
isastettler Oct 5, 2023
26be513
fix onSubmit test, fix dark mode border bug
isastettler Oct 5, 2023
f7532ca
update confirmation props, update storybook, tests and Snapshots acco…
isastettler Oct 5, 2023
fa0c034
update storybook, add privacyPolicyLink prop
isastettler Oct 6, 2023
11aafed
fix title storybook bug, updated styles
isastettler Oct 6, 2023
87cc43d
fix descriptionText bug
isastettler Oct 10, 2023
7d141cc
fix "blogs" border color, add test for html props passed
isastettler Oct 10, 2023
f6bdbe9
add NewsletterSignup to component export file
isastettler Oct 10, 2023
abbe6eb
Add working onSubmit function and update .mdx
aarnold101 Oct 10, 2023
a4460e1
Merge branch 'development' into RENO-3681/newslettersignup-component
aarnold101 Oct 10, 2023
2b15b97
add props for error message, update styling and tests, add Newsletter…
isastettler Oct 11, 2023
9b31ea7
update storybook stories, fix typo
isastettler Oct 11, 2023
650e487
add interavtive example to storybook
isastettler Oct 11, 2023
cfcab79
add actions for onChange and onSubmit
isastettler Oct 11, 2023
8e5430f
remove styling, fix typos
isastettler Oct 12, 2023
d34bcc7
update as per PR comments
isastettler Oct 12, 2023
d926063
update snapshots
isastettler Oct 12, 2023
d99e940
Merge branch 'development' into RENO-3681/newslettersignup-component
isastettler Oct 12, 2023
1e634b1
update props, comments as discussed, update test and doc accordingly
isastettler Oct 12, 2023
3a56d1f
update breakpoints, remove reduntant content
isastettler Oct 12, 2023
917feb0
update breakpoint - the one that got away
isastettler Oct 12, 2023
3f42b6a
change html tags to ds components
isastettler Oct 12, 2023
77c31d7
update Text styles, fix typo for privacyPolicyLink, update tests
isastettler Oct 12, 2023
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
24 changes: 24 additions & 0 deletions src/components/NewsletterSignup/NewsletterSignup.stories.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -58,6 +58,15 @@
},
onChange: { control: false, action: "onChange" },
onSubmit: { control: false, action: "onSubmit" },
privacyPolicyLink: {
control: "text",
table: {
defaultValue: {
summery:
"https://www.nypl.org/help/about-nypl/legal-notices/privacy-policy",
},
},
},
title: {
control: "text",
table: {
Expand Down Expand Up @@ -106,6 +115,8 @@
event.preventDefault();
action("onSubmit")(event.target[0].value);
},
privacyPolicyLink:
"https://www.nypl.org/help/about-nypl/legal-notices/privacy-policy",
title: undefined,
isastettler marked this conversation as resolved.
Show resolved Hide resolved
valueEmail: undefined,
view: undefined,
Expand All @@ -119,6 +130,7 @@
},
render: (args) => <NewsletterSignup {...args} />,
};
/* Counter to allow the interactive example to show different states uponn submit*/
let counter = 0;

function NewsletterSignupOnSubmitExampleComponent() {
Expand All @@ -139,7 +151,7 @@

React.useEffect(() => {
if (counter === 3) counter = 0;
}, [counter]);

Check warning on line 154 in src/components/NewsletterSignup/NewsletterSignup.stories.tsx

View workflow job for this annotation

GitHub Actions / Lint

React Hook React.useEffect has an unnecessary dependency: 'counter'. Either exclude it or remove the dependency array. Outer scope values like 'counter' aren't valid dependencies because mutating them doesn't re-render the component

const handleChange = (event) => {
console.log(`onChange Email Input value: ${event.target.value}`);
Expand Down Expand Up @@ -272,3 +284,15 @@
</VStack>
),
};

/* To fix focus issue where the page focuses on the last NewsletterSignup
isastettler marked this conversation as resolved.
Show resolved Hide resolved
component example */
const setFocus = () => {
const heading = document.getElementById(
"anchor--components-form-elements-newslettersignup--with-controls"
);
heading.focus();
heading.scrollIntoView({ behavior: "smooth" });
};

setTimeout(setFocus, 1000);
10 changes: 5 additions & 5 deletions src/components/NewsletterSignup/NewsletterSignup.tsx
aarnold101 marked this conversation as resolved.
Show resolved Hide resolved
Original file line number Diff line number Diff line change
Expand Up @@ -81,7 +81,7 @@ export const NewsletterSignup = chakra(
formHelperText,
id,
isInvalidEmail = false,
newsletterSignupType = "whatsOn",
newsletterSignupType = "blogs",
onChange,
onSubmit,
privacyPolicyLink = "https://www.nypl.org/help/about-nypl/legal-notices/privacy-policy",
Expand Down Expand Up @@ -114,10 +114,10 @@ export const NewsletterSignup = chakra(
{...rest}
>
<VStack __css={styles.pitch} alignItems="flex-start">
{title && <Heading level="h3" text={title} margin="unset" />}
{title && <Heading level="h3" text={title} /*margin="unset"*/ />}
{descriptionText ? (
typeof descriptionText === "string" ? (
<Text margin="unset">{descriptionText}</Text>
<Text /*margin="unset"*/>{descriptionText}</Text>
) : (
descriptionText
)
Expand All @@ -126,7 +126,8 @@ export const NewsletterSignup = chakra(
<Link
href={privacyPolicyLink}
type="external"
margin="unset"
// margin="unset"
isUnderlined={false}
__css={styles.privacy}
>
Privacy Policy
Expand Down Expand Up @@ -208,7 +209,6 @@ export const NewsletterSignup = chakra(
name="errorFilled"
size="large"
/>
{/* This text is boilerplate and not meant to be customized. */}
<Text
color="ui.error.primary"
fontSize="xl"
Expand Down
Loading