@royaloperahouse/chord 2.1.3 → 2.1.4-b-chord-development-chord-development

package/CHANGELOG.md

@@ -1,3 +1,6 @@
+## [2.1.4]
+- Fix screenreader accessibility for accordions
+
 ## [2.1.3]
 - Navigation: make links links
 - Clear testing errors
@@ -46,6 +49,10 @@
 ## [1.24.0]
 - Implement Upsell Section & Upsell Cards
 
+## [1.23.2-b]
+- Add Stepper component
+- Fix solo negative sign NaN bug
+- 
 ## [1.23.2]
 - Fix BrandingStyle not showing bug
 
@@ -60,12 +67,15 @@
 - TickBox: Added disabled and error states
 
 ## [1.21.1]
-- Made select state reset behaviour opt-in
+- Compact PageHeading: Refactored to support iOS devices
 
 ## [1.21.0]
 - TextLogo: add new component
 - Compact PageHeading: swap logo with TextLogo to fix iOS issues
 
+## [1.20.2-a]
+- Made select state reset behaviour opt-in
+
 ## [1.20.2]
 - Card: change cursor to default for cards without buttons
 - HighlightsCarousel: hide button in case there are no text for them
@@ -101,7 +111,7 @@
 - Change to pagination component to align page numbers with URL numbers.
 
 ## [1.15.0]
-- Implemented a new Select component
+- Add Compact PageHeading component
 
 ## [1.14.0]
 - Added standfirst functionality to Accordion/ Accordions component and theme for Schools Platform
@@ -121,21 +131,55 @@
 ## [1.12.0]
 - Search: Extend ContentSummary and SearchBar components to support new search results
 
+## [1.11.0-a]
+- Pagination: add new component
+
 ## [1.11.0]
 - Carousels: Rewrite logic that cuts description, to support older browsers
 
+## [1.10.0-a]
+- ContentSummary: make image clickable
+
 ## [1.10.0]
 - PageHeadingImpact: Replace CSS background-image with HTML image element
 
 ## [1.9.0]
 - Card: Add a new prop for the hover line color
+## [1.9.0-c]
+- ContentSummary: make image clickable
+
+## [1.9.0-b]
+- ContentSummary: change image size
+- SearchBar: Export component
+
+## [1.9.0-a]
+- ContentSummary: change image props
+
+## [1.9.0]
+- Card: Add a new prop for the hover line color
+
+## [1.8.1-h]
+- ContentSummary: add exports
+
+## [1.8.1-g]
+- ContentSummary: refactor
+
+## [1.8.1-f]
+- ContentSummary: update conditional rendering
+
+## [1.8.1-c]
+- ContentSummary: update for Search results
+- ContentSummarySection: remove
+
+## [1.8.1-b]
+- Card: add new prop for the line color on hover
 
 ## [1.8.0]
 - SearchBar: Disable
 
 ## [1.7.2]
 - Add imageAltText to Card component
-
+ 
 ## [1.7.0]
 - Add new ContentSummary component
 - Add new ContentSummarySection component

package/README.md

@@ -1,67 +1,277 @@
-# Royal Opera House - chord 
+# Royal Opera House Chord - Front End Design System
 
-Welcome to the Royal Opera House library components  `@royaloperahouse/chord`.
+Welcome to the Royal Opera House Front End Design System, `@royaloperahouse/chord`
 
 This package is a library of UI components intended to be used in the ROH website.
 
+It uses React, TypeScript, TSDX and Storybook.
 
-## Install
+The NPM packages are published at [https://www.npmjs.com/package/@royaloperahouse/chord](https://www.npmjs.com/package/@royaloperahouse/chord)
+
+The Storybook for the latest version of the library is hosted at [chord.roh.org.uk](chord.roh.org.uk)
+
+
+## Prerequisites
+---
+
+* Make sure the required package dependencies are installed using `yarn install`
+
+* If this is the first time you are working on Chord, make sure to run `yarn build` to build all the modules.
+
+* **To deploy the storybook publicly** you will need credentials for the **parent** 'Royal Opera House' AWS account in your `~/.aws/credentials` file.
+  * The deploy script expects these to be called `[parent]`.
+
+  * You can find these values at:
+  
+    *AWS 'Your Applications' page -> 'Royal Opera House' -> 'Developer Access' -> 'Command line or programmatic access'*
+
+* **To deploy a preview** for demo / QA you will need the `CHROMATIC_PROJECT_TOKEN` -- find this in 1Password
+
+
+* **To publish the NPM package** you will need the ROH `NPM_TOKEN` -- find this in 1Password
+
+
+## Versioning
+---
+
+We use [Semantic Versioning](https://semver.org/) for Chord.
+
+Given a version number MAJOR.MINOR.PATCH, increment the:
+
+* MAJOR version when you make incompatible library changes
+* MINOR version when you add functionality in a backwards compatible manner
+* PATCH version when you make backwards compatible bug fixes
+
+When you create a new release always update the `CHANGELOG` and `package.json`
+
+Increment your version from the latest stable version on `chord-releases`
+
+
+## Deploying the Storybook
+---
+
+### **1. To build and deploy locally**
+
+You can quickly build and serve the contents of `/storybook-static` from your local machine, for development and manual testing. To do this run:
+
+```bash
+yarn storybook
+```
+
+By default the storybook will be accessible at
+[http://localhost:6006/]()
+
+The server will hot-reload on most changes.
+
+### **2. To deploy a preview version remotely**
+
+You can deploy a release candidate, or work-in-progress to [Chromatic](https://www.chromatic.com/) for showcase or QA.
+
+**NOTE:** you will need the `CHROMATIC_PROJECT_TOKEN` as described in *Prerequisites* above.
+
+Either set the token as a shell variable or use a `.env` file in the chord root directory containing the token, as so:
+
+```
+CHROMATIC_PROJECT_TOKEN=tokengoeshere
+```
+
+Run:
+
+```
+yarn deploy-storybook-dev
+```
+
+to build and deploy the storybook from your repo to Chromatic.
+
+You will find the address of your deployed storybook in the console output, looking something like this:
+
+`→ View your Storybook at https://randomHexString-randomAlphabeticalString.chromatic.com`
+
+### **3. To deploy a release version**
+ 
+You can deploy the contents of `./storybook-static` to S3 as a static site, which is permanently accessible at [chord.roh.org.uk](chord.roh.org.uk)
+
+**NOTE:** To do this you will need the correct `AWS` credentials set up, as described in *Prerequisites* above.
+
+First build the storybook using:
+
+```
+yarn build-storybook
+```
+
+then run:
+
+```bash
+yarn deploy-storybook
+```
+
+> **IMPORTANT:** This URL is intended to showcase the latest stable version of Chord, it should be kept up to date with the main `chord-releases` branch and should **only** be used for release versions. If you want to deploy a development version follow the steps above under *To deploy a preview version remotely*.
+
+
+## Releasing a New Package Version
+---
+
+This is the procedure for releasing a new Chord NPM package.
+
+There are two types of package:
+
+A **snapshot** can be published to use a development version of the Chord library in our frontend staging environments, to perform integrated, manual testing.
+
+A **stable** package is published for use in production.
+
+**NOTE:** You will need the `NPM_TOKEN`, as described in 'Prerequisites' above.
+
+### **1. To release a snapshot version**
+
+> **IMPORTANT:** Make sure you always publish your **snapshots** from `chord-development` after merging in your feature branch.
+
+For snapshots, `RELEASE_VERSION` should be the same as the latest **stable** version of the Chord Library, as found in `chord-releases`, followed by an **unused** lowercase letter.
+
+For example, if the latest stable release was 1.42.0, and the last snapshot published to NPM was 1.42.0-w, you would use RELEASE_VERSION `1.42.0-x`
+
+To publish a snapshot use:
 
 ```bash
-    npm i --save @royaloperahouse/chord
+NPM_ROH_TOKEN={NPM_TOKEN} RELEASE_VERSION={RELEASE_VERSION} yarn publish-snapshot
 ```
+
+The version published to NPM will be named, e.g. `1.42.0-a-chord-development`, use this to install the snapshot package in the frontend repos.
+
+### **2. To release a stable version**
+
+> **IMPORTANT:** Make sure you always publish your **stable** packages from `chord-releases` after merging in your feature branch.
+
+For stable releases, increment your version from the latest **stable** version found on `chord-releases`, following the rules described in *Versioning* above, and use this as `RELEASE_VERSION`. (Make sure it matches the version in your `package.json` and `CHANGELOG.md`!)
+
+To publish a stable package use:
+
 ```bash
-    yarn add @royaloperahouse/chord
+NPM_ROH_TOKEN={NPM_TOKEN} RELEASE_VERSION={RELEASE_VERSION} yarn publish-release
 ```
 
-## Usage
 
-```javascript
-import {
-  Footer, GlobalStyles, ThemeProvider, ThemeType,
-} from '@royaloperahouse/chord';
+## Using the Package
+---
+
+The package is deployed to NPM, and can be installed using yarn or npm:
 
+```bash
+npm i --save @royaloperahouse/chord
 ```
-## Sample
+```bash
+yarn add @royaloperahouse/chord
+```
+
 
-```javascript
+## Testing
+---
 
-import {
-  Footer, GlobalStyles, ThemeProvider, ThemeType,
-} from '@royaloperahouse/chord';
+To **Lint** the package use:
 
+```bash
+yarn lint
+# Use the --fix option to perform automatic fixes
+```
 
-const App = ({ children }: InnerProps): React.ReactElement => (
-  <ErrorBoundary>
-    <ThemeProvider theme={ThemeType.Core}>
-      <GlobalStyles />
-      <Wrapper>
-        <Header />
-        <GlobalStyle />
-        <HealthBanner />
-        <ErrorBoundary>
-          <Content>
-            {children}
-          </Content>
-        </ErrorBoundary>
-        <LiveChat />
-      </Wrapper>
-      <Footer data={footerData} />
-    </ThemeProvider>
-  </ErrorBoundary>
-);
+To run the **unit tests** (using Jest) use:
 
+```bash
+yarn test
+
+# Use the -u option to update snapshots if needed
+# Run `yarn test:watch` to re-run tests on changes
 ```
 
-## Components
+To **run and and store the unit tests** for display in storybook use:
+
+```bash
+yarn test-storybook
+# Use the -u option to update snapshots if needed
+```
+
+
+## Development Process
+---
+
+### **1. Branching Model**
+* The Chord project lives in the `roh-components` *monorepo*
+* When working on Chord, treat the `roh-components/packages/chord` directory as your root
+* The Chord **development** branch is currently `chord-development`
+* The Chord **main** branch is currently `chord-releases`
+* All work should be done on a correctly named **feature branch** of the format: `issueType/ticketNumber-short-description-of-feature`, e.g. `feature/RD-1234-update-chord-documentation`
+* Always create your feature branch FROM, and submit pull requests TO, the **main** branch
+* For **QA / UAT** of a Chord component in isolation: following review you should deploy a preview version to Chromatic straight from your **feature branch** _without_ merging to development
+* For releasing an **NPM snapshot**: merge your feature branch into the **development** branch and publish the NPM snapshot from there
+
+### **2. Detailed Workflow**
+
+#### **Before review:**
+
+* Git pull the latest version of `chord-releases`
+* Checkout a new, correctly-named feature branch from `chord releases`
+* Do your work on this feature branch
+* Run: `yarn lint`
+* Run: `yarn test`
+* Commit your changes
+* Create a pull request from your feature branch to `chord-releases`
+* Add the appropriate reviewers
+
+#### **Before QA / UAT:**
+
+After your PR is approved, you have two options:
+
+**i.** For QA of a component in isolation:
 
-Accordion, Accordions, AltHeader, AnchorTabBar, AnnouncementBanner, BodyText, Card, Cards, Carousel, CinemaBadge, ContactCard, ControlledDropdown, devices, Editorial, Footer, GlobalStyles, Grid, GridItem, Header, HighlightsCarousel, HighlightsCarouselCinema, HighlightsCarouselCore, HighlightsCarouselStream, Icon, ImageWithCaption, Navigation, Overline, PageHeadingCinema, PageHeadingCore, PageHeadingImpact, PageHeadingStream, PeopleListing, PrimaryButton, PromoWithTags, PromoWithTitle, RotatorButtons, SecondaryButton, SecondaryLogo, SectionSplitter, SectionTitle, Sponsorship, StickyBar, StreamBadge, Subtitle, Tab, TabLink, Tabs, TertiaryButton, TextLink, TextOnly, Tickbox, TitleWithCTA, ThemeProvider, TypeTags, Quote
+* Follow the steps above in *Deploying the Storybook -> 2. Deploying a preview version remotely*
+* Share the generated Chromatic URL with the appropriate people for QA / UAT
+
+**ii.** For integrated testing of a component in the frontend staging environments:
+
+* Check out the `chord-development` branch
+* Merge your work in to `chord-development`
+* Follow the steps above under *Publishing a New Package Version -> 1. To release a snapshot version*
+* Git push the updated `chord-development` branch to remote
+* Finally, in the frontend repo:
+
+1. Update the `package.json` to reference the new snapshot version
+2. Run: `yarn install`
+3. Deploy the frontend to a staging environment
+
+#### **When ready for production:**
+
+* Return to your chord **feature branch**
+* Update the chord version in `package.json`, incrementing from the last **stable** release, found in `chord-releases`, and following the rules described in the *Versioning* section
+* Update `CHANGELOG.md`, adding the new version number and a short description of your changes
+* Run: `yarn lint`
+* Run: `yarn test`
+* Run: `yarn test-storybook -u`
+* Run: `yarn build-storybook`
+* Git commit, push, and update your PR for final approval
+* Merge your approved PR to `chord-releases`
+* Follow the steps in *Publishing a New Package Version -> 2. To release a stable version* to publish the NPM package
+* Finally, follow the steps in *Deploying The Storybook -> 3. To deploy a release version*, to update the public Chord Storybook page
+
+Once you have successfully published a new release version, you can open a ticket to update to this version in any of the frontend repos.
+
+
+## Notes
+---
+### The `storybook-static` folder
+
+This folder contains all the HTML / JS required to deploy the Storybook as a static site. It's autogenerated whenever the Storybook is built and therefore will show up a lot of merge conflicts when you merge other branches into your feature branch. It is best to leave generating the storybook until the later part of your workflow, but in case you run into conflicts earlier, it's ok to just delete the folder and regenerate using:
+
+```bash
+yarn build-storybook
+```
 
-## Types
+For the same reason changes to this folder will cause a lot of big diffs, when looking at a PR, but can mostly be skimmed over unless something looks unusual.
 
-AspectRatio, CarouselType, Colors, FooterData, INavigationProps, INavTopProps, TickboxMode, ThemeType
+---
+### Examples
 
+There is an example implementation in the example folder. Alternatively there are also integration examples in storybook. Make sure to keep these updated so as to showcase the current components available.
 
-## Documentation
+---
+### Bundle analysis
 
-See Storybook as a reference
+Calculate the real cost of your library using [size-limit](https://github.com/ai/size-limit) with `yarn size` and visualize it with `yarn analyze`.

package/dist/chord.cjs.development.js

@@ -1581,6 +1581,21 @@ var Twitter = (function (_ref) {
   }));
 });
 
+var X = (function (_ref) {
+  var _ref$color = _ref.color,
+      color = _ref$color === void 0 ? '#1A1A1A' : _ref$color;
+  return /*#__PURE__*/React.createElement("svg", {
+    width: "100%",
+    height: "100%",
+    viewBox: "0 0 24 24",
+    fill: "none",
+    xmlns: "http://www.w3.org/2000/svg"
+  }, /*#__PURE__*/React.createElement("path", {
+    d: "M13.7124 10.6179L20.4133 3H18.8254L13.007 9.61448L8.35992 3H3L10.0274 13.0023L3 20.9908H4.58799L10.7324 14.0056L15.6401 20.9908H21L13.7124 10.6179ZM11.5375 13.0904L10.8255 12.0944L5.16016 4.16911H7.59922L12.1712 10.5651L12.8832 11.5611L18.8262 19.8748H16.3871L11.5375 13.0904Z",
+    fill: color
+  }));
+});
+
 var Youtube = (function (_ref) {
   var _ref$color = _ref.color,
       color = _ref$color === void 0 ? '#1A1A1A' : _ref$color;
@@ -2135,6 +2150,7 @@ var IconLibrary = {
   UserSignedIn: UserSignedIn,
   Volume: Volume,
   Watch: Watch,
+  X: X,
   Youtube: Youtube
 };
 
@@ -5663,20 +5679,29 @@ var Accordion = function Accordion(_ref) {
     return visibleStandfirst ? /*#__PURE__*/React__default.createElement(ContentContainer, null, /*#__PURE__*/React__default.createElement(ChildrenContainer, null, visibleStandfirst)) : null;
   };
 
-  return /*#__PURE__*/React__default.createElement(AccordionContainer, null, /*#__PURE__*/React__default.createElement(LineContainer, null), /*#__PURE__*/React__default.createElement(TitleContainer$3, {
+  var contentContainerId = title + "-accordion-content";
+  return /*#__PURE__*/React__default.createElement(AccordionContainer, {
+    tabIndex: 0,
+    onKeyDown: keyDown
+  }, /*#__PURE__*/React__default.createElement(LineContainer, null), /*#__PURE__*/React__default.createElement(TitleContainer$3, {
     onClick: toggleAccordion,
-    onKeyDown: keyDown,
-    tabIndex: 0
+    tabIndex: -1,
+    role: "button",
+    "aria-label": title,
+    "aria-expanded": openAccordion,
+    "aria-controls": contentContainerId
   }, /*#__PURE__*/React__default.createElement("h4", null, /*#__PURE__*/React__default.createElement("a", {
     className: "accordion-title"
   }, title)), children && /*#__PURE__*/React__default.createElement(Icon, {
     iconName: iconName
-  })), /*#__PURE__*/React__default.createElement(Standfirst, null), /*#__PURE__*/React__default.createElement(ContentContainer, {
+  })), visibleStandfirst && /*#__PURE__*/React__default.createElement(Standfirst, null), /*#__PURE__*/React__default.createElement(ContentContainer, {
     "data-testid": "richcontainer",
     ref: content,
     style: {
       maxHeight: "" + textHeight
-    }
+    },
+    id: contentContainerId,
+    "aria-live": "polite"
   }, /*#__PURE__*/React__default.createElement(ChildrenContainer, {
     style: childrenVisibility
   }, children)), showLine && /*#__PURE__*/React__default.createElement(LineContainer, null));
@@ -6717,7 +6742,7 @@ var PrimaryButtonWithInversion = /*#__PURE__*/styled__default(PrimaryButton)(_te
   return invert ? theme.colors.white : theme.colors.black;
 });
 
-var renderBranding = function renderBranding(brandingStyle, invert, brandingText) {
+var renderBranding = function renderBranding(brandingStyle, invert, brandingText, brandingLink) {
   var color = invert ? exports.Colors.Black : exports.Colors.White;
 
   switch (brandingStyle) {
@@ -6731,6 +6756,13 @@ var renderBranding = function renderBranding(brandingStyle, invert, brandingText
         }
       });
 
+    case 'TextLink':
+      return (brandingText == null ? void 0 : brandingText.trim()) && /*#__PURE__*/React__default.createElement(TextLink, {
+        target: 'blank',
+        href: brandingLink,
+        color: color
+      }, truncate(stripAllHtmlTags(brandingText), 25));
+
     case 'StreamLogo':
       return /*#__PURE__*/React__default.createElement(Logo, {
         color: color,
@@ -6754,14 +6786,15 @@ var CompactHeaderBranding = function CompactHeaderBranding(_ref) {
   var brandingStyle = _ref.brandingStyle,
       _ref$invert = _ref.invert,
       invert = _ref$invert === void 0 ? false : _ref$invert,
-      brandingText = _ref.brandingText;
-  return /*#__PURE__*/React__default.createElement(CompactHeaderLogoWrapper, null, renderBranding(brandingStyle, invert, brandingText));
+      brandingText = _ref.brandingText,
+      brandingLink = _ref.brandingLink;
+  return /*#__PURE__*/React__default.createElement(CompactHeaderLogoWrapper, null, renderBranding(brandingStyle, invert, brandingText, brandingLink));
 };
 
 var isIOS = function isIOS() {
   try {
     console.warn('Do not use this on server side rendering, it may throw an error.');
-    if (typeof navigator === undefined) return false;
+    if (typeof navigator === 'undefined') return false;
     return ['iPad Simulator', 'iPhone Simulator', 'iPod Simulator', 'iPad', 'iPhone', 'iPod'].includes(navigator.platform) || // iPad on iOS 13 detection
     navigator.userAgent.includes('Mac') && 'ontouchend' in document;
   } catch (e) {
@@ -6777,7 +6810,7 @@ var useIOS = function useIOS() {
       setIOS = _useState[1];
 
   React.useEffect(function () {
-    if (typeof navigator === undefined) return;
+    if (typeof navigator === 'undefined') return;
     setIOS(isIOS());
   }, []);
   return IOS;
@@ -6917,7 +6950,8 @@ var PageHeadingCompact = function PageHeadingCompact(_ref4) {
       invert = _ref4$invert === void 0 ? false : _ref4$invert,
       _ref4$brandingStyle = _ref4.brandingStyle,
       brandingStyle = _ref4$brandingStyle === void 0 ? 'BlockText' : _ref4$brandingStyle,
-      brandingText = _ref4.brandingText;
+      brandingText = _ref4.brandingText,
+      brandingLink = _ref4.brandingLink;
 
   var _ref5 = link || {},
       linkText = _ref5.text,
@@ -6949,6 +6983,7 @@ var PageHeadingCompact = function PageHeadingCompact(_ref4) {
   }, /*#__PURE__*/React__default.createElement(CompactHeaderBranding, {
     brandingStyle: brandingStyle,
     brandingText: brandingText,
+    brandingLink: brandingLink,
     invert: invert
   }), /*#__PURE__*/React__default.createElement(CompactHeaderTitleWrapper, null, /*#__PURE__*/React__default.createElement(Header, {
     level: titleSize,