feat(per-page-persistence): remember selected rows per page in localStorage
table-list component now saves per_page selection to localStorage and restores it when navigating back to any list without explicit per_page in URL. URL parameter takes priority over stored value to ensure bookmarked links work predictably. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
124
.paul/phases/72-per-page-persistence/72-01-PLAN.md
Normal file
124
.paul/phases/72-per-page-persistence/72-01-PLAN.md
Normal file
@@ -0,0 +1,124 @@
|
||||
---
|
||||
phase: 72-per-page-persistence
|
||||
plan: 01
|
||||
type: execute
|
||||
wave: 1
|
||||
depends_on: []
|
||||
files_modified: [resources/views/components/table-list.php]
|
||||
autonomous: true
|
||||
---
|
||||
|
||||
<objective>
|
||||
## Goal
|
||||
Zapamiętywanie wybranej liczby wierszy na stronie (per_page) w localStorage przeglądarki na liście zamówień i innych listach korzystających z komponentu table-list.
|
||||
|
||||
## Purpose
|
||||
Użytkownik zmienia per_page na 50 lub 100, ale po powrocie na listę wartość resetuje się do domyślnej 20. Zmiana powinna być trwała w danej przeglądarce.
|
||||
|
||||
## Output
|
||||
Zmodyfikowany `resources/views/components/table-list.php` — zapis per_page do localStorage przy zmianie i odczyt przy ładowaniu strony.
|
||||
</objective>
|
||||
|
||||
<context>
|
||||
## Source Files
|
||||
@resources/views/components/table-list.php (linie 311-325 per-page selector, 329-432 localStorage, 428-432 change handler)
|
||||
</context>
|
||||
|
||||
<acceptance_criteria>
|
||||
|
||||
## AC-1: Zapis per_page do localStorage
|
||||
```gherkin
|
||||
Given użytkownik jest na liście zamówień z per_page=20
|
||||
When zmieni per_page na 50
|
||||
Then wartość 50 zostaje zapisana w localStorage pod kluczem tableList_[basePath]_[listKey]_per_page
|
||||
```
|
||||
|
||||
## AC-2: Odczyt per_page z localStorage
|
||||
```gherkin
|
||||
Given użytkownik wcześniej wybrał per_page=50 (zapisane w localStorage)
|
||||
When wchodzi na listę zamówień bez parametru per_page w URL
|
||||
Then formularz per_page automatycznie ustawia 50 i strona przeładowuje się z per_page=50
|
||||
```
|
||||
|
||||
## AC-3: Parametr URL ma priorytet nad localStorage
|
||||
```gherkin
|
||||
Given użytkownik ma per_page=50 w localStorage
|
||||
When wchodzi na listę z jawnym per_page=20 w URL
|
||||
Then wyświetla 20 rekordów (URL ma priorytet) i localStorage zostaje zaktualizowane na 20
|
||||
```
|
||||
|
||||
## AC-4: Brak pętli przeładowań
|
||||
```gherkin
|
||||
Given użytkownik ma per_page=50 w localStorage i wchodzi na stronę
|
||||
When strona przeładowuje się z per_page=50
|
||||
Then nie następuje kolejne przeładowanie (brak nieskończonej pętli)
|
||||
```
|
||||
|
||||
</acceptance_criteria>
|
||||
|
||||
<tasks>
|
||||
|
||||
<task type="auto">
|
||||
<name>Task 1: Dodanie persistence per_page w table-list.php</name>
|
||||
<files>resources/views/components/table-list.php</files>
|
||||
<action>
|
||||
1. Dodać klucz localStorage: `var perPageKey = storagePrefix + '_per_page';` (obok istniejących filterKey, colsKey, queryKey ~linia 336)
|
||||
|
||||
2. W handlerze change selecta per_page (~linia 428-432): przed `form.submit()` zapisać wybraną wartość do localStorage:
|
||||
`writeJson(perPageKey, parseInt(select.value, 10));`
|
||||
|
||||
3. Na końcu sekcji DOMContentLoaded (po istniejących restore bloków), dodać logikę restore per_page:
|
||||
- Odczytać `readJson(perPageKey, null)`
|
||||
- Jeśli wartość istnieje i jest różna od aktualnie wybranej w selekcie per_page:
|
||||
- Sprawdzić czy per_page jest obecne w URL (URLSearchParams)
|
||||
- Jeśli per_page NIE jest w URL (użytkownik wszedł bez jawnego parametru):
|
||||
- Ustawić wartość w selekcie i submitować formularz (to spowoduje przeładowanie z prawidłowym per_page w URL)
|
||||
- Jeśli per_page JEST w URL: zaktualizować localStorage z wartości z URL (sync)
|
||||
- Jeśli wartość jest taka sama jak w selekcie: nic nie rób (brak pętli)
|
||||
|
||||
Wzorzec: użyć istniejących helperów readJson/writeJson.
|
||||
Unikać: nieskończonej pętli przeładowań — kluczowy warunek: nie submitować jeśli wartość w selekcie już odpowiada localStorage.
|
||||
</action>
|
||||
<verify>
|
||||
1. Otworzyć /orders/list — domyślnie 20
|
||||
2. Zmienić na 50 — strona przeładowuje z per_page=50
|
||||
3. Wejść na /orders/list (bez per_page w URL) — automatycznie przeładuje z per_page=50
|
||||
4. Wejść na /orders/list?per_page=20 — wyświetli 20 i zaktualizuje localStorage
|
||||
5. Sprawdzić brak pętli przeładowań
|
||||
</verify>
|
||||
<done>AC-1, AC-2, AC-3, AC-4 satisfied</done>
|
||||
</task>
|
||||
|
||||
</tasks>
|
||||
|
||||
<boundaries>
|
||||
|
||||
## DO NOT CHANGE
|
||||
- Backend PHP (OrdersController, OrdersRepository)
|
||||
- Struktura HTML per-page selectora
|
||||
- Inne localStorage keys (filters_open, hidden_cols, query)
|
||||
|
||||
## SCOPE LIMITS
|
||||
- Tylko frontend JS w table-list.php
|
||||
- Bez zmian backendu — per_page nadal przychodzi jako query param
|
||||
|
||||
</boundaries>
|
||||
|
||||
<verification>
|
||||
Before declaring plan complete:
|
||||
- [ ] Brak błędów JS w konsoli przeglądarki
|
||||
- [ ] Per_page zapisuje się w localStorage po zmianie
|
||||
- [ ] Per_page odtwarza się po wejściu bez parametru w URL
|
||||
- [ ] Brak nieskończonej pętli przeładowań
|
||||
- [ ] Inne listy (np. paragony) też korzystają z persistence
|
||||
</verification>
|
||||
|
||||
<success_criteria>
|
||||
- Wybrana liczba wierszy jest zapamiętywana w localStorage
|
||||
- Wejście na listę bez per_page w URL automatycznie stosuje zapamiętaną wartość
|
||||
- Brak regresji w istniejącej funkcjonalności table-list
|
||||
</success_criteria>
|
||||
|
||||
<output>
|
||||
After completion, create `.paul/phases/72-per-page-persistence/72-01-SUMMARY.md`
|
||||
</output>
|
||||
94
.paul/phases/72-per-page-persistence/72-01-SUMMARY.md
Normal file
94
.paul/phases/72-per-page-persistence/72-01-SUMMARY.md
Normal file
@@ -0,0 +1,94 @@
|
||||
---
|
||||
phase: 72-per-page-persistence
|
||||
plan: 01
|
||||
subsystem: ui
|
||||
tags: [localStorage, pagination, table-list, UX]
|
||||
|
||||
requires: []
|
||||
provides:
|
||||
- Per page persistence via localStorage for all table-list instances
|
||||
affects: []
|
||||
|
||||
tech-stack:
|
||||
added: []
|
||||
patterns: [per-page localStorage key in table-list component]
|
||||
|
||||
key-files:
|
||||
created: []
|
||||
modified: [resources/views/components/table-list.php]
|
||||
|
||||
key-decisions:
|
||||
- "Dedykowany klucz per_page zamiast polegania na query restore"
|
||||
- "URL per_page ma priorytet nad localStorage — sync a nie override"
|
||||
|
||||
patterns-established:
|
||||
- "perPageKey = storagePrefix + '_per_page' — nowy klucz w rodzinie tableList_ storage"
|
||||
|
||||
duration: 5min
|
||||
started: 2026-04-07T12:20:00Z
|
||||
completed: 2026-04-07T12:25:00Z
|
||||
---
|
||||
|
||||
# Phase 72 Plan 01: Per Page Persistence Summary
|
||||
|
||||
**Zapamiętywanie wybranej liczby wierszy w localStorage — per_page persist dla wszystkich list table-list.**
|
||||
|
||||
## Performance
|
||||
|
||||
| Metric | Value |
|
||||
|--------|-------|
|
||||
| Duration | ~5min |
|
||||
| Tasks | 1 completed |
|
||||
| Files modified | 1 |
|
||||
|
||||
## Acceptance Criteria Results
|
||||
|
||||
| Criterion | Status | Notes |
|
||||
|-----------|--------|-------|
|
||||
| AC-1: Zapis per_page do localStorage | Pass | writeJson przy change + sync z URL |
|
||||
| AC-2: Odczyt per_page z localStorage | Pass | Auto-submit gdy brak per_page w URL |
|
||||
| AC-3: URL ma priorytet nad localStorage | Pass | urlParams.has('per_page') → sync only |
|
||||
| AC-4: Brak pętli przeładowań | Pass | Po submit URL zawiera per_page → brak ponownego submit |
|
||||
|
||||
## Accomplishments
|
||||
|
||||
- Dodano `perPageKey` do rodziny kluczy localStorage w table-list.php
|
||||
- Zapis przy zmianie selecta per_page (przed submit)
|
||||
- Odczyt i auto-redirect przy wejściu bez per_page w URL
|
||||
- Sync localStorage z URL gdy per_page jawnie podane
|
||||
|
||||
## Files Created/Modified
|
||||
|
||||
| File | Change | Purpose |
|
||||
|------|--------|---------|
|
||||
| `resources/views/components/table-list.php` | Modified | perPageKey + save/restore logic |
|
||||
|
||||
## Decisions Made
|
||||
|
||||
| Decision | Rationale | Impact |
|
||||
|----------|-----------|--------|
|
||||
| Dedykowany klucz per_page | Query restore przywraca caly URL (status, page, search) — per_page powinien byc niezalezny | Mozna zmienic per_page bez resetowania filtrow |
|
||||
| URL ma priorytet | Jawny link z per_page=20 powinien dzialac przewidywalnie | Brak niespodzianek przy linkach bookmarkowanych |
|
||||
|
||||
## Deviations from Plan
|
||||
|
||||
None — plan executed exactly as written
|
||||
|
||||
## Issues Encountered
|
||||
|
||||
None
|
||||
|
||||
## Next Phase Readiness
|
||||
|
||||
**Ready:**
|
||||
- Wszystkie listy table-list automatycznie korzystaja z persistence per_page
|
||||
|
||||
**Concerns:**
|
||||
- None
|
||||
|
||||
**Blockers:**
|
||||
- None
|
||||
|
||||
---
|
||||
*Phase: 72-per-page-persistence, Plan: 01*
|
||||
*Completed: 2026-04-07*
|
||||
Reference in New Issue
Block a user