Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat: add Apple touch icons for iOS/iPadOS home screen support #2488

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Oct 10, 2025
Merged

feat: add Apple touch icons for iOS/iPadOS home screen support #2488

merged 1 commit into from
Oct 10, 2025

Conversation

@HarshMN2345
Copy link
Member

@HarshMN2345 HarshMN2345 commented Oct 10, 2025 •
edited by coderabbitai bot
Loading

What does this PR do?

(Provide a description of what this PR does.)

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

Summary by CodeRabbit

  • New Features
    • Added Apple-specific web app integration on iOS.
    • High-resolution app icons for Home Screen (multiple sizes for different devices).
    • Supports launching as a standalone web app when added to Home Screen.
    • Customized status bar appearance in standalone mode.
    • Displays a dedicated app title on Home Screen and within the web app.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 10, 2025 •
edited
Loading

Walkthrough

Updates src/app.html by adding Apple-specific web app metadata in the head element. Introduces four apple-touch-icon link tags for sizes 180x180, 167x167, 152x152, and 120x120. Adds meta tags: apple-mobile-web-app-capable, apple-mobile-web-app-status-bar-style, and apple-mobile-web-app-title. No changes to scripts, logic, or control flow—only static head metadata additions.

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title clearly and concisely summarizes the primary change by indicating that Apple touch icons are being added to support iOS and iPadOS home screens, which directly reflects the head metadata additions in src/app.html. It avoids unnecessary detail or ambiguity and uses straightforward phrasing consistent with the repository’s commit conventions.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat-SER-373-add-apple-metadata-icons
📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f63a91d and e75d4d8.

⛔ Files ignored due to path filters (4)
  • static/images/logos/icons/apple-touch-icon-120x120.png is excluded by !**/*.png
  • static/images/logos/icons/apple-touch-icon-152x152.png is excluded by !**/*.png
  • static/images/logos/icons/apple-touch-icon-167x167.png is excluded by !**/*.png
  • static/images/logos/icons/apple-touch-icon-180x180.png is excluded by !**/*.png
📒 Files selected for processing (1)
  • src/app.html (1 hunks)
🔇 Additional comments (1)
src/app.html (1)

10-34: Confirm apple-touch icon assets exist

Please double-check that all four referenced PNGs are present in the deployed static assets; if any are missing iOS will show a blank icon on the home screen.

You can verify quickly with:

Warning

Review ran into problems

🔥 Problems

Git: Failed to clone repository. Please run the @coderabbitai full review command to re-trigger a full review. If the issue persists, set path_filters to include or exclude specific files.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@ItzNotABug ItzNotABug merged commit d5a3d56 into main Oct 10, 2025
4 checks passed
@ItzNotABug ItzNotABug deleted the feat-SER-373-add-apple-metadata-icons branch October 10, 2025 12:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ItzNotABug ItzNotABug ItzNotABug approved these changes

@atharvadeosthale atharvadeosthale Awaiting requested review from atharvadeosthale

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@HarshMN2345 @ItzNotABug