Skip to content

Fullscreen button#2647

Closed
RrOoSsSsOo wants to merge 2 commits intobooklore-app:developfrom
RrOoSsSsOo:fullscreen-button
Closed

Fullscreen button#2647
RrOoSsSsOo wants to merge 2 commits intobooklore-app:developfrom
RrOoSsSsOo:fullscreen-button

Conversation

@RrOoSsSsOo
Copy link
Copy Markdown

🚀 Pull Request

📝 Description

Add a fullscreen button to have, in browser tab or in PWA mode, the fullscreen.

🛠️ Changes Implemented

  • Add the fullscreen button and his code.

🧪 Testing Strategy

  • Run ./gradlew test
  • Create a custom Docker Image that I run in selfhosted

📸 Visual Changes (if applicable)

image

⚠️ Required Pre-Submission Checklist

  • Code adheres to project style guidelines and conventions
  • Branch synchronized with latest develop branch (please resolve any merge conflicts)
  • 🚨 CRITICAL: Automated unit tests added/updated to cover changes (MANDATORY for ALL Spring Boot backend and Angular frontend changes - this is non-negotiable)
  • 🚨 CRITICAL: All tests pass locally (run ./gradlew test for Spring Boot backend, and ng test for Angular frontend - NO EXCEPTIONS)
  • 🚨 CRITICAL: Manual testing completed in local development environment (verify your changes work AND no existing functionality is broken - test related features thoroughly)
  • Flyway migration versioning follows correct sequence (if database schema was modified)
  • Documentation PR submitted to booklore-docs (required for features or enhancements that introduce user-facing or visual changes)

💬 Additional Context (optional)

@acx10 acx10 closed this Apr 29, 2026
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