Skip to content

Bug/audio brief fix#16

Open
mariel-delagarza wants to merge 5 commits intomainfrom
bug/audio-brief-fix
Open

Bug/audio brief fix#16
mariel-delagarza wants to merge 5 commits intomainfrom
bug/audio-brief-fix

Conversation

@mariel-delagarza
Copy link
Member

What this does

This PR fixes a couple of bugs with the audio briefs player (used in the Global Forecast digital reports).

  • Fixes an issue where the menu would appear below or outside of the component
  • Fixes closing the menu; it now closes on clicking outside of it, matching expected behavior

Everything else remains the same.

To test in Shorthand, use the below script and <csis-audio-brief></csis-audio-brief>

<script src="https://cdn.jsdelivr.net/gh/CSIS-iLab/ilab-web-components@c2a5f270f526aa2cdcbec74b2f35a99f8d1db40f/dist/widgets-dev.js"></script>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant