feature/modal-logo-mobile #8

Merged

patillacode merged 22 commits from feature/modal-logo-mobile into main 2026-04-21 22:42:37 +02:00

Conversation 0 Commits 22 Files changed 15 +576 -93

patillacode commented 2026-04-21 17:46:25 +02:00

Owner

Copy link

Replaces the native browser confirm() dialog with a reusable modal for entry deletion. If the entry is
published, the modal warns the user and automatically unpublishes it (via DELETE /share) before deleting.

Adds the alpha logo image next to the Piruetas wordmark in the header (image-only on mobile to save space).

Mobile toolbar cleanup: publish, copy-link, and delete actions move out of the fixed bottom toolbar into the
header nav, with proper touch targets (44px min-height), a lifted save toast, and danger styling on the delete
button.

Replaces the native browser confirm() dialog with a reusable modal for entry deletion. If the entry is published, the modal warns the user and automatically unpublishes it (via `DELETE /share`) before deleting. Adds the alpha logo image next to the **Piruetas** wordmark in the header (image-only on mobile to save space). Mobile toolbar cleanup: publish, copy-link, and delete actions move out of the fixed bottom toolbar into the header nav, with proper touch targets (`44px min-height`), a lifted save toast, and danger styling on the delete button.

patillacode added 12 commits 2026-04-21 17:46:25 +02:00

feat: add Playwright E2E infrastructure (fixtures, deps, justfile) ec285925d0

feat: add Playwright E2E infrastructure (fixtures, deps, justfile) cf0f4b62e2

feat(e2e): add auth tests (login, wrong password, redirect, logout, rate limit) cfa84bffcb

feat(e2e): add auth tests (login, wrong password, redirect, logout, rate limit) c7653dfa4b

feat(e2e): add journal tests (dashboard, CRUD, images, share token) f27ac5acb6

feat(e2e): add journal tests (dashboard, CRUD, images, share token) 71f92bcd18

feat(e2e): add account tests (page load, password change success/failure) 0fb5718968

feat(e2e): add account tests (page load, password change success/failure) 860db4c182

feat(e2e): add admin tests (dashboard, create/delete user, reset password, access control) d933c7bdcb

feat(e2e): add admin tests (dashboard, create/delete user, reset password, access control) dd095edc9a

feat(ci): add Playwright E2E job to Forgejo workflow eb89b418e7

feat: delete modal, alpha logo, and mobile UX improvements ... b1654efaa4

- Replace native confirm() with a reusable modal for entry deletion;
  conditionally warns when the entry is published and explicitly
  unpublishes (DELETE /share) before deleting the entry
- Add alpha logo image next to the Piruetas wordmark in the header;
  image-only on mobile to preserve header space
- Move publish, copy-link, and delete out of the fixed bottom toolbar
  on mobile into the header nav; increase touch targets to 44px min-height;
  lift save toast above the toolbar; add danger tint to delete button

patillacode added 1 commit 2026-04-21 21:12:06 +02:00

chore: resolve merge conflicts with main ... 67f2c62aa2

Merge in calendar legend strings, sidebar stats strings, entryExists
tracking, and updateDeleteBtn from main while preserving modal delete,
alpha logo, and mobile UX changes.

patillacode added 1 commit 2026-04-21 21:34:07 +02:00

chore: formatting and minor copy changes 0491b58adb

patillacode added 1 commit 2026-04-21 21:36:02 +02:00

fix: hide mobile copy-link button when entry is not published f7f2201930

patillacode added 4 commits 2026-04-21 22:02:02 +02:00

docs: add mobile bottom sheet navigation spec 8ce99f5360

docs: add mobile bottom sheet implementation plan 76fe2a8e0e

feat(mobile): add bottom sheet CSS and update header visibility rules 2581278917

feat(mobile): add bottom sheet HTML and menu trigger to layout 7e815aaf06

patillacode added 1 commit 2026-04-21 22:04:59 +02:00

feat(mobile): add bottom sheet nav for journal actions ... e79678d55a

- Simplify journal nav_links to publish-btn-mobile only
- Add mobile_sheet_items block with copy-link and delete sheet items
- Add setupMobileSheet() to ui.js: open/close animation, theme toggle, Escape key
- Wire sheet-copy-link and sheet-delete-btn in editor.js
- Remove duplicate setupDelete() call

patillacode force-pushed feature/modal-logo-mobile from e79678d55a to 3ac7f4b489 2026-04-21 22:06:03 +02:00 Compare

patillacode added 1 commit 2026-04-21 22:10:56 +02:00

fix(mobile): remove inline style blocking sign-out hide on mobile e7b34eb432

patillacode added 1 commit 2026-04-21 22:13:15 +02:00

fix(mobile): move delete entry to bottom of sheet, just before sign out 1f3d5ad6a6

patillacode added 1 commit 2026-04-21 22:16:06 +02:00

fix(mobile): add account link to bottom sheet 24283e1908

patillacode added 1 commit 2026-04-21 22:36:14 +02:00

chore: resolve merge conflicts with main 04b8fdfaf4

patillacode merged commit 90d48208b5 into main 2026-04-21 22:42:37 +02:00

patillacode deleted branch feature/modal-logo-mobile 2026-04-21 22:42:37 +02:00

patillacode referenced this pull request from a commit 2026-04-21 22:42:37 +02:00

feature/modal-logo-mobile (#8)

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

patillacode/piruetas!8

No description provided.