nextcloud / nextcloud/ios

Images are partially obscured on iPhone and iPad

Open
#3,650 1 comment 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

bug
Dominant language
Swift
Stars
2.5k
Forks
1k
Avg merge
2d 18h
Merged PRs (30d)
13

Description

How to use GitHub
  • Please use the 👍 reaction to show that you are affected by the same issue.
  • Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
  • Subscribe to receive notifications on status change and new comments.

Steps to reproduce
iPhone
  1. Use an iPhone with Dynamic Island (might also possibly affect models with notch)
  2. Open an image file that covers the entire length of the phone
  3. Watch as the Dynamic Island partially covers the image
iPad
  1. Open an image file that covers the entire length of the tablet
  2. Watch as the "Files Favorites Media Activity More" header partially covers the image
Expected behaviour
iPhone

Client app should account for the Dynamic Island for images that are high/wide enough to partially render beneat it and slightly downscale the image. This does not need to happen when zooming in on the image, however.

iPad

Client app should hide the header when image files (or any other type of files) are open. If I want to switch between Files, Favorites, Media, Activity, or More, I can close the file first and return to the folder view.

Actual behaviour

Image files are partially obscured by the Dynamic Island on iPhones that have one. They are also partially obscured by the header on iPads. This can obscure certain details at the top end of the image, and it is impossible to scroll up, as this will immediately close the image file.

Logs

No logs.

Reasoning or why should it be changed/implemented?

This partially obscures the top-center of an image. This can be bad depending on what type of image it is:

  • Schematics: can cover text, making it hard to read
  • Concept art: can cover certain details
  • Photos: can be distracting, especially on iPad due to the size of the header
Environment data

iOS version: 18.5

Nextcloud iOS app version: 7.0.6.1

Server operating system: nextcloud:latest ("Linux 4.4.302+ x86_64" according to Nextcloud server)

Web server: Apache (according to Docker repo)

Database: mariadb:latest ("mysql 11.8.2" according to Nextcloud server)

PHP version: 8.3.23 according to Nextcloud server

Nextcloud version: "Nextcloud Hub 10 (31.0.7)"

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

The issue names no source files or tests. Start by reproducing the image-viewing behavior on an iPhone with Dynamic Island and an iPad, then trace the client app's image viewer and surrounding navigation UI. Done means images are not obscured by the Dynamic Island or iPad header, while zooming behavior remains acceptable.

Written by the indexing model from the issue text.

Assessment

Tech stack
swift
Domain
mobile-dev
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.