Skip to content
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.

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

Det: Issues Filtering Listings on Mobile #1641

Closed
iph-97 opened this issue Oct 2, 2023 · 3 comments
Closed

Det: Issues Filtering Listings on Mobile #1641

iph-97 opened this issue Oct 2, 2023 · 3 comments

Comments

@iph-97
Copy link

iph-97 commented Oct 2, 2023

A city staff person reported they were unable to scroll on the filter window on mobile when browsing listings. They said, "I was trying to look up a property to see if it was listed on Detroit home connect and I was actually unable to apply the filters in the "find rentals for you" section. The problem is that the "done" and "clear" buttons are below the screen on my phone, and I cannot scroll up to see them and click on them at the same time. I think we need a little more space below those buttons or something." I was able to replicate this issue and took this screen recording for a more clear illustration. This is a low priority issue.
[Uploading... RPReplay_Final1696253842.MP4]

@github-actions github-actions bot added this to Needs triage in Backlog-Sprint Oct 2, 2023
@github-actions github-actions bot added this to Triage in Milestones Oct 2, 2023
@ludtkemorgan
Copy link
Collaborator

Initial findings:

  • This is only happening on phones.
  • We are not using the "actions" section of the drawer which is meant to have the buttons stay on the bottom of the screen. I don't know if that was intentional or not.
  • There was some minor overrides added to this when pulling in the ui-c component

@sarahlazarich This might take a bit of investigating, but I think it's a good ticket to pull into this sprint if somebody is looking for a small bit of work

@emilyjablonski
Copy link
Collaborator

emilyjablonski commented Oct 2, 2023

QA Notes
This needs to be tested on a mobile phone. You can check out the deploy preview link and ensure you can scroll down to the buttons.

@emilyjablonski emilyjablonski self-assigned this Oct 4, 2023
@sarahlazarich
Copy link
Collaborator

lgtm!

Backlog-Sprint automation moved this from Needs triage to Closed Oct 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

4 participants