Change the filtered notification count to be in the account avatar (#31262)

This commit is contained in:
Renaud Chaput 2024-08-02 15:18:19 +02:00 committed by GitHub
parent 99c446b7cf
commit cc453f2221
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
4 changed files with 37 additions and 18 deletions

View file

@ -11,6 +11,8 @@ interface Props {
style?: React.CSSProperties; style?: React.CSSProperties;
inline?: boolean; inline?: boolean;
animate?: boolean; animate?: boolean;
counter?: number | string;
counterBorderColor?: string;
} }
export const Avatar: React.FC<Props> = ({ export const Avatar: React.FC<Props> = ({
@ -19,6 +21,8 @@ export const Avatar: React.FC<Props> = ({
size = 20, size = 20,
inline = false, inline = false,
style: styleFromParent, style: styleFromParent,
counter,
counterBorderColor,
}) => { }) => {
const { hovering, handleMouseEnter, handleMouseLeave } = useHovering(animate); const { hovering, handleMouseEnter, handleMouseLeave } = useHovering(animate);
@ -43,6 +47,14 @@ export const Avatar: React.FC<Props> = ({
style={style} style={style}
> >
{src && <img src={src} alt='' />} {src && <img src={src} alt='' />}
{counter && (
<div
className='account__avatar__counter'
style={{ borderColor: counterBorderColor }}
>
{counter}
</div>
)}
</div> </div>
); );
}; };

View file

@ -38,12 +38,11 @@ export const NotificationRequest = ({ id, accountId, notificationsCount }) => {
return ( return (
<div className='notification-request'> <div className='notification-request'>
<Link to={`/notifications/requests/${id}`} className='notification-request__link'> <Link to={`/notifications/requests/${id}`} className='notification-request__link'>
<Avatar account={account} size={36} /> <Avatar account={account} size={40} counter={toCappedNumber(notificationsCount)} />
<div className='notification-request__name'> <div className='notification-request__name'>
<div className='notification-request__name__display-name'> <div className='notification-request__name__display-name'>
<bdi><strong dangerouslySetInnerHTML={{ __html: account?.get('display_name_html') }} /></bdi> <bdi><strong dangerouslySetInnerHTML={{ __html: account?.get('display_name_html') }} /></bdi>
<span className='filtered-notifications-banner__badge'>{toCappedNumber(notificationsCount)}</span>
</div> </div>
<span>@{account?.get('acct')}</span> <span>@{account?.get('acct')}</span>

View file

@ -1,16 +1,3 @@
@mixin avatar-radius {
border-radius: 4px;
background: transparent no-repeat;
background-position: 50%;
background-clip: padding-box;
}
@mixin avatar-size($size: 48px) {
width: $size;
height: $size;
background-size: $size $size;
}
@mixin search-input { @mixin search-input {
outline: 0; outline: 0;
box-sizing: border-box; box-sizing: border-box;

View file

@ -1980,17 +1980,15 @@ body > [data-popper-placement] {
} }
.account__avatar { .account__avatar {
@include avatar-radius;
display: block; display: block;
position: relative; position: relative;
overflow: hidden;
img { img {
display: block; display: block;
width: 100%; width: 100%;
height: 100%; height: 100%;
object-fit: cover; object-fit: cover;
border-radius: 4px;
} }
&-inline { &-inline {
@ -2027,6 +2025,29 @@ body > [data-popper-placement] {
font-size: 15px; font-size: 15px;
} }
} }
&__counter {
$height: 16px;
$h-padding: 5px;
position: absolute;
bottom: -3px;
inset-inline-end: -3px;
padding-left: $h-padding;
padding-right: $h-padding;
height: $height;
border-radius: $height;
min-width: $height - 2 * $h-padding; // to ensure that it is never narrower than a circle
line-height: $height + 1px; // to visually center the numbers
background-color: $ui-button-background-color;
color: $white;
border-width: 1px;
border-style: solid;
border-color: var(--background-color);
font-size: 11px;
font-weight: 500;
text-align: center;
}
} }
a .account__avatar { a .account__avatar {