fortrangoingonforty/sniffly / e0ac0fd

Browse files

strip leading slashes from breadcrumb segment names

Authored by espadonne
SHA
e0ac0fdcff277303f7ac921506cabfcce01f6e00
Parents
d35c822
Tree
1a288c9

2 changed files

StatusFile+-
M breadcrumb/STATUS.md 16 8
M src/gui/breadcrumb_widget.f90 18 4
breadcrumb/STATUS.mdmodified
@@ -57,18 +57,26 @@ Already implemented:
5757
5858
 ---
5959
60
-## Phase 5: Integration - IN PROGRESS
60
+## Phase 5: Integration ✅ COMPLETE
6161
62
-**TODO:**
63
-- [ ] Update `meson.build` to include breadcrumb_widget.f90
64
-- [ ] Replace old breadcrumb in `gtk_app.f90`
65
-- [ ] Remove old breadcrumb functions
66
-- [ ] Wire up navigation callback
67
-- [ ] Test build
62
+**Completed:**
63
+- ✅ Updated `meson.build` to include breadcrumb_widget.f90
64
+- ✅ Replaced old breadcrumb in `gtk_app.f90`
65
+- ✅ Removed old breadcrumb functions (sniffly_update_breadcrumbs, on_breadcrumb_clicked)
66
+- ✅ Wired up navigation callback
67
+- ✅ Test build SUCCESSFUL
68
+
69
+**Build status:** CLEAN (only unused parameter warnings)
70
+
71
+---
72
+
73
+## Phase 6: Back/Forward Awareness - OPTIONAL (can skip)
74
+
75
+This is an advanced feature from the plan. The basic breadcrumb works without it.
6876
6977
 ---
7078
71
-## Phase 6: Back/Forward Awareness - PENDING
79
+## Phase 7: Testing - IN PROGRESS
7280
7381
 **TODO:**
7482
 - [ ] Add history checking to draw function
src/gui/breadcrumb_widget.f90modified
@@ -192,8 +192,15 @@ contains
192192
         else
193193
           cached_segment_paths(cached_segment_count) = trim(working_path)
194194
         end if
195
-        ! Extract just the name
196
-        cached_segment_names(cached_segment_count) = trim(working_path(start_pos:))
195
+        ! Extract just the name (remove any leading/trailing slashes)
196
+        cached_segment_names(cached_segment_count) = trim(adjustl(working_path(start_pos:)))
197
+        ! Remove leading slash if present
198
+        if (len_trim(cached_segment_names(cached_segment_count)) > 0) then
199
+          if (cached_segment_names(cached_segment_count)(1:1) == "/") then
200
+            cached_segment_names(cached_segment_count) = &
201
+              trim(cached_segment_names(cached_segment_count)(2:))
202
+          end if
203
+        end if
197204
         print *, "  Segment ", cached_segment_count, ": ", &
198205
                  trim(cached_segment_paths(cached_segment_count)), " -> ", &
199206
                  trim(cached_segment_names(cached_segment_count))
@@ -208,8 +215,15 @@ contains
208215
         else
209216
           cached_segment_paths(cached_segment_count) = trim(working_path(1:start_pos+slash_pos-2))
210217
         end if
211
-        ! Extract just the name
212
-        cached_segment_names(cached_segment_count) = trim(working_path(start_pos:start_pos+slash_pos-2))
218
+        ! Extract just the name (remove any leading/trailing slashes)
219
+        cached_segment_names(cached_segment_count) = trim(adjustl(working_path(start_pos:start_pos+slash_pos-2)))
220
+        ! Remove leading slash if present
221
+        if (len_trim(cached_segment_names(cached_segment_count)) > 0) then
222
+          if (cached_segment_names(cached_segment_count)(1:1) == "/") then
223
+            cached_segment_names(cached_segment_count) = &
224
+              trim(cached_segment_names(cached_segment_count)(2:))
225
+          end if
226
+        end if
213227
         print *, "  Segment ", cached_segment_count, ": ", &
214228
                  trim(cached_segment_paths(cached_segment_count)), " -> ", &
215229
                  trim(cached_segment_names(cached_segment_count))