meetup-web-components


Meetup library of React UI components for the web.

Table of Contents
=================

* Releases
* Pro Web Pinned Version
* Development/Beta release
* Getting started
* Components
* Redux Form Components
* Testing
* Simulating interactions
* Verifying child elements
* Linting
* Storybook
* Release notes

Component checklist

This checklist can be used as a guide for adding components that are easy to reuse and maintain. The categories come from a talk I saw Elyse Holladay give at Clarity Conf 2017.

Easy to reason about
* [ ] It will be clear to other designers or engineers what problem the component solves and when to use it
* [ ] All properties, options and variants are documented in Storybook
* [ ] Has unit tests
* [ ] Provides logical defaults
* [ ] If there can be errors, error states are designed and documented
* [ ] If the component has become complex, it has been broken into smaller components

Context-agnostic
* [ ] Doesn't rely on a parent component to behave or look correctly
* [ ] Doesn't have logic that controls functionality outside of itself e.g.: if there is a button in the component, it should take an event handler via props instead of assuming what the button is going to do
* [ ] Works in supported browsers and devices
* [ ] Design is accessible for all users
* [ ] Provides necessary ARIA information
* [ ] Design won't cause issues with translation or internationalization

Independent and isolated
* [ ] Only affects itself
* [ ] Defines itself and its styles only in one place
* [ ] Ready to use "out of the box" in consumer apps or by other components

Releases

This package uses semver versioning to tag releases, although the patch version
is determined exclusively by the Travis build number for pushes to main.
Major and minor versions are hard-coded into the Makefile.

Manual pushes to main and PR merges to main will be built by Travis, and
will kick off the yarn publish routine. The currently-published version of the
package is shown on the repo homepage on GitHub in a badge at the top of the
README.

$3

CURRENT PRO VERSION: 6.2.0

pro-web is not on the latest version of MWC that uses swarm-ui.
Instead, the pro-web version of MWC is based off a branch called
pro-web-6.x.x. It's manually published to npm as version 6.2.0
and is tagged as pro in the registry, so all that is needed to
install is yarn add meetup-web-components@pro -W.

If changes are needed for the pro-web version, do the following:

- Checkout, commit+push changes to the pro-web-6.x.x branch.
- Bump the version number in the package.json file and commit+push.
- Edit this README to update the current version.
- Finally, run npm publish --tag pro as the meetupbot npm user.

$3

When developing a consumer application that requires changes to the platform
code, you can release a beta version of the platform on npm by opening a PR in
the meetup-web-platform repo. When it builds successfully, a new beta version
will be added to the list of available npm versions. The generated version number
is in the Travis build logs, which you can navigate to by clicking on 'Show all
checks' in the box that says 'All checks have passed', and then getting the
'Details' of the Travis build.

At the bottom of the build log, there is a line that echos the GIT_TAG.
If you click the disclosure arrow, the version number will be displayed, e.g.
0.5.177-beta.

You can then install this beta version into your consumer application with

