Skip to content

chore: small ui fixes for better mobile viewing #23

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

Merged
zduey merged 1 commit into from
Apr 18, 2026
Merged

chore: small ui fixes for better mobile viewing #23

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
22 changes: 11 additions & 11 deletions apps/kitchen_mate/frontend/src/App.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -144,18 +144,18 @@ function HomePage() {
// Authenticated users see their recipe collection
return (
<div>
<div className="flex items-center gap-6 mb-6">
<div className="shrink-0">
<h2 className="text-xl font-semibold text-brown-dark">My Recipes</h2>
<p className="text-brown-medium">{getSubtitle()}</p>
<div className="mb-6 space-y-3">
<div className="flex justify-between items-center">
<div>
<h2 className="text-xl font-semibold text-brown-dark">My Recipes</h2>
<p className="text-brown-medium">{getSubtitle()}</p>
</div>
<div className="flex items-center gap-3 shrink-0">
<ViewModeToggle mode={viewMode} onChange={handleViewChange} />
<AddRecipeDropdown variant="plus" />
</div>
</div>
<div className="flex-1 max-w-md mx-auto">
<SearchInput value={searchQuery} onChange={handleSearchChange} />
</div>
<div className="flex items-center gap-6 shrink-0">
<ViewModeToggle mode={viewMode} onChange={handleViewChange} />
</div>
<AddRecipeDropdown variant="plus" />
<SearchInput value={searchQuery} onChange={handleSearchChange} />
</div>

{(tagFilter || searchQuery) && (
Expand Down
67 changes: 44 additions & 23 deletions apps/kitchen_mate/frontend/src/components/Header.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -44,9 +44,9 @@ export function Header({ onSignInClick }: HeaderProps) {

return (
<header className="bg-white shadow-sm">
<div className="max-w-5xl mx-auto px-4 py-4 flex justify-between items-center">
<div className="flex items-center gap-8">
{/* Logo */}
<div className="max-w-5xl mx-auto px-4 py-3 flex flex-col sm:flex-row sm:justify-between sm:items-center gap-2">
{/* Row 1 (mobile): Logo + Auth */}
<div className="flex justify-between items-center sm:gap-8">
<Link to="/" className="flex items-center gap-2 hover:opacity-80 transition-opacity">
<BookmarkIcon className="w-5 h-6" />
<span className="font-serif text-2xl text-brown-dark tracking-tight">
Expand All @@ -55,8 +55,29 @@ export function Header({ onSignInClick }: HeaderProps) {
</span>
</Link>

{/* Navigation */}
<nav className="flex items-center gap-6">
{/* Auth — visible on mobile inline with logo */}
<div className="flex items-center gap-3 sm:hidden">
{isAuthEnabled && !loading && (
<>
{user ? (
<UserDropdown user={user} onSignOut={handleSignOut} />
) : (
<button
onClick={onSignInClick}
title="Save recipes, build collections, and more"
className="px-3 py-1.5 text-sm text-coral hover:bg-coral hover:bg-opacity-10 rounded-lg transition-colors"
>
Sign In
</button>
)}
</>
)}
</div>
</div>

{/* Row 2 (mobile): Nav + Auth (desktop) */}
<div className="flex items-center justify-between sm:justify-end sm:gap-6">
<nav className="flex items-center gap-4">
{isAuthorized ? (
<Link
to="/"
Expand All @@ -77,25 +98,25 @@ export function Header({ onSignInClick }: HeaderProps) {

<AddRecipeDropdown variant="button" />
</nav>
</div>

{/* Auth */}
<div className="flex items-center gap-4">
{isAuthEnabled && !loading && (
<>
{user ? (
<UserDropdown user={user} onSignOut={handleSignOut} />
) : (
<button
onClick={onSignInClick}
title="Save recipes, build collections, and more"
className="px-4 py-2 text-sm text-coral hover:bg-coral hover:bg-opacity-10 rounded-lg transition-colors"
>
Sign In
</button>
)}
</>
)}
{/* Auth — visible on desktop only */}
<div className="hidden sm:flex items-center gap-4">
{isAuthEnabled && !loading && (
<>
{user ? (
<UserDropdown user={user} onSignOut={handleSignOut} />
) : (
<button
onClick={onSignInClick}
title="Save recipes, build collections, and more"
className="px-4 py-2 text-sm text-coral hover:bg-coral hover:bg-opacity-10 rounded-lg transition-colors"
>
Sign In
</button>
)}
</>
)}
</div>
</div>
</div>
</header>
Expand Down
100 changes: 49 additions & 51 deletions apps/kitchen_mate/frontend/src/components/SavedRecipeView.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -272,59 +272,57 @@ export function SavedRecipeView() {

<div className="p-6">
{/* Header with title and actions */}
<div className="flex justify-between items-start mb-4">
<div className="flex-1">
<h2 className="text-2xl font-bold text-brown-dark">{recipe.title}</h2>

{userRecipe.is_modified && (
<span className="inline-block mt-2 px-2 py-1 bg-coral bg-opacity-10 text-coral-dark text-xs rounded">
Modified from original
</span>
)}
</div>

<div className="flex gap-2 ml-4">
{id && <ShareButton recipeId={id} />}
{id && <ShareToKitchenButton recipeId={id} />}
<button
onClick={() => setIsEditing(true)}
className="p-2 text-brown-medium hover:text-coral hover:bg-coral hover:bg-opacity-10 rounded"
title="Edit recipe"
>
<svg
className="h-5 w-5"
fill="none"
stroke="currentColor"
viewBox="0 0 24 24"
<div className="mb-4">
<div className="flex justify-between items-start gap-2">
<h2 className="text-2xl font-bold text-brown-dark leading-tight">{recipe.title}</h2>
<div className="flex gap-2 shrink-0">
{id && <ShareButton recipeId={id} />}
{id && <ShareToKitchenButton recipeId={id} />}
<button
onClick={() => setIsEditing(true)}
className="p-2 text-brown-medium hover:text-coral hover:bg-coral hover:bg-opacity-10 rounded"
title="Edit recipe"
>
<path
strokeLinecap="round"
strokeLinejoin="round"
strokeWidth={2}
d="M11 5H6a2 2 0 00-2 2v11a2 2 0 002 2h11a2 2 0 002-2v-5m-1.414-9.414a2 2 0 112.828 2.828L11.828 15H9v-2.828l8.586-8.586z"
/>
</svg>
</button>
<button
onClick={() => setShowDeleteConfirm(true)}
className="p-2 text-brown-medium hover:text-red-600 hover:bg-red-50 rounded"
title="Delete recipe"
>
<svg
className="h-5 w-5"
fill="none"
stroke="currentColor"
viewBox="0 0 24 24"
<svg
className="h-5 w-5"
fill="none"
stroke="currentColor"
viewBox="0 0 24 24"
>
<path
strokeLinecap="round"
strokeLinejoin="round"
strokeWidth={2}
d="M11 5H6a2 2 0 00-2 2v11a2 2 0 002 2h11a2 2 0 002-2v-5m-1.414-9.414a2 2 0 112.828 2.828L11.828 15H9v-2.828l8.586-8.586z"
/>
</svg>
</button>
<button
onClick={() => setShowDeleteConfirm(true)}
className="p-2 text-brown-medium hover:text-red-600 hover:bg-red-50 rounded"
title="Delete recipe"
>
<path
strokeLinecap="round"
strokeLinejoin="round"
strokeWidth={2}
d="M19 7l-.867 12.142A2 2 0 0116.138 21H7.862a2 2 0 01-1.995-1.858L5 7m5 4v6m4-6v6m1-10V4a1 1 0 00-1-1h-4a1 1 0 00-1 1v3M4 7h16"
/>
</svg>
</button>
<svg
className="h-5 w-5"
fill="none"
stroke="currentColor"
viewBox="0 0 24 24"
>
<path
strokeLinecap="round"
strokeLinejoin="round"
strokeWidth={2}
d="M19 7l-.867 12.142A2 2 0 0116.138 21H7.862a2 2 0 01-1.995-1.858L5 7m5 4v6m4-6v6m1-10V4a1 1 0 00-1-1h-4a1 1 0 00-1 1v3M4 7h16"
/>
</svg>
</button>
</div>
</div>
{userRecipe.is_modified && (
<span className="inline-block mt-2 px-2 py-1 bg-coral bg-opacity-10 text-coral-dark text-xs rounded">
Modified from original
</span>
)}
</div>

{/* Source URL */}
Expand Down Expand Up @@ -468,7 +466,7 @@ export function SavedRecipeView() {
</div>

{/* Footer */}
<div className="flex justify-between items-center pt-4 border-t border-gray-200">
<div className="flex flex-col sm:flex-row sm:justify-between sm:items-center gap-3 pt-4 border-t border-gray-200">
<Link
to="/"
className="text-coral hover:text-coral-dark flex items-center"
Expand Down
Loading