Fix James's nits
This commit is contained in:
parent
b5e672a5df
commit
088bccb815
|
@ -77,7 +77,7 @@ export function GroupOverview(props: {
|
||||||
maxResults={MAX_TRENDING_POSTS}
|
maxResults={MAX_TRENDING_POSTS}
|
||||||
defaultFilter={'all'}
|
defaultFilter={'all'}
|
||||||
additionalFilter={{ groupSlug: group.slug }}
|
additionalFilter={{ groupSlug: group.slug }}
|
||||||
persistPrefix={`group-${group.slug}`}
|
persistPrefix={`group-trending-${group.slug}`}
|
||||||
/>
|
/>
|
||||||
<GroupAbout
|
<GroupAbout
|
||||||
group={group}
|
group={group}
|
||||||
|
@ -181,22 +181,14 @@ function GroupOverviewPinned(props: {
|
||||||
{element}
|
{element}
|
||||||
|
|
||||||
{editMode && (
|
{editMode && (
|
||||||
<div>
|
<CrossIcon
|
||||||
<button
|
onClick={() => {
|
||||||
className=" text-gray-500 hover:text-gray-700"
|
const newPinned = group.pinnedItems.filter((item) => {
|
||||||
onClick={() => {
|
return item.itemId !== group.pinnedItems[index].itemId
|
||||||
console.log(group.pinnedItems)
|
})
|
||||||
const newPinned = group.pinnedItems.filter((item) => {
|
updateGroup(group, { pinnedItems: newPinned })
|
||||||
return item.itemId !== group.pinnedItems[index].itemId
|
}}
|
||||||
})
|
/>
|
||||||
updateGroup(group, { pinnedItems: newPinned })
|
|
||||||
}}
|
|
||||||
>
|
|
||||||
<div className="absolute top-0 left-0 right-0 bottom-0 bg-gray-200 bg-opacity-50">
|
|
||||||
<XCircleIcon className="h-12 w-12 text-gray-600" />
|
|
||||||
</div>
|
|
||||||
</button>
|
|
||||||
</div>
|
|
||||||
)}
|
)}
|
||||||
</div>
|
</div>
|
||||||
))}
|
))}
|
||||||
|
@ -370,3 +362,17 @@ export function GroupAbout(props: {
|
||||||
</>
|
</>
|
||||||
)
|
)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
function CrossIcon(props: { onClick: () => void }) {
|
||||||
|
const { onClick } = props
|
||||||
|
|
||||||
|
return (
|
||||||
|
<div>
|
||||||
|
<button className=" text-gray-500 hover:text-gray-700" onClick={onClick}>
|
||||||
|
<div className="absolute top-0 left-0 right-0 bottom-0 bg-gray-200 bg-opacity-50">
|
||||||
|
<XCircleIcon className="h-12 w-12 text-gray-600" />
|
||||||
|
</div>
|
||||||
|
</button>
|
||||||
|
</div>
|
||||||
|
)
|
||||||
|
}
|
||||||
|
|
Loading…
Reference in New Issue
Block a user