feature: Add Divider component to Wave #221
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
JanHamara
requested review from
nlopin,
martimalek,
div-Leo,
rafael-sepeda and
jonotrujillo
as code owners
JanHamara
added
enhancement
div-Leo
reviewed
May 3, 2022
|
@JanHamara if it is not ready to review, you can make a PR draft |
div-Leo
changed the title
nlopin
reviewed
May 5, 2022
added 2 commits
May 5, 2022 14:29
add additional tests to check the default offset
Needed to change stylelintrc config to allow usage of px unit for divider
div-Leo
approved these changes
May 6, 2022
rafael-sepeda
approved these changes
May 6, 2022
martimalek
approved these changes
May 6, 2022
Co-authored-by: martimalek <[email protected]>
Co-authored-by: martimalek <[email protected]>
Co-authored-by: martimalek <[email protected]>
Co-authored-by: martimalek <[email protected]>
Co-authored-by: martimalek <[email protected]>
martimalek
added a commit
that referenced
this pull request
Jul 11, 2022
* chore(release): 1.15.1 [skip ci] ### [1.15.1](v1.15.0...v1.15.1) (2021-10-15) ### Bug Fixes * adding disabled state for datepicker ([#181](#181)) ([313f424](313f424)) * test: re-enabling and fixing the tests for PhoneInput (#183) Author: @lloydaf * feat: added Accordion component (#172) * feat: add Accordion component and basic documentation * feat: added test * chore: remove default export and add PropsWithChildren * chore: update api table * chore: chheck place holder to not be in document * chore: added onExpand and onCollapse * chore: solve lint issues * chore: solve lint issues * chore: solve lint issues Co-authored-by: David Barrera <[email protected]> * chore(release): 1.16.0 [skip ci] ## [1.16.0](v1.15.1...v1.16.0) (2021-10-25) ### Features * added Accordion component ([#172](#172)) ([439b67c](439b67c)) * fix: add layout props to TableHeaderCell and TableCell (#185) * chore(release): 1.16.1 [skip ci] ### [1.16.1](v1.16.0...v1.16.1) (2021-10-26) ### Bug Fixes * add layout props to TableHeaderCell and TableCell ([#185](#185)) ([0915c58](0915c58)) * fix: cleanup Modal timeout (#193) * chore(release): 1.16.2 [skip ci] ### [1.16.2](v1.16.1...v1.16.2) (2021-11-17) ### Bug Fixes * cleanup Modal timeout ([#193](#193)) ([fcb1341](fcb1341)) * feat: add secondary variant for text component (#170) * chore(release): 1.17.0 [skip ci] ## [1.17.0](v1.16.2...v1.17.0) (2021-12-09) ### Features * add secondary variant for text component ([#170](#170)) ([f056fea](f056fea)) * Replace Chris with Leo * Support responsive `size` values for Headline (#196) * fix: spread Box props correctly Closes issue #190 * test: add test that reproduces the bug * fix: add default theme declaration * fix: avoid collision of `color` prop * chore: remove StyledComponentBase typing * fix: update prop compatibility * test: remove snapshot * chore(release): 1.17.1 [skip ci] ### [1.17.1](v1.17.0...v1.17.1) (2022-01-26) ### Bug Fixes * spread Box props correctly ([9dac265](9dac265)), closes [#190](#190) * Feat/add missing icons (#200) * feat: update assets icons to match design icons * feat: generate icons with new asset icons * fix: redirect import deprecated icons to icon/index * fix: import icons from icons/index file * docs: add new icons & divide in sections * test: update snapshots style-components classes names Co-authored-by: Leonardo Di Vittorio <[email protected]> * feat: add grid system components (#198) * test: add test that reproduces the bug * fix: add default theme declaration * fix: avoid collision of `color` prop * chore: remove StyledComponentBase typing * fix: update prop compatibility * feat: add grid system components * refactor: use flexbox approach * refactor: rework flex approach * docs: add stacking card example * fix: remove unnused import * test: add snapshot tests * feat: avoid setting marging left when 0 * feat: only use 4 decimals * test: remove old snapshot * Feat/add missing icons (#200) * feat: update assets icons to match design icons * feat: generate icons with new asset icons * fix: redirect import deprecated icons to icon/index * fix: import icons from icons/index file * docs: add new icons & divide in sections * test: update snapshots style-components classes names Co-authored-by: Leonardo Di Vittorio <[email protected]> * 1.17.2-canary.0 * chore: expose grid components Co-authored-by: Leonardo <[email protected]> Co-authored-by: Leonardo Di Vittorio <[email protected]> * chore(release): 1.18.0 [skip ci] ## [1.18.0](v1.17.1...v1.18.0) (2022-02-01) ### Features * add grid system components ([#198](#198)) ([c435683](c435683)), closes [#200](#200) * docs: update changelog * fix: replace weak prop with secondary (#206) * chore(release): 1.18.1 [skip ci] ### [1.18.1](v1.18.0...v1.18.1) (2022-02-05) ### Bug Fixes * replace weak prop with secondary ([#206](#206)) ([37a7f4f](37a7f4f)) * feat: remove brand colors from semantic colors object (#208) * chore(release): 1.19.0 [skip ci] ## [1.19.0](v1.18.1...v1.19.0) (2022-03-08) ### Features * remove brand colors from semantic colors object ([#208](#208)) ([1ccb794](1ccb794)) * feat: adds select page size component (#201) * chore(release): 1.20.0 [skip ci] ## [1.20.0](v1.19.0...v1.20.0) (2022-04-21) ### Features * adds select page size component ([#201](#201)) ([068b958](068b958)) * Update MAINTAINERS.md * Update CODEOWNERS * chore: use generated id as key in datepicker (#214) Co-authored-by: Martí Malek <[email protected]> * docs: updated docs for table (#223) * feature: Add Divider component to Wave (#221) * feat: add Divider component to Wave * feat(divider): add default offset to the divider add additional tests to check the default offset * fix(divider): allow usage of 1px in .stylelintrc Needed to change stylelintrc config to allow usage of px unit for divider * Update src/components/Divider/Divider.spec.tsx Co-authored-by: martimalek <[email protected]> * Update src/components/Divider/Divider.spec.tsx Co-authored-by: martimalek <[email protected]> * Update src/components/Divider/docs/Divider.mdx Co-authored-by: martimalek <[email protected]> * Update src/components/Divider/docs/Divider.mdx Co-authored-by: martimalek <[email protected]> * Update src/components/Divider/docs/Divider.mdx Co-authored-by: martimalek <[email protected]> Co-authored-by: Ján Hamara <[email protected]> Co-authored-by: martimalek <[email protected]> * 1.21.0 * Update changelog.mdx Had to update changelog manually, as v1.21.0 had to also be published manually (due to PR having used feature: keyword, not feat: keyword - which caused that workflow for creating release, updating changelog and publishing package did not get triggered) * feat: update missing icons (#228) * feat: add missin icon svg * feat: generate missing icons Co-authored-by: Leonardo Di Vittorio <[email protected]> * chore(release): 1.22.0 [skip ci] ## [1.22.0](v1.21.0...v1.22.0) (2022-05-23) ### Features * update missing icons ([#228](#228)) ([2f840e5](2f840e5)) * docs: fix the header UI navigation in mobile view (#231) * feat: introduce `useToggle` hook (#220) * chore(release): 1.23.0 [skip ci] ## [1.23.0](v1.22.0...v1.23.0) (2022-05-26) ### Features * introduce `useToggle` hook ([#220](#220)) ([83cbad8](83cbad8)) * feat: Add Popover component to wave (#230) * feat(popover): add popover component to wave * feat(popover): add props table to docz * refactor: move util and helper methods to common utils * feat(popover): simplify the implementation of popover * feat(popover): remove unneeded comments * feat(popover): add aria attributes, describe placements in props table * feat(popover): change variable name for popover content state reference * Update src/components/Popover/docs/Popover.mdx * feat(popover): fix some of the docs texts * feat(popover): update useEffect with openByDefault * feat(popover): remove useStateWithTimeout hook * feat: popover (#234) * feat(popover): add popover component to wave * feat(popover): add props table to docz * refactor: move util and helper methods to common utils * feat(popover): simplify the implementation of popover * feat(popover): remove unneeded comments * feat(popover): add aria attributes, describe placements in props table * feat(popover): change variable name for popover content state reference * Update src/components/Popover/docs/Popover.mdx Co-authored-by: martimalek <[email protected]> * feat(popover): fix some of the docs texts * feat(popover): update useEffect with openByDefault * feat(popover): remove useStateWithTimeout hook Co-authored-by: Ján Hamara <[email protected]> Co-authored-by: Jan Hamara <[email protected]> * Use service user's token * test another token * Another try * Revert to github token * chore(release): 1.24.0 [skip ci] ## [1.24.0](v1.23.0...v1.24.0) (2022-06-02) ### Features * Add Popover component to wave ([#230](#230)) ([0c49ab3](0c49ab3)) * popover ([#234](#234)) ([8e4075b](8e4075b)) * docs: add JanHamara as a contributor for code, doc, review (#236) * docs: update README.md [skip ci] * docs: update .all-contributorsrc [skip ci] Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com> * fix: adding layout to the compose styles (#233) * chore(release): 1.24.1 [skip ci] ### [1.24.1](v1.24.0...v1.24.1) (2022-06-03) ### Bug Fixes * adding layout to the compose styles ([#233](#233)) ([48af1d1](48af1d1)) * docs: add div-Leo as a contributor for code, doc, bug, review (#243) * docs: update README.md [skip ci] * docs: update .all-contributorsrc [skip ci] Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com> * docs: add rafael-sepeda as a contributor for design, blog (#244) * improvement: adding bugs and PRs to Lloyd's contributions readme section (#235) Co-authored-by: Lloyd Francis <[email protected]> * fix: export popover component in components index (#246) Authored-by: Martí Malek <[email protected]> * chore(release): 1.24.2 [skip ci] ### [1.24.2](v1.24.1...v1.24.2) (2022-06-08) ### Bug Fixes * export popover component in components index ([#246](#246)) ([0def6b8](0def6b8)) * fix: giving singlevalue full max width to override the selectlist styles (#248) * feat: add InfoBanner component (#225) * feat: initial InfoBanner version with limited variants * feat: add infobanner with all the variants * test: cover different variants with tests * docs: add examples and combinations for InfoBanner * docs: improve the basic description of the InfoBanner * chore: clean unnused component * refactor: use radii and Spaces instead of hard coded values * feat: add better accessibility through setting a role attribute * feat: align infobanner text to the left * feat: export InfoBanner from components index * chore: update lock file Co-authored-by: Martí Malek <[email protected]> Co-authored-by: Nikolai Lopin <[email protected]> * chore(release): 1.25.0 [skip ci] ## [1.25.0](v1.24.2...v1.25.0) (2022-06-13) ### Features * add InfoBanner component ([#225](#225)) ([e434f5a](e434f5a)) ### Bug Fixes * giving singlevalue full max width to override the selectlist styles ([#248](#248)) ([115ae70](115ae70)) * fix: window is undefined error in next.js projects the issue is solved by updating `react-window-select` library to v4 The update brought some follow-up changes described in the pull request #222 * chore(release): 1.25.1 [skip ci] ### [1.25.1](v1.25.0...v1.25.1) (2022-06-16) ### Bug Fixes * window is undefined error in next.js projects ([b89bfc9](b89bfc9)), closes [#222](#222) * docs: remove the experimental label for table (#255) Co-authored-by: semantic-release-bot <[email protected]> Co-authored-by: Lloyd Francis <[email protected]> Co-authored-by: David Barrera <[email protected]> Co-authored-by: David Barrera <[email protected]> Co-authored-by: Nikolai Lopin <[email protected]> Co-authored-by: Jonathan Trujillo <[email protected]> Co-authored-by: Leonardo <[email protected]> Co-authored-by: Leonardo Di Vittorio <[email protected]> Co-authored-by: Vinayaka Hegde <[email protected]> Co-authored-by: Nikolai Lopin <[email protected]> Co-authored-by: Martí Malek <[email protected]> Co-authored-by: Vitaly Baev <[email protected]> Co-authored-by: Jan Hamara <[email protected]> Co-authored-by: Ján Hamara <[email protected]> Co-authored-by: Leonardo Di Vittorio <[email protected]> Co-authored-by: allcontributors[bot] <46447321+allcontributors[bot]@users.noreply.github.com>
github-actions bot
pushed a commit
that referenced
this pull request
Aug 20, 2022
## [2.1.0-next.2](v2.1.0-next.1...v2.1.0-next.2) (2022-08-20) ### Bug Fixes * adding disabled state for datepicker ([#181](#181)) ([313f424](313f424)) * conflicts between next and main ([#257](#257)) ([0a103ef](0a103ef)), closes [#181](#181) [#183](#183) [#172](#172) [#172](#172) [#185](#185) [#185](#185) [#193](#193) [#193](#193) [#170](#170) [#170](#170) [#196](#196) [#190](#190) [#190](#190) [#200](#200) [#198](#198) [#200](#200) [#200](#200) [#206](#206) [#206](#206) [#208](#208) [#208](#208) [#201](#201) [#201](#201) [#214](#214) [#223](#223) [#221](#221) [#228](#228) [#228](#228) [#231](#231) [#220](#220) [#220](#220) [#230](#230) [#234](#234) [#230](#230) [#234](#234) [#236](#236) [#233](#233) [#233](#233) [#243](#243) [#244](#244) [#235](#235) [#246](#246) [#246](#246) [#248](#248) [#225](#225) [#225](#225) [#248](#248) [#222](#222) [#222](#222) [#255](#255) * conflicts between next and main v2 ([#281](#281)) ([bec6bdc](bec6bdc))
|
🎉 This PR is included in version 3.0.0-next.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What:
This PR adds a new Divider component to the Wave Design System, as per: #218
Why:
Closes #218
A number of our tools that use Wave DS, including BOMT, DMT, PMT (and possibly more) have many instances of a horizontal or vertical line separating pieces of content. A Divider component is often a core component in design systems, therefore, this issue proposed the addition of new component to our Wave design system for these and other use-cases.
Media:
Checklist: