f6e99dec8b
* wip * Removed reference to cached_tag_list as the feed enpoint only returns tag_list now. * Removed reference to cached_properties. * Updated <CommentsCount /> and <TagList /> to support additinal className * wip * wip * Undid init scrolling for Algolia articles * Converted homePageFeed.jsx to an ERB template to access Sprockets. * Now the homePageFeed.jsx files is an ERB so it can load Sprockets assets. * wip * Removed accidental commit of debugger statement in ERB. * Now different feed time frames are loaded based on the homepage feed tab clicked. * Prevent default Algolia paging when on the homepage. * Added a tiny comment about feed time frame. * Now feed loads properly for each time frame on home page. * prop name refactor. * Reneabled default Algolia scrolling. * On the homepage, we no longer overwrite article HTML with Algolia data for initial articles. * Assumption is the proper URL for main_image of featured article will be sent in the feed. * Fixed Google Analytics for featured article. * No longer need the new-feed ID for checking if we're on the homepage. * Trying to make codeclimate happy. * wip * Removed reference to cached_tag_list as the feed enpoint only returns tag_list now. * Removed reference to cached_properties. * Updated <CommentsCount /> and <TagList /> to support additinal className * wip * wip * Undid init scrolling for Algolia articles * Converted homePageFeed.jsx to an ERB template to access Sprockets. * Now the homePageFeed.jsx files is an ERB so it can load Sprockets assets. * wip * Removed accidental commit of debugger statement in ERB. * Now different feed time frames are loaded based on the homepage feed tab clicked. * Prevent default Algolia paging when on the homepage. * Added a tiny comment about feed time frame. * Now feed loads properly for each time frame on home page. * prop name refactor. * Reneabled default Algolia scrolling. * On the homepage, we no longer overwrite article HTML with Algolia data for initial articles. * Assumption is the proper URL for main_image of featured article will be sent in the feed. * Fixed Google Analytics for featured article. * No longer need the new-feed ID for checking if we're on the homepage. * Added postcst episodes to <Feed /> state. * Breaking up Article.jsx to make codeclimate happy. * wip * Created the <TodaysPodcastEpisodes /> and <PodcastEpisode /> components. * Fixed issues with some components that were updated for the <FeaturedArticle /> * Added tests for <FeaturedArticle /> * Barreled up <FeaturedArticle /> * Added Storybook stories for <FeaturedArticle /> * Featured articles don't display org, so removed that test. * Added proper today's podcasts components to feed. * Converted some leftover ERB + removed pro checkmark as it doesn't exist yet. * Now logic is same as master for feed timeframe. * Updated <FeaturedArticle /> snapshots. * On frontpage, bookmark button state is managed by Preact components now. * bookmarkedFeedItems is now a prop on <Feed />. * Fixed logic check for initializing reading list icons. * Now <Article /> and <FeaturedArticle /> receive a bookmarkClick prop. * Removed beginnings of scroll handler in <Feed /> as we're using what's there already for now. * Removed logic to not show main image on other time frame feeds as it was not correct. * Fixed issue with 0 rendering when no podcast episodes in feed. * Converted <SaveButton /> to a stateful component. * wip - <Feed /> render prop now passes bookmarked feed item ID set and a click event. * Moved FEED_ICONS to an export so it's easier to use JSX/JS in <Feed /> * Moved <Feed /> into article components. * Now reading list save buttons work properly in Preact components. * Fixed issue with Save buttons in Algolia paging/<Feed /> * Updated snapshots. * Took <Feed /> out of articles barrel file as jest was trying to import FEED_ICONs which is an ERB template. * Now the Preact feed is dynamically imported. * Now the Preact <Feed /> only loads for a logged on user. * Fixed <FeaturedArticle /> so that the save button behaves properly now. * Updated article stories for Storybook. * Putting this back to the order it was because I've removed my code changes for this block. * Fixed when renderFeed is imported for a logged on user. * Add podcasts to feed and render home from server * Add padding to feature laoding div * Update tests * Add more loading articles * Fix conflicts * Add readinglist javascript to initscrolling * Ensure first article is one with image * Organize feed items outside of feed instead of in it Co-authored-by: Ben Halpern <bendhalpern@gmail.com> Co-authored-by: Josh Puetz <josh@grorichpuetz.com>
35 lines
815 B
JavaScript
35 lines
815 B
JavaScript
import { h } from 'preact';
|
|
import PropTypes from 'prop-types';
|
|
|
|
export const CommentsCount = ({ count, articlePath, icon, className }) => {
|
|
if (count > 0) {
|
|
return (
|
|
<div
|
|
className={`article-engagement-count comments-count${
|
|
className ? ` ${className}` : ''
|
|
}`}
|
|
>
|
|
<a href={`${articlePath}#comments`}>
|
|
<img src={icon} alt="chat" loading="lazy" />
|
|
<span className="engagement-count-number">{count}</span>
|
|
</a>
|
|
</div>
|
|
);
|
|
}
|
|
|
|
return null;
|
|
};
|
|
|
|
CommentsCount.defaultProps = {
|
|
count: 0,
|
|
className: null,
|
|
};
|
|
|
|
CommentsCount.propTypes = {
|
|
count: PropTypes.number,
|
|
articlePath: PropTypes.string.isRequired,
|
|
icon: PropTypes.string.isRequired,
|
|
className: PropTypes.string,
|
|
};
|
|
|
|
CommentsCount.displayName = 'CommentsCount';
|