GitHub/storybook
1
0
Fork 0
mirror of https://github.com/storybookjs/storybook.git synced 2025-04-06 07:21:16 +08:00
Code Issues Packages Projects Releases Wiki Activity

FIX an issue where forwardRefs make addon-info throw

Browse Source
This commit is contained in:
Norbert de Langen 2019-05-22 11:05:01 +02:00
parent 3eaf111aa3
commit b558cbb531
2 changed files with 251 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
addons/info/src/components/Story.js
View File

@ -358,17 +358,22 @@ class Story extends Component {
extract(innerChildren.props.children);
}
if (isForwardRef(innerChildren)) {
extract(innerChildren.type.render(innerChildren.props));
try {
// this might fail because of hooks being used
extract(innerChildren.type.render(innerChildren.props));
} catch (e) {
// do nothing
}
}
if (
typeof innerChildren === 'string' ||
typeof innerChildren.type === 'string' ||
isForwardRef(innerChildren) ||
(Array.isArray(propTablesExclude) && // also ignore excluded types
~propTablesExclude.indexOf(innerChildren.type)) // eslint-disable-line no-bitwise
) {
return;
}
if (innerChildren.type && !types.has(innerChildren.type)) {
types.set(innerChildren.type, true);
}

244
examples/official-storybook/stories/__snapshots__/addon-info.stories.storyshot
View File

@ -555,6 +555,128 @@ exports[`Storyshots Addons|Info/ForwardRef Displays forwarded ref components cor
</tbody>
</table>
</div>
<div>
<h3
style="margin:20px 0 0 0"
>
"Unknown" Component
</h3>
<table
class="info-table"
>
<thead>
<tr>
<th>
property
</th>
<th>
propType
</th>
<th>
required
</th>
<th>
default
</th>
<th>
description
</th>
</tr>
</thead>
<tbody>
<tr>
<td
class="info-table-monospace"
>
disabled
</td>
<td
class="info-table-monospace"
>
<span>
bool
</span>
</td>
<td>
-
</td>
<td>
<span>
false
</span>
</td>
<td />
</tr>
<tr>
<td
class="info-table-monospace"
>
label
</td>
<td
class="info-table-monospace"
>
<span>
string
</span>
</td>
<td>
yes
</td>
<td>
-
</td>
<td />
</tr>
<tr>
<td
class="info-table-monospace"
>
onClick
</td>
<td
class="info-table-monospace"
>
<span>
func
</span>
</td>
<td>
-
</td>
<td>
<span>
onClick
</span>
</td>
<td />
</tr>
<tr>
<td
class="info-table-monospace"
>
style
</td>
<td
class="info-table-monospace"
>
<span>
other
</span>
</td>
<td>
-
</td>
<td>
<span>
{}
</span>
</td>
<td />
</tr>
</tbody>
</table>
</div>
</div>
</div>
</div>
@ -791,6 +913,128 @@ exports[`Storyshots Addons|Info/ForwardRef Uses forwardRef displayName if availa
</tbody>
</table>
</div>
<div>
<h3
style="margin:20px 0 0 0"
>
"ButtonDisplayName" Component
</h3>
<table
class="info-table"
>
<thead>
<tr>
<th>
property
</th>
<th>
propType
</th>
<th>
required
</th>
<th>
default
</th>
<th>
description
</th>
</tr>
</thead>
<tbody>
<tr>
<td
class="info-table-monospace"
>
disabled
</td>
<td
class="info-table-monospace"
>
<span>
bool
</span>
</td>
<td>
-
</td>
<td>
<span>
false
</span>
</td>
<td />
</tr>
<tr>
<td
class="info-table-monospace"
>
label
</td>
<td
class="info-table-monospace"
>
<span>
string
</span>
</td>
<td>
yes
</td>
<td>
-
</td>
<td />
</tr>
<tr>
<td
class="info-table-monospace"
>
onClick
</td>
<td
class="info-table-monospace"
>
<span>
func
</span>
</td>
<td>
-
</td>
<td>
<span>
onClick
</span>
</td>
<td />
</tr>
<tr>
<td
class="info-table-monospace"
>
style
</td>
<td
class="info-table-monospace"
>
<span>
other
</span>
</td>
<td>
-
</td>
<td>
<span>
{}
</span>
</td>
<td />
</tr>
</tbody>
</table>
</div>
</div>
</div>
</div>