Skip to content

Feature: add view image page#1389

Merged
reloxx13 merged 2 commits intoPhotoboothProject:devfrom
reloxx13:feature/add-view-image-page
Jan 6, 2026
Merged

Feature: add view image page#1389
reloxx13 merged 2 commits intoPhotoboothProject:devfrom
reloxx13:feature/add-view-image-page

Conversation

@reloxx13
Copy link
Copy Markdown
Collaborator

@reloxx13 reloxx13 commented Jan 4, 2026

Prerequisites checklist

What is the purpose of this pull request? (put an "x" next to an item)

  • Documentation update
  • Bug fix
  • New feature
  • Other, please explain:

What changes did you make? (Give an overview)

  • Added a touch-friendly viewer (view.php) for single photo/video display, tailored for QR codes and direct downloads.
  • Clarified QR/download usage in the docs (FAQ link to viewer vs. direct api/download.php).
grafik

Is there anything you'd like reviewers to focus on?

@reloxx13 reloxx13 force-pushed the feature/add-view-image-page branch from 7d30e34 to 598c157 Compare January 4, 2026 10:31
@andi34 andi34 changed the title Feature/add view image page Feature; add view image page Jan 5, 2026
@andi34 andi34 changed the title Feature; add view image page Feature: add view image page Jan 5, 2026
Copy link
Copy Markdown
Contributor

@andi34 andi34 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me and seems to work fine. Only found a bug on the qr generation, Http:// vs http:// inside the url definition causing a wrong url. Independent of this PR.

@andi34
Copy link
Copy Markdown
Contributor

andi34 commented Jan 5, 2026

#1391

@reloxx13 reloxx13 force-pushed the feature/add-view-image-page branch from 598c157 to de3db48 Compare January 6, 2026 08:47
@reloxx13 reloxx13 merged commit 8741edf into PhotoboothProject:dev Jan 6, 2026
9 checks passed
@reloxx13 reloxx13 deleted the feature/add-view-image-page branch January 7, 2026 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants