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

remove unnecessary itemprops #998

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

remove unnecessary itemprops #998

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
47357cf
remove unnecessary itemprops
evoX0 Feb 28, 2019
65585b0
Merge branch 'master' into fix/duplicating-jsonld-schema
evoX0 Feb 28, 2019
95ca815
Merge branch 'master' into fix/duplicating-jsonld-schema
evoX0 Mar 2, 2019
e22548a
add bool noSchema
evoX0 Mar 6, 2019
ac59c0c
Merge remote-tracking branch 'origin/fix/duplicating-jsonld-schema' i…
evoX0 Mar 6, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 10 additions & 4 deletions src/atoms/BylineWithTwoLines.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,17 +52,23 @@ Email.propTypes = {
};


const BylineWithTwoLines = ({ src, name, email }) => (
const BylineWithTwoLines = ({ src, name, email }, noSchema) => (
<div>
{ src ? (
<BylineImage
src={src}
/>
) : <WrapAvatar><SvgIcon name="user" /></WrapAvatar>}
<TextBlock primaryText={name}>
<Name itemProp="author" itemScope itemType="http://schema.org/Person">
<span itemProp="name">{name}</span>
</Name>
{ noSchema ? (
<Name>
<span>{name}</span>
</Name>
) : (
<Name itemProp="author" itemScope itemType="http://schema.org/Person">
<span itemProp="name">{name}</span>
</Name>
)}
<Email email={email}>
{email}
</Email>
Expand Down