|  | Commit message (Collapse) | Author | Age | Files | Lines | 
|---|
| | 
| 
| 
| 
| | They do not help to make the layout more readable (on the contrary I
think...) so the props drilling is useless. | 
| | |  | 
| | |  | 
| | 
| 
| 
| 
| 
| 
| 
| 
| | * remove `icon` prop (it is confusing because a copyright should have
the copyright symbol, the license is not part of the copyright)
* reorganize copyright informations
I also updated the CC BY SA icon because the elements was in the wrong
order. | 
| | 
| 
| 
| 
| 
| 
| | * extract NavItem from NavList
* remove `kind` and `listClassName` props (since the consumer has
control over NavList, NavItem and NavLink components these props are
obsolete) | 
| | 
| 
| 
| 
| | * merge Columns, Gallery and CardsList into Grid component
* add more options to control the grid | 
| | 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| | * make the component more generic
* merge `<Summary />` and `<Comment />` styles into card component
to avoid repeating the same structure
* remove most of the props to use composition
However the CSS is a bit complex because of the two variants...
Also, the component should be refactored when the CSS pseudo-class
`:has` has enough support: the provider and the `cover` and `meta`
props should be removed. | 
| | 
| 
| 
| 
| 
| | Instead of using helpers functions to format the date each time we need
to use a time element, it makes more sense to create a new component
dedicated to this task. | 
| | 
| 
| 
| 
| 
| | It removes items complexity by allowing consumers to use any label/value
association. Translations should also be defined by the consumer.
Each item can now be configured separately (borders, layout...). | 
| | 
| 
| 
| 
| 
| 
| 
| | The styles applied to ResponsiveImage are related to the figure and
figcaption elements. Those elements could be use with other contents
than images. So I extracted them in a Figure component. The
ResponsiveImage component is no longer useful: the consumer should use
the Image component from `next` and wrap it in a link if needed. | 
| | 
| 
| 
| 
| 
| | The component should only be responsible of the layout for the logo,
the name and the optional baseline. Also, the homepage url could
be different from `/` so the consumer should give the right url. | 
| | 
| 
| 
| 
| 
| | The flipping animation is used at several places so it makes sense to
use a single component to handle the animation. It will avoid styles
duplication. | 
| | 
| 
| 
| 
| | * replace `link` prop with `anchor` prop
* add a `label` prop to let consumer handle the accessible name | 
| | 
| 
| 
| 
| 
| 
| 
| 
| | * rename `external` prop to `isExternal`
* rename `download` prop to `isDownload`
* rewrite CSS to reduce code length and complexity
* move link styles in Sass placeholders to avoid repeats because of WordPress
articles
* move NavLink component to molecules | 
| | 
| 
| 
| 
| 
| 
| | * Column is a simple div meant to be used inside Columns component so
they should live together.
* NoScript is only used in Layout component and its name does not
make sense since it needs to be used inside a `noscript` element. | 
| | 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| | * add a `spacing` prop
* replace `layout` prop with `isInline` prop
* remove `items` prop (and classNames props) in favor of new components:
Description, Group, Term
* remove `withSeparator` prop (CSS content is announced by screen readers
and Firefox/Safari have no support for alternative text so the consumer
should add itself an element with `aria-hidden` if it need a separator)
Be aware, Meta component and its consumers can be visually broken, they
should be refactored before using them in production. | 
| | 
| 
| 
| 
| 
| 
| 
| 
| | * change `items` prop to children
* replace `kind` prop with `isHierarchical`, `isOrdered` & `isInline` props
* add `hideMarker` prop
* add `spacing` prop to control item spacing
* move lists styles to Sass placeholders to avoid repeats because of
headless WordPress | 
| | 
| 
| 
| 
| 
| 
| | * Avoid browser vendors by adding an extra div
* Add a loading state
* Add an option to center the progress bar (no longer the default)
* Remove min since it is always 0 | 
| | 
| 
| 
| 
| 
| 
| | * Message should be set as children
* Default message is no longer available (depending on use case, the
consumer might prefer aria-label instead)
* It is now possible to define the message position | 
| | 
| 
| 
| 
| | Sizes are also predefined and can be set using the `size` prop,
so the consumers should no longer adjust the size in CSS. | 
| | 
| 
| 
| 
| | Some components have been renamed to be able to create Footer, Header
and Nav. | 
| | 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| 
| | Both:
* move styles to Sass placeholders
Button:
* add `isPressed` prop to Button
* add `isLoading` prop to Button (to differentiate state from
disabled)
ButtonLink:
* replace `external` prop with `isExternal` prop
* replace `href` prop with `to` prop | 
| | |  | 
| | |  | 
| | 
| 
| 
| 
| 
| | Next expect a default export for pages so only those components should
use default exports. Everything else should use named exports to
reduce the number of import statements. | 
| | 
| 
| 
| 
| 
| | Using paths aliases starting with "@" can be confusing and can lead to
conflict with existings modules. I prefer to use relative paths to
avoid extra configuration in tools because of these aliases. | 
| | |  | 
| | 
| 
| 
| 
| 
| 
| 
| 
| | Previously, only the first 10 comments was loaded. So I update the
fetching method to retrieve all the comments on a post.
Also, I choose to order comments on client side because of a bug
with WPGraphQL.
Finally, I renamed the Comment type to SingleComment to avoid conflict
with existing types. | 
| | 
| 
| 
| 
| 
| | The progress element is a form element, so a label is required to be
accessible. Since I'm not using it without label, I transform the
optional info parameter to a mandatory label parameter. | 
| | |  | 
| | |  | 
| | 
| 
| 
| | I also configure Jest to avoid conflicts between Cypress and Jest. | 
| | 
| 
| 
| 
| | Checkbox and radio buttons are working the same way so I decided to
group them in a same component. | 
| | |  | 
| | |  | 
| | |  | 
| | |  | 
| | |  | 
| | 
| 
| 
| 
| 
| | There is a bug with next/image and Storybook. I was manually adding
`unoptimized` to images. Instead we can use a workaround by adding an
extra config in Storybook `preview.js` file. | 
| | |  | 
| | 
| 
| 
| 
| 
| 
| | * Improve wp-blocks-columns, wp-block-quote and some images
* Make Sharing widget consistent between pages
* Remove margin option from lists
* Fix Topic logo alignement | 
| | |  | 
| | |  | 
| | |  | 
| | |  | 
| | 
| 
| 
| 
| | * change animation on article card hover
* change comments section alignment | 
| | |  | 
| | |  | 
| | |  | 
| | |  |