Skip to content

Commit

Permalink
Blog carousel comments
Browse files Browse the repository at this point in the history
Added comments clarifying the difference between BlogCarouselFull.js and BlogCarouselCropped.js
  • Loading branch information
camillacrane committed Sep 8, 2024
1 parent 93327f7 commit f423c16
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 0 deletions.
2 changes: 2 additions & 0 deletions components/homepage/BlogCarouselCropped.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,8 @@ import Link from 'next/link'
import AudioPlayerStream from '../audioplayers/AudioPlayerStream'
import PostPreview from '../PostPreview'

// There are two BlogCarousel components. On desktop, if there is content in the "WXYC This Week" section, the blog post carousel renders underneath "WXYC This Week" and the audio player at full-screen (BlogCarouselFull.js) However, if there is no content in "WXYC This Week", BlogCarouselCropped.js is called, and is aligned with the audio player.

const BlogCarouselCropped = (props) => {
return (
<div>
Expand Down
2 changes: 2 additions & 0 deletions components/homepage/BlogCarouselFull.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
import PostPreview from '../PostPreview'
import Link from 'next/link'

// There are two BlogCarousel components. On desktop, if there is content in the "WXYC This Week" section, the blog post carousel renders underneath "WXYC This Week" and the audio player at full-screen (BlogCarouselFull.js) However, if there is no content in "WXYC This Week", BlogCarouselCropped.js is called, and is aligned with the audio player.

const BlogCarouselFull = (props) => {
return (
<div className="-mt-5 flex flex-col md:-mt-10 md:mr-10 lg:mt-5 lg:w-full lg:max-w-screen-xl">
Expand Down

0 comments on commit f423c16

Please sign in to comment.