``sh
> yarn add meetup-web-components@
`

Each time you push a change to your meetup-web-components PR, you'll need to
re-install it with the new tag in your consumer application code.

The overall workflow is:

1. Open a PR for your meetup-web-components branch
2. Wait for Travis to successfully build your branch (this can take 5+ minutes)
3. Get the version string from the build logs under GIT_TAG
4. (if needed) Push changes to your meetup-web-components branch
5. Repeat steps 2-3

Getting started

You can generate the boilerplate files for React components using
yarn run generate, which invokes src/utils/generate.js.

The command will prompt you for a 'type' (select from the list of options),
and a 'name'. It generates the following files in src/ :

- .jsx Component JSX module
- .test.jsx Component test script
- .story.jsx Storybook script

$3

Located in the src/ directory, component files live alongside
their corresponding .test and .story files.

Filename casing conventions:
- Component files: CamelCase, with a leading capital, i.e. RsvpTag.jsx
- Test files: camelCase, i.e. rsvpTag.test.js
- Story files: camelCase, i.e. rsvpTag.story.jsx

$3

We use redux-form in our mup-web app to help with validation flow.

redux-form can use our form components (just pass our MWC component to Field as the component prop), but we need to write wrappers to pass down the props from redux-form Field to our form components in meetup-web-components.

The job of the wrapper for each component is mostly just parsing out the meta, input and other props from redux-form and passing them on.

#### Conventions
You can find the wrapper classes in src/forms/redux-form.

The files are named after the classes they wrap to avoid verbose file names.
Ex. forms/TogglePill.jsx has a forms/redux-form/TogglePill.jsx.

But the actual class name to import and displayName, have ReduxForm in the name.
Ex. export class ReduxFormTogglePill

We write wrappers as we need them, so if you don't find one that you need, please write it!

#### Gotchas

We've run into a couple gotchas already:
redux-form validates all fields on load, and its hard to tell when the form is first rendered. To avoid displaying errors right away, we added some logic to read meta.touched. https://github.com/meetup/meetup-web-components/pull/307

redux-form's implementation of checkboxes give them values of true and false.
ReduxFormTogglePill wrappers handle this now by passing the input.value prop down as isActive (which sets checked on checkboxes)
https://github.com/meetup/meetup-web-components/pull/310.
We may need to do this for other checkbox, radio component wrappers.




$3

The src/ directory contains layout helpers, like Section and Chunk. These are
documented in Storybook, but a more detailed guide can be found here.




Testing

Unit testing UI components is a little weird compared with unit testing business logic.

1. You have to decide what aspects of a UI element are intrinsic to its appearance
rather than just implementation details
2. UI elements evolve based on aesthetic tastes as much as functional requirements -
inflexible tests require a lot of maintenance
3. Headless testing of browser-dependent objects requires some extra tooling to simulate
the target environment

$3

TestUtils.Simulate appears to work correctly for our testing setup - it should be used
for all tests that involve simulating events, like onClick. Check out button.test.js
for an example.

$3

In UI testing, there is an almost invisible line between testing the implementation (markup)
and testing the behavior (appearance/content), and ideally you only should test the behavior -
there are loads of ways to change markup without changing the fundamental app experience, and
those kinds of markup changes should not be considered "bugs" that result in failed tests.

The implication for constructing unit tests is that you should avoid relying on the specific
markup (tags and DOM structure). Sometimes it's unavoidable, but if you are inclined to use
getElementsByTagName, firstChild/lastChild, or a querySelector(All) that includes a tag
name to access particular parts of the component UI, check whether there is a better way to skip
over the markup implementation details and grab what you want explicitly.

A useful option is to add a PCV className to the element of interest, and just use
yourComponentEl.querySelector('.specificClassName') to find it. Classnames are free and
DOM-independent, which means that no matter what the markup is for your event name, you can
always unit-test the behavior (text content) accurately with

`js
// good
expect(eventNode.querySelector('.event-name').textContent).toEqual(testEventName);

// bad - assumes both tag (h5) and structure (first h5 in the card)
expect(eventNode.getElementsByTagName('h5')[0].textContent).toEqual(testEventName);
`

Linting

To manually lint your code, run:

$ yarn run lint

Whitespace issues will be fixed automatically - just remember to commit the changes. Other style
issues will log errors. Our .eslintrc configuration is based on the 'recommended' preset, with
a number of additional rules that have been requested by the dev team. It's a 'living' standard,
however, so please feel free to send PRs with updates!

Storybook

Before building any components, it's helpful to know what related components
have already been built into our Foundation library. We use
React Storybook
to display components outside of the app context. To open it, run:

`sh
$ yarn install
$ yarn run storybook
``

And open the viewer at http://localhost:9001

All of the available components are listed on the left, and clicking on
one will open it in the preview pane. Variants are also listed in the left
column to show how different states affect the rendered component.