Procházet zdrojové kódy

Fix mobile sidebar scroll, lint errors, and add tests for (#724)

  - Fix mobile sidebar not scrollable when nav items exceed viewport
maziggy před 5 měsíci
rodič
revize
7ab9b3e39f

+ 2 - 0
CHANGELOG.md

@@ -9,6 +9,8 @@ All notable changes to Bambuddy will be documented in this file.
 
 
 ### Fixed
 ### Fixed
 - **Webhook Notifications Missing Camera Snapshot** ([#679](https://github.com/maziggy/bambuddy/issues/679)) — Webhook notification providers did not include camera snapshots (e.g. from First Layer Complete notifications), even though providers like Telegram, Pushover, ntfy, and Discord already attached them. The webhook payload now includes a base64-encoded `image` field when a snapshot is available (generic format only, not Slack format). Reported by @Arn0uDz.
 - **Webhook Notifications Missing Camera Snapshot** ([#679](https://github.com/maziggy/bambuddy/issues/679)) — Webhook notification providers did not include camera snapshots (e.g. from First Layer Complete notifications), even though providers like Telegram, Pushover, ntfy, and Discord already attached them. The webhook payload now includes a base64-encoded `image` field when a snapshot is available (generic format only, not Slack format). Reported by @Arn0uDz.
+- **Mobile Sidebar Not Scrollable** — On mobile devices with many navigation items, the sidebar did not scroll, making bottom items unreachable. Added overflow scrolling to the nav section while keeping the logo and footer pinned.
+- **User Notification Ruff/Lint Fixes** ([#693](https://github.com/maziggy/bambuddy/pull/693)) — Fixed missing `timezone` import in email timestamp, unused lambda argument, PEP 8 blank line spacing for `mark_printer_stopped_by_user`, and SQLAlchemy forward reference in `UserEmailPreference` model.
 
 
 ### Changed
 ### Changed
 
 

+ 1 - 1
frontend/src/components/Layout.tsx

@@ -487,7 +487,7 @@ export function Layout() {
         </div>
         </div>
 
 
         {/* Navigation */}
         {/* Navigation */}
-        <nav className="flex-1 p-2">
+        <nav className="flex-1 p-2 overflow-y-auto">
           <ul className="space-y-2">
           <ul className="space-y-2">
             {orderedSidebarIds.map((id) => {
             {orderedSidebarIds.map((id) => {
               const isExternal = isExternalLinkId(id);
               const isExternal = isExternalLinkId(id);

Rozdílová data souboru nebyla zobrazena, protože soubor je příliš velký
+ 0 - 0
static/assets/index-KP2gJRqC.js


+ 1 - 1
static/index.html

@@ -23,7 +23,7 @@
 
 
     <!-- Splash screens for iOS -->
     <!-- Splash screens for iOS -->
     <link rel="apple-touch-startup-image" href="/img/android-chrome-512x512.png" />
     <link rel="apple-touch-startup-image" href="/img/android-chrome-512x512.png" />
-    <script type="module" crossorigin src="/assets/index-DKZ25Iex.js"></script>
+    <script type="module" crossorigin src="/assets/index-KP2gJRqC.js"></script>
     <link rel="stylesheet" crossorigin href="/assets/index-Dq3gZWq6.css">
     <link rel="stylesheet" crossorigin href="/assets/index-Dq3gZWq6.css">
   </head>
   </head>
   <body>
   <body>

Některé soubory nejsou zobrazeny, neboť je v těchto rozdílových datech změněno mnoho souborů