Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

WSTEAM1-733 Display audio on the Live Page #12031

Open
wants to merge 18 commits into
base: latest
Choose a base branch
from
Open

WSTEAM1-733 Display audio on the Live Page #12031

Show file tree
Hide file tree
Changes from 11 commits
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
978a0be
render audio player on live posts
emilysaffron Oct 4, 2024
198b788
Merge branch 'latest' into WSTEAM1-733-display-audio-on-live-page
emilysaffron Oct 4, 2024
f79040e
styling
emilysaffron Oct 4, 2024
35dd653
update integration snapshots
emilysaffron Oct 4, 2024
c271919
update integration snapshots again
emilysaffron Oct 4, 2024
95fed28
update in line with snapshots
emilysaffron Oct 10, 2024
945e938
Merge branch 'latest' into WSTEAM1-733-display-audio-on-live-page
emilysaffron Oct 10, 2024
ec03215
update integration snapshots
emilysaffron Oct 10, 2024
d0b1716
Merge branch 'WSTEAM1-733-display-audio-on-live-page' of github.com:b…
emilysaffron Oct 10, 2024
b4bb536
update unit test
emilysaffron Oct 10, 2024
1f28d77
Merge branch 'latest' into WSTEAM1-733-display-audio-on-live-page
emilysaffron Oct 10, 2024
e98fd89
update unit and integration test
emilysaffron Oct 14, 2024
233b357
Merge branch 'latest' into WSTEAM1-733-display-audio-on-live-page
emilysaffron Oct 14, 2024
aff9c63
typo
emilysaffron Oct 14, 2024
d3fd256
Merge branch 'latest' into WSTEAM1-733-display-audio-on-live-page
emilysaffron Oct 15, 2024
12a06d4
fixes spacings
emilysaffron Oct 15, 2024
e492ae3
Merge branch 'WSTEAM1-733-display-audio-on-live-page' of github.com:b…
emilysaffron Oct 15, 2024
3b69d94
Merge branch 'latest' into WSTEAM1-733-display-audio-on-live-page
emilysaffron Oct 15, 2024
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
2 changes: 1 addition & 1 deletion src/app/components/MediaLoader/configs/clipMedia.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,7 @@ export default ({
...(videoId && { clipPID: videoId }),
},
},
placeholderConfig,
...(type !== 'audio' && { placeholderConfig }),
showAds,
};
};
5 changes: 2 additions & 3 deletions src/app/components/MediaLoader/index.styles.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -50,10 +50,9 @@ export default {
!isEmbedded && commonMarginSpacing,
],

audioMediaContainer: ({ palette }: Theme) =>
audioMediaContainer: () =>
css({
backgroundColor: palette.WHITE,
height: '165px',
height: '130px',
emilysaffron marked this conversation as resolved.
Show resolved Hide resolved
}),

