Skip to content
This repository has been archived by the owner on Nov 14, 2023. It is now read-only.

fix: Panel add class name #70

Merged
merged 2 commits into from
Sep 3, 2019
Merged

fix: Panel add class name #70

merged 2 commits into from
Sep 3, 2019

Conversation

shenghou
Copy link
Contributor

@shenghou shenghou commented Sep 3, 2019

No description provided.

@netlify
Copy link

netlify bot commented Sep 3, 2019

Deploy preview for wizard-ui ready!

Built with commit 7a554f6

https://deploy-preview-70--wizard-ui.netlify.com

@codecov
Copy link

codecov bot commented Sep 3, 2019

Codecov Report

Merging #70 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #70   +/-   ##
=======================================
  Coverage   81.34%   81.34%           
=======================================
  Files          33       33           
  Lines        1131     1131           
  Branches      234      234           
=======================================
  Hits          920      920           
  Misses        208      208           
  Partials        3        3
Impacted Files Coverage Δ
src/components/Panel/index.tsx 100% <100%> (ø) ⬆️

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 c1b9940...7a554f6. Read the comment docs.

<BootstrapPanel {...restProps} className={classNames(bgClass, textClass)}>
<BootstrapPanel
{...restProps}
className={classNames(bgClass, textClass, { className: true })}
Copy link
Member

Choose a reason for hiding this comment

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

className 是个常量? 这样 class 变成了 xxx className

@wangkailang wangkailang added the changes Code Need Change label Sep 3, 2019
@shenghou shenghou added the 3-Changed Code has Changed label Sep 3, 2019
@wangkailang wangkailang added the 4-Approved Code PR approved label Sep 3, 2019
@wangkailang wangkailang merged commit 1e588fe into master Sep 3, 2019
@wangkailang wangkailang deleted the fix/panel_classname branch September 3, 2019 09:22
wangkailang added a commit that referenced this pull request Sep 4, 2019
…f github.com:xsky-fe/wizard-ui into refactor/compare-with-react-bootstrap

* 'refactor/compare-with-react-bootstrap' of github.com:xsky-fe/wizard-ui:

* 'master' of github.com:xsky-fe/wizard-ui:
  fix: Panel add class name (#70)

# Conflicts:
#	src/components/Panel/index.tsx
#	src/interface.tsx
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
1-Review Code review 3-Changed Code has Changed 4-Approved Code PR approved changes Code Need Change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants