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.

Sign up for GitHub

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

Small tweaks on Subs & Posts #1968

Merged
merged 2 commits into from
Dec 13, 2023
Merged

Small tweaks on Subs & Posts #1968

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
ecad04e
clients/maintainerpage: fix bug with subscribers count in sidebar
emilwidlund Dec 13, 2023
748dc09
clients/posts: add sticky sidebar to posts page
emilwidlund Dec 13, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions clients/apps/web/src/app/maintainer/[organization]/(topbar)/posts/ClientPage.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ const ClientPage = () => {
</div>
<div className="flex flex-col gap-y-12">
{(posts.data?.items?.length ?? 0) > 0 ? (
<StaggerReveal className="flex w-full flex-col gap-y-6">
<StaggerReveal className="flex w-full flex-col gap-y-4">
{posts?.data?.items
? posts.data.items.map((post) => (
<StaggerReveal.Child key={post.id}>
Expand All @@ -90,7 +90,7 @@ const ClientPage = () => {
)}
</div>
</div>
<div className="flex flex-shrink-0 flex-col gap-y-8 xl:w-1/3">
<div className="sticky top-8 flex flex-shrink-0 flex-col gap-y-8 xl:w-1/3">
<div className="flex w-full flex-grow flex-row items-center justify-between">
<h3 className="dark:text-polar-50 text-lg font-medium text-gray-950">
Analytics
Expand Down
6 changes: 3 additions & 3 deletions clients/apps/web/src/components/Organization/OrganizationPublicSidebar.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ export const OrganizationPublicSidebar = ({
)

const subscribersHiddenCount = useMemo(
() => subscriptionSummary.slice(9).length,
() => subscribersCount - subscribers.length,
[subscriptionSummary],
)

Expand Down Expand Up @@ -222,8 +222,8 @@ export const OrganizationPublicSidebar = ({
</React.Fragment>
))}
{subscribersHiddenCount > 0 && (
<div className="dark:border-polar-600 dark:text-polar-500 flex h-10 w-10 flex-col items-center justify-center rounded-full border border-blue-200 text-xs font-medium text-blue-400">
{subscribersHiddenCount}
<div className="dark:border-polar-700 dark:bg-polar-900 dark:text-polar-400 flex h-10 w-10 flex-col items-center justify-center rounded-full border-2 border-blue-200 text-xs font-medium text-blue-400">
+{subscribersHiddenCount}
</div>
)}
</div>
Expand Down
Loading