standardMediaContainer: ({ palette }: Theme) =>
Expand Down
3 changes: 1 addition & 2 deletions src/app/pages/OnDemandAudioPage/__snapshots__/index.test.tsx.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -698,8 +698,7 @@ exports[`OnDemand Radio Page should match snapshot 1`] = `
}

.emotion-30 {
background-color: #FFFFFF;
height: 165px;
height: 130px;
}

<div>
Expand Down
2 changes: 1 addition & 1 deletion src/integration/pages/liveRadio/gahuza/__snapshots__/canonical.test.js.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -116,7 +116,7 @@ exports[`Canonical Live Radio Media Loader renders a valid container 1`] = `
data-e2e="media-loader__container"
>
<div
class="bbc-1eq7znh"
class="bbc-1i3n52v"
data-e2e="media-player"
/>
</figure>
Expand Down
2 changes: 1 addition & 1 deletion src/integration/pages/liveRadio/korean/__snapshots__/canonical.test.js.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,7 @@ exports[`Canonical Korean Live Radio Page Media Loader renders a valid container
data-e2e="media-loader__container"
>
<div
class="bbc-1eq7znh"
class="bbc-1i3n52v"
data-e2e="media-player"
/>
</figure>
Expand Down
2 changes: 1 addition & 1 deletion src/integration/pages/liveRadio/sinhala/__snapshots__/canonical.test.js.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,7 @@ exports[`Canonical Sinhala Live Radio Page Media Loader renders a valid containe
data-e2e="media-loader__container"
>
<div
class="bbc-1eq7znh"
class="bbc-1i3n52v"
data-e2e="media-player"
/>
</figure>
Expand Down
2 changes: 1 addition & 1 deletion src/integration/pages/onDemandAudioPage/indonesia/__snapshots__/canonical.test.js.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -124,7 +124,7 @@ exports[`Canonical On Demand Audio Page Media Loader renders a valid container 1
data-e2e="media-loader__container"
>
<div
class="bbc-1eq7znh"
class="bbc-1i3n52v"
data-e2e="media-player"
/>
</figure>
Expand Down
2 changes: 1 addition & 1 deletion src/integration/pages/onDemandAudioPage/pashto/__snapshots__/canonical.test.js.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -159,7 +159,7 @@ exports[`Canonical On Demand Audio Page Media Loader renders a valid container 1
data-e2e="media-loader__container"
>
<div
class="bbc-1eq7znh"
class="bbc-1i3n52v"
data-e2e="media-player"
/>
</figure>
Expand Down
2 changes: 1 addition & 1 deletion src/integration/pages/onDemandAudioPage/pashtoBrand/__snapshots__/canonical.test.js.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -159,7 +159,7 @@ exports[`Canonical On Demand Audio Page Media Loader renders a valid container 1
data-e2e="media-loader__container"
>
<div
class="bbc-1eq7znh"
class="bbc-1i3n52v"
data-e2e="media-player"
/>
</figure>
Expand Down
2 changes: 1 addition & 1 deletion src/integration/pages/podcastPage/portugueseBrand/__snapshots__/canonical.test.js.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -159,7 +159,7 @@ exports[`Canonical Podcast Page Media Loader renders a valid container 1`] = `
data-e2e="media-loader__container"
>
<div
class="bbc-1eq7znh"
class="bbc-1i3n52v"
data-e2e="media-player"
/>
</figure>
Expand Down
2 changes: 1 addition & 1 deletion src/integration/pages/podcastPage/portugueseEpisode/__snapshots__/canonical.test.js.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -159,7 +159,7 @@ exports[`Canonical Podcast Page Media Loader renders a valid container 1`] = `
data-e2e="media-loader__container"
>
<div
class="bbc-1eq7znh"
class="bbc-1i3n52v"
data-e2e="media-player"
/>
</figure>
Expand Down
79 changes: 3 additions & 76 deletions ws-nextjs-app/integration/pages/live/pidgin/__snapshots__/canonical.test.ts.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -226,82 +226,9 @@ exports[`Canonical Live Media Loader renders a valid container 1`] = `
data-e2e="media-loader__container"
>
<div
class="css-171plmm"
data-e2e="media-loader__placeholder"
>
<div
class="css-czokpf"
data-e2e="media-player__guidance"
>
<strong
aria-hidden="true"
class="guidance-message css-e3010g"
>
Contains strong language and some upsetting scenes.
</strong>
<noscript
class="css-lllaou"
>
&lt;strong&gt;Dem no support media player for your device&lt;/strong&gt;
</noscript>
</div>
<button
class="focusIndicatorRemove css-1yv3cvg"
type="button"
>
<span
class="css-m04vo2"
>
Contains strong language and some upsetting scenes. Play video, "Lionel Messi Skills", Duration 0,34
</span>
<div
aria-hidden="true"
class="css-1xdc5yd"
>
<svg
aria-hidden="true"
class="css-vja2s5 e1s2dgdk5"
focusable="false"
height="12"
viewBox="0 0 12 12"
width="12"
xmlns="http://www.w3.org/2000/svg"
>
<g
fill="none"
fill-rule="evenodd"
>
<path
d="M.5.6h12v12H.5z"
/>
<path
d="M2.144.96v11.28l8.712-5.64z"
fill="currentColor"
/>
</g>
</svg>
</div>
<time
aria-hidden="true"
class="css-1lcsotz"
datetime="PT34S"
>
00:34
</time>
</button>
<div
class="css-lgx6cw"
style="padding-bottom:0;overflow:hidden"
>
<img
alt=""
class="css-1uq6tv3"
src="https://ichef.test.bbci.co.uk/images/ic/512xn/p01vzbd6.jpg.webp"
srcset="https://ichef.test.bbci.co.uk/images/ic/240xn/p01vzbd6.jpg.webp 240w, https://ichef.test.bbci.co.uk/images/ic/320xn/p01vzbd6.jpg.webp 320w, https://ichef.test.bbci.co.uk/images/ic/480xn/p01vzbd6.jpg.webp 480w, https://ichef.test.bbci.co.uk/images/ic/624xn/p01vzbd6.jpg.webp 624w, https://ichef.test.bbci.co.uk/images/ic/800xn/p01vzbd6.jpg.webp 800w"
style="aspect-ratio:auto"
/>
</div>
</div>
class="css-9p9hvp"
data-e2e="media-player"
/>
<figcaption
class="css-12vqlmi"
dir="ltr"
Expand Down
Loading
Loading