Skip to content

Commit

Permalink
🐛 Fix top-bar flex sizing for mobile"
Browse files Browse the repository at this point in the history
Fix a small UI issue for mobile/tablet viewports that caused the top-bar to be smaller than it should and the pagination controls on some pages hidden out of frame
  • Loading branch information
aaronleopold committed Apr 14, 2024
1 parent b286339 commit e03442b
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 4 deletions.
2 changes: 1 addition & 1 deletion packages/browser/src/components/media/BookSearch.tsx
Expand Up @@ -68,7 +68,7 @@ export default function BookSearch({ page, page_size, setPage, onBookSelect, sho
{...(showFilters ? { entity: 'media', orderBy: true } : {})}
/>

<div className="flex w-full flex-col space-y-6 pt-4">
<div className="flex w-full flex-col space-y-6 pb-[64px] pt-4 md:pb-0">
{hasStuff && (
<Pagination
pages={total_pages}
Expand Down
Expand Up @@ -12,12 +12,12 @@ export const TOPBAR_HEIGHT_PX = 53
*/
export default function TopBar() {
return (
<header className="sticky top-0 z-10 flex h-[50px] w-full border-b border-edge-200 bg-sidebar px-4 md:hidden">
<header className="sticky top-0 z-10 flex h-14 w-full shrink-0 border-b border-edge-200 bg-sidebar px-4 md:hidden">
<div className="grid w-full grid-cols-8 items-center gap-2">
<div className="col-span-1">
<MobileSheet />
</div>
<div className="col-span-6 flex items-center justify-center gap-2">
<div className="col-span-6 flex h-full items-center justify-center gap-2">
<Link to="/" className="flex shrink-0 items-center justify-start gap-2">
<img src="/assets/favicon.ico" className="h-6 w-6 object-scale-down" />
<Heading variant="gradient" size="xs">
Expand Down
1 change: 1 addition & 0 deletions packages/browser/src/scenes/home/HomeScene.tsx
Expand Up @@ -38,6 +38,7 @@ export default function HomeScene() {
<ContinueReadingMedia />
<RecentlyAddedMedia />
<RecentlyAddedSeries />
<div className="pb-5 sm:pb-0" />
</SceneContainer>
)
}
2 changes: 1 addition & 1 deletion packages/browser/src/scenes/series/SeriesBooksScene.tsx
Expand Up @@ -91,7 +91,7 @@ function SeriesOverviewScene() {
orderBy
/>

<div className="flex w-full flex-col gap-y-6">
<div className="flex w-full flex-col gap-y-6 pb-[64px] md:pb-0">
{hasStuff && (
<Pagination pages={total_pages} currentPage={current_page} onChangePage={setPage} />
)}
Expand Down

0 comments on commit e03442b

Please sign in to comment.