Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit 520dc49

Browse filesBrowse files
committed
feat: CSidebarNavItem: add possibility of passing icon as node
1 parent 4de6508 commit 520dc49
Copy full SHA for 520dc49

File tree

Expand file treeCollapse file tree

1 file changed

+3
-3
lines changed
Filter options
Expand file treeCollapse file tree

1 file changed

+3
-3
lines changed

‎src/template/CSidebarNavItem.js

Copy file name to clipboardExpand all lines: src/template/CSidebarNavItem.js
+3-3Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import React from 'react'
1+
import React, { isValidElement } from 'react'
22
import PropTypes from 'prop-types'
33
import classNames from 'classnames'
44
import { CLink, CBadge } from '../index'
@@ -35,7 +35,7 @@ const CSidebarNavItem = props => {
3535
addLinkClass
3636
)
3737

38-
const routerLinkProps = rest.to && { exact: true, activeClassName: 'c-active'}
38+
const routerLinkProps = rest.to && { exact: true, activeClassName: 'c-active' }
3939
return (
4040
<li className={classes} ref={innerRef}>
4141
{ children ||
@@ -44,7 +44,7 @@ const CSidebarNavItem = props => {
4444
{...routerLinkProps}
4545
{...rest}
4646
>
47-
{ icon && <CIcon {...iconProps(icon)}/>}
47+
{ icon && isValidElement(icon) ? icon : <CIcon {...iconProps(icon)}/> }
4848
{ fontIcon && <i className={`c-sidebar-nav-icon ${fontIcon}`}/>}
4949
{name}
5050
{ badge && <CBadge {...{...badge, text: null}}>{badge.text}</CBadge>}

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.