Skip to content
This repository was archived by the owner on Jun 1, 2026. It is now read-only.

fix(popover): fix the undefined classname - #307

Merged
unix merged 1 commit into
geist-org:masterfrom
firede:fix-popover-classname
Jul 2, 2020
Merged

fix(popover): fix the undefined classname#307
unix merged 1 commit into
geist-org:masterfrom
firede:fix-popover-classname

Conversation

@firede

@firede firede commented Jul 2, 2020

Copy link
Copy Markdown
Contributor

Checklist

  • Fix linting errors
  • Label has been added

Change information

Changes

  • Popover: add default portalClassName
  • Popover.Item: add default className

Snapshots

image
image

@codecov

codecov Bot commented Jul 2, 2020

Copy link
Copy Markdown

Codecov Report

Merging #307 into master will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #307   +/-   ##
=======================================
  Coverage   99.95%   99.95%           
=======================================
  Files         186      186           
  Lines        2373     2373           
  Branches      568      568           
=======================================
  Hits         2372     2372           
  Misses          1        1           
Impacted Files Coverage Δ
components/popover/popover.tsx 100.00% <ø> (ø)
components/popover/popover-item.tsx 100.00% <100.00%> (ø)

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 1f8a6a3...e6792ff. Read the comment docs.

@unix unix added the type: bug Something isn't working label Jul 2, 2020

@unix unix left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Great work! Thank you for your contribution.

@unix
unix merged commit c51e158 into geist-org:master Jul 2, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

type: bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants