Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
feat: Identifier component MVP (#1044)
* Add files for New Component: Indentifier, wrote and passing one test for IdentifierGov * Adding Storybook example for default Identifier rendering IdentifierGov * Adding Storybook example for default Identifier rendering IdentifierGov * Moving comments to the bottom of each file * Adding IdentifierMasthead and IdentifierLinks, importing to Identifier; rendering text confirming import in Storybook * Adding test for ability to pass in section attributes via props * d * Add template for each Storybook example; hard coded IdentifierGov component to display a description and a link to USA.gov * Adding props to IdentifierLinks to pass links as props * Adding test for passing attributes as props * Adding placeholder links array to Identifier component, adding test for IdentifierLinks accepting custom styles * Adding test checking for passed in language prop * Adding content map for IdentifierGov component copy * Rendering imported copy from content map in IdentifierGov component * Adding storybook examples matching USWDS * Forgot to push my changes yesterday * Passing test for correct copy based on passed in language * Noting established pattern in existing component * Updating the content mapobject naming to make it easier to conditionally render the correct language * Moving interface to content file, using Record to create safe way to use a string to access properties of the content map object * Adding another test checking for IdentifierGov default behavior (renders English) when passed no props * Adding test checking for passing in attributes via props, adding the code to IdentifierMasthead to pass the test * Adding bare bones structure based on USWDS code to IdentifierMasthead * Some cleanup in the content file, adding classnames method to the Identifier parent component * Adding children components to storybook, rendering in spanish in storybook when passed as props to IdentifierGov * Commiting yarn build updates to yarn.lock * Removing duplicate IdentifierGov test * Rendering complete IdentifierGov component in storybook, commiting snapshots taken for test purposes * Adding test checking for ability to pass in a className, commiting updated snapshots * Moving IdentifierGov copy map back to parent file, updating snapshots * Add test checking for ability to pass in custom className to IdentifierMasthead component * Import subcomponents to Identifier.test in order to pass render check, update IdentifierMasthead test for aria-label to actually test for custom label, update IdentifierGov test to be more elegantly worded * Add test to IdentifierMasthead to check for ability to pass in Spanishlanguage prop * Update Identifier test wording to be more specific * Replace hardcoded text with reference to copy object, include update to Identifier.test test props that I forgot to save in the previous commit * Add IdentifierMasthead props to Identifier.stories and Identifier.test, fix spacing between identityDisclaimer and the link to the parentAgency * Add optional boolean for logo and second logo to IdentifierMasthead, add tests for no logo, passed in logo, and multiple logos, also add new props to Identifier.stories rendering appropriate amount of logos and in appropriate language * Clean up comments, add note re: necessity of supporting more than two logos * Add appropriate props to multiple-logos + Spanish Storybook example * Add IdentifierLinkItem subfolder and files, importing to IdentifierLinks and rendering a div * Nest IdentifierLinks files appropriately; add IdentifierLink, IdentifierLinkItem subcomponents * Add IdentifierLink and tests following conventions established in BreadcrumbLink component * Add tests for IdentifierLinkItem, remove wayward aria-label and associated test * Remove superfluous data-testids * Add Logo folder and files with mostly pseudocode * Update Identifier logo subcomponent name to IdentifierLogo to differentiate between it and the Logo in Footer * Update Identifier.stories to pass links into IdentifierLinks properly, import IdentifierLogo into IdentifierMasthead * Remove comments, refactored code, update IdentifierMasthead in .stories * Add tests for IdentifierLogo checking for ability to pass in props, add map accounting for multiple parent agencies * Add new IdentifierMasthead test data to pass check in Identifier.test, add USWDS’ taxpayer disclaimer sample text to Identifier.stories, add conditional taxpayer disclaimer prop and conditional rendering for parent agencies to IdentifierMasthead * Add usa-link class to IdentifierLink and amend tests to check for usa-class * Update IdentifierLinks test to use subcomponents, remove DRY testvars in IdentifierLink in favor of DAMP * Add query to IdentifierLinks test, update IdentifierMasthead tests to reflect MVP check for rendering one agency and one logo * Conditionally render taxpayer disclaimer prop in IdentifierMasthead, update Identifier.stories to render subcomponents with USWDS example text * Add check for IdentifierLinks - render its subcomponents and render a custom link * Add test to Identifier to check for ability to render without errors in English and Spanish, and render without a logo * Add testParentAgencyNoLogos for no logo storybook examples * Move taxpayer disclaimer out of the parent agency link * Respond to PR feedback: move div in Masthead, remove commented out import in index.ts, remove disable in .stories, update .stories export to match new convention * Export all new Identifier components within index.ts * Remove IdentifierMasthead content map, transfer those props to the ParentAgency interface * Update Identifier.test and IdentifierMasthead.test with new test data * Fix Identifier test query, remove deprecated test data * Place logos div inside of conditional logo rendering * Refactor IdentifierLogo out of IdentifierMasthead file, passing it in and rendering as children in .stories file * Refactor agency interface out of IdentifierMasthead, refactor test agency objects out of Identifier.stories, passing directly into components in stories * Move container div from IdentifierMasthead into Identifier.stories * Remove ariaLabel prop from IdentifierMasthead * Remove superfluous test data from IdentifierMasthead tests * Remove test data from IdentifierMasthead tests again * Refactor content out of IdentifierGov, add example content to .stories * Refactor img out of IdentifierLogo, may be able to get rid of this component altogether, TBD * Render storybook examples for multiple parents and logos, english and spanish; update naming convention to remove the word identifier from storybook examples except for Default and Spanish * Refactor hardcoded aria-label out of IdentifierLinks, passing it in via navProps in each storybook example with English and Spanish * Add storybook example for more than two parent agencies, add storybook example for multiple parent agencies plus tax disclaimer copy * Update Identifier tests to reflect refactored subcomponents and structure, add tests checking for cases with no logos, cases with more than two logos * Remove superfluous import * Update IdentifierMasthead tests with new structure and test data, check for rendering without errors, rendering in Spanish, rendering without a logo, rendering more than two logos * Update Identifier.stories test data props * Update test variable declarations, the test images didn't need to be arrays * Update IdentifierLogo test file * Remove comment from IdentifierLinks test file, pass a string instead of the test variable so expected doesn't point to the same place as actual * Update IdentifierGov test file * Fixes from PR feedback; remove custom className from storybook example, update IdentifierLogo prop agencyUrl to simply href and update everywhere, all hrefs now point to # and eslint check is disabled to reflect that in Identifier.stories * Add fixes from PR feedback from Suz; structure IdentifierLogo like the rest of the components including JSX.IntrinsicElements in function signature instead of passed in props, update Identifier to take in and spread a divProps property on the parent div to handle anything the user might want to pass in to be applied to the entire component * Remove unnecessary dependencies array from package.json Co-authored-by: Brandon Lenz <brandonalenz@gmail.com>
- Loading branch information
1 parent
847ec06
commit fa85e5f