-
-
Notifications
You must be signed in to change notification settings - Fork 48
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
Unavailable videos comparisons list #1522
base: main
Are you sure you want to change the base?
Conversation
… comparisonThumbnailComponent
…the error on a toggle bar when it is not compact and on an overlay on top of the entity picture when it is compact
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks good to me.
I'm going to refactor a duplicate piece of code before merging.
<Box | ||
display="flex" | ||
justifyContent="center" | ||
alignItems="center" | ||
position="absolute" | ||
top="0" | ||
color="white" | ||
bgcolor="rgba(0,0,0,.6)" | ||
width="100%" | ||
height="100%" | ||
sx={{ | ||
[theme.breakpoints.down('sm')]: { | ||
fontSize: '0.6rem', | ||
}, | ||
}} | ||
> | ||
<Typography textAlign="center" fontSize="inherit"> | ||
{entity.type == TypeEnum.VIDEO | ||
? t('video.notAvailableAnymore') | ||
: t('entityCard.thisElementIsNotAvailable')} | ||
</Typography> | ||
</Box> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The same components are also used by <EntitySelectorInnerAuth>
.
Let's factorize them in their own component.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just realized that those changes break the visual of the My comparisons page on the presidentielle2022
poll.
We should fix it.
Related to #1070
This PR has for goal the display of unavailable entity on the comparisons list page.
The unavailable video must be highlighted with a message overlay on top of the image like for the comparison page