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.

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

refactor download for items with captions/transcripts #2286

Merged
merged 1 commit into from
Jan 23, 2025

Conversation

dnoneill
Copy link
Contributor

@dnoneill dnoneill commented Jan 10, 2025

closes #2280
Screenshot 2025-01-22 at 12 47 22 PM
Screenshot 2025-01-22 at 12 46 53 PM

@dnoneill dnoneill marked this pull request as ready for review January 16, 2025 20:23
Copy link
Contributor

@jcoyne jcoyne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add a test for when headers are shown on the CompanionWindowComponent?

@dnoneill dnoneill force-pushed the 2280-download-media branch 2 times, most recently from b8c6f70 to 08127a5 Compare January 22, 2025 16:55
@dnoneill dnoneill force-pushed the 2280-download-media branch from 08127a5 to ce22e3b Compare January 22, 2025 17:42
@dnoneill dnoneill requested a review from jcoyne January 22, 2025 17:45
@dnoneill
Copy link
Contributor Author

test added, other comments implemented

@jcoyne jcoyne merged commit 5f3a10e into main Jan 23, 2025
3 checks passed
@jcoyne jcoyne deleted the 2280-download-media branch January 23, 2025 15:12
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.

Improve download modal file list display
2 participants