Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(modal): fix type of Modal.Action #351

Merged
merged 1 commit into from
Jul 27, 2020
Merged

fix(modal): fix type of Modal.Action #351

merged 1 commit into from
Jul 27, 2020

Conversation

unix
Copy link
Member

@unix unix commented Jul 27, 2020

Checklist

  • Fix linting errors
  • Tests have been added / updated (or snapshots)

Change information

fixed #350

@unix unix self-assigned this Jul 27, 2020
@unix unix added the type: bug Something isn't working label Jul 27, 2020
@codecov
Copy link

codecov bot commented Jul 27, 2020

Codecov Report

Merging #351 into rc will increase coverage by 0.05%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##               rc     #351      +/-   ##
==========================================
+ Coverage   99.90%   99.95%   +0.05%     
==========================================
  Files         171      186      +15     
  Lines        2136     2399     +263     
  Branches      497      577      +80     
==========================================
+ Hits         2134     2398     +264     
+ Misses          2        1       -1     
Impacted Files Coverage Δ
components/css-baseline/css-baseline.tsx 100.00% <ø> (ø)
components/description/description.tsx 100.00% <ø> (ø)
components/modal/modal-actions.tsx 100.00% <ø> (ø)
components/modal/modal-content.tsx 100.00% <ø> (ø)
components/modal/modal-title.tsx 100.00% <ø> (ø)
components/modal/modal-wrapper.tsx 100.00% <ø> (ø)
components/popover/popover.tsx 100.00% <ø> (ø)
components/radio/radio-context.ts 100.00% <ø> (+25.00%) ⬆️
components/radio/radio-description.tsx 100.00% <ø> (ø)
components/select/select-context.ts 100.00% <ø> (ø)
... and 65 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c5b437a...a122846. Read the comment docs.

@unix unix merged commit c2e6cd0 into geist-org:rc Jul 27, 2020
@unix unix deleted the modal branch July 27, 2020 10:32
unix added a commit that referenced this pull request Aug 14, 2020
unix added a commit that referenced this pull request Aug 14, 2020
* feat(snippet): add custom symbol and toast

* docs(snippet): add docs for custom symbol and toast

* fix(snippet): default toast type as normal type

* test(snippet): add custom symbol and toast

* docs(snippet): add enum type of APIs

* docs(toast): add normal-types

* chore: release v1.8.0-canary.1

* feat(modal): optimize the animation of layer

* test: update snapshots

* chore: release v1.8.0-canary.2

* docs: format import statement

* feat(modal): optimize the max width, height and alignment of layer

* test: update snapshots

* chore: release v1.8.0-canary.3

* Button (#319)

* feat(button): center the icon if the button is empty

* test(button): add testcase for icon only

* docs(button): add example for icon only

* chore: release v1.8.0-canary.4

* fix(snippet): remove space when the symbol is empty (#325)

* fix(snippet): remove space when the symbol is empty

style(snippet): remove unnecessary escape symbols

* fix(snippet): ignore spaces in symbol

style: fix lint warning

* chore: release v1.8.0-canary.5

* feat(tabs): sync the label and set value to required (#334)

* feat(tabs): sync the label and set value to required

* test(tabs): add testcase for label sync

* docs(tabs): update value to required

* feat(modal): use Button to reconstrust Modal.Action (#332)

* feat(modal): use Button to reconstrust Modal.Action

* docs(modal): add example for action loading

* test: update snapshots

* chore: release v1.8.0-canary.6

* fix(input): always synchronize external value and ignore changes (#336)

* fix(input): always synchronize external value and ignore changes

* feat(input): support imperative API to update

* fix(textarea): imperative api same as input (#341)

* feat(dropdown): allow dropdown to set the specified container (#344)

* feat(dropdown): allow dropdown to set the specified container

* test(modal): update snapshots

* docs(select): add example for custom popup container

* fix(dropdown): fix type of getPopupContainer

* test(dropdown): add testcase for specified container rendering

* chore: release v1.8.0-canary.7

* fix(link): fix hard-coded background color (#347)

* style(description): fix title font-weight (#348)

* docs(link): fix duplicate examples (#346)

* style(popover-item): the whole item should be clickable when using with link (#345)

* fix(modal): fix type of Modal.Action (#351)

* chore: release v1.8.0-canary.8

* feat(modal): lock tab action inside modal (#354)

* feat(button): add style to focus buttons

* feat(collections): add util function

* feat(modal): lock tab action inside modal

* test(modal): add tests for modal focus

* test: update style of button

* fix(table): fix column's props are not tracked (#362)

* chore: release v1.8.0-canary.9

* fix(table): children of column should be kept in sync (#364)

* chore: release v1.8.0-canary.10

Co-authored-by: Augusto <augustoconti10@gmail.com>
Co-authored-by: yqrashawn <namy.19@gmail.com>
Co-authored-by: Zhao Lei <firede@firede.us>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
type: bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant