Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

Header does not fit on phones #1043

Closed
1 task
dhruvkb opened this issue Mar 3, 2022 · 4 comments · Fixed by #1257
Closed
1 task

Header does not fit on phones #1043

dhruvkb opened this issue Mar 3, 2022 · 4 comments · Fixed by #1257
Labels
🕹 aspect: interface Concerns end-users' experience with the software 🛠 goal: fix Bug fix 🟧 priority: high Stalls work on the project or its dependents

Comments

@dhruvkb
Copy link
Member

dhruvkb commented Mar 3, 2022

Description

The end-side of the header is cut off on smaller widths such as phones.

Reproduction

  1. Visit Openverse from a smaller width device.
  2. See error.

Expectation

The Search bar could possibly shrink to accommodate the content.

Screenshots

IMG_5935

Environment

  • Device: iPhone Xr
  • OS: iOS 15.3
  • Browser: Safari
  • Version: 15.3
  • Other info: viewport width = 414px

Resolution

  • 🙋 I would be interested in resolving this bug.
@dhruvkb dhruvkb added 🟧 priority: high Stalls work on the project or its dependents 🛠 goal: fix Bug fix 🕹 aspect: interface Concerns end-users' experience with the software labels Mar 3, 2022
@obulat
Copy link
Contributor

obulat commented Mar 3, 2022

Is this new? Which version of the front-end is this?
Just for context, when I open the currently deployed site at wordpress.org/openverse in Firefox or Chrome on my Android phone, I don't have a problem with the header not fitting.

@zackkrida
Copy link
Member

@obulat I believe it referrers to the header after scrolling on the search results (which only seems to change on ssr-rendered pages, interesting bug). I'm able to view this on android chrome + ff.

@dhruvkb
Copy link
Member Author

dhruvkb commented Mar 3, 2022

The iPhone I used has just 414px of width so that might be one of the reasons for the header getting cropped. Android phones with high resolution screens might not have this problem.

@sarayourfriend
Copy link
Contributor

Is this and #845 related?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
🕹 aspect: interface Concerns end-users' experience with the software 🛠 goal: fix Bug fix 🟧 priority: high Stalls work on the project or its dependents
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants