Skip to content

Commit db6f960

Browse files
committed
Add mobile menu todo and adjust spacing/theme variables for better layout
1 parent 16434bc commit db6f960

File tree

4 files changed

+7
-3
lines changed

4 files changed

+7
-3
lines changed

HUMAN-TODO.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
# todo
22

3+
- [ ] add mobile menu for PWA that shows hamburger menu. instead of topnav bar.
34
- [ ] add terms and privacy and refund policy pages
45
- [ ] make sure all payment pages reference USDC too, its missing on register page
56
- [ ] use new cryptapi.io api endpoints

public/css/layout.css

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
width: 100%;
99
max-width: 1200px;
1010
margin: 0 auto;
11-
padding: 0 var(--spacing-md);
11+
padding: 0 var(--spacing-md-lg);
1212
}
1313

1414
.container-sm {
@@ -314,7 +314,7 @@ body {
314314
display: flex;
315315
flex-direction: column;
316316
min-height: 100vh;
317-
padding: 0 var(--spacing-md);
317+
padding: 0 var(--spacing-md-lg);
318318
}
319319

320320
#app {

public/css/theme.css

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@
5252
--spacing-xs: 4px; /* Extra small spacing */
5353
--spacing-sm: 8px; /* Small spacing */
5454
--spacing-md: 16px; /* Medium spacing */
55+
--spacing-md-lg: 1.2rem; /* Medium-large spacing (19.2px at default font size) */
5556
--spacing-lg: 24px; /* Large spacing */
5657
--spacing-xl: 32px; /* Extra large spacing */
5758
--spacing-xxl: 48px; /* Double extra large spacing */

public/js/components/pf-header.js

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,10 +41,12 @@ class PfHeader extends HTMLElement {
4141
justify-content: center;
4242
cursor: pointer;
4343
transition: background-color 0.2s;
44+
color: var(--text-primary, #111827); /* Add explicit color for default state */
4445
}
4546
4647
.theme-toggle:hover {
47-
background-color: #f0f0f0;
48+
background-color: var(--surface-variant, #f3f4f6); /* Use theme-appropriate background */
49+
color: var(--text-primary, #111827); /* Maintain text color on hover */
4850
}
4951
5052
.theme-toggle svg {

0 commit comments

Comments
 (0)