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

Report details page layout on tablet (🅼4) #85

Open
Tracked by #76
schlos opened this issue Aug 10, 2021 · 4 comments
Open
Tracked by #76

Report details page layout on tablet (🅼4) #85

schlos opened this issue Aug 10, 2021 · 4 comments
Labels
🕷️ bug Something isn't working help wanted Extra attention is needed html/css

Comments

@schlos
Copy link
Member

schlos commented Aug 10, 2021

Describe the bug

Report details page on tablet looks awkward:

  • images overlap on map
  • some page content is hidden behind the map

To Reproduce
Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Expected behavior

As per Figma.

Screenshots

Screenshot_20210810-224735_Opera Touch.jpg

Screenshot_20210810-224749_Opera Touch.jpg

Figma (expected):

Screenshot_20210812-172202_Brave.jpg

Desktop (please complete the following information):

  • OS: [e.g. Windows]
  • Browser [e.g. chrome, safari]
  • Version [e.g. 22]

Smartphone (please complete the following information):

  • Device: [e.g. iPhone6] Samsung Galaxy Tab S5e
  • OS: [e.g. iOS8.1]
  • Browser [e.g. stock browser, safari]
  • Version [e.g. 22]

Additional context

@schlos schlos added the 🕷️ bug Something isn't working label Aug 10, 2021
@schlos schlos changed the title popravito: Report details page layout on tablet popravito: Report details page layout on tablet (🅼4) Aug 12, 2021
@schlos
Copy link
Member Author

schlos commented Aug 12, 2021

For comparison, this is how Report details screen looks with different theme cobrand:

Screenshot_20210812-165757_Opera Touch.jpg

Screenshot_20210812-165811_Opera Touch.jpg

Screenshot_20210812-165821_Opera Touch.jpg

Screenshot_20210812-165828_Opera Touch.jpg

@schlos
Copy link
Member Author

schlos commented Aug 23, 2021

Partially fixed with the latest PR:

  • Looks better, although text "Sent to Grad Zagreb 4 minutes later" is still partially covered by map (on tablet):

photo_2021-08-23 22 39 42

photo_2021-08-23 22 39 48

Just for info, this is current look on desktop:
https://snippyly.com/img/?q=FMp7kN397GTz63gcPFr4 (&EDIT=JK2BMiw1yvLnrC2ydYuN)

@schlos schlos added html/css help wanted Extra attention is needed labels Nov 11, 2021
@schlos schlos changed the title popravito: Report details page layout on tablet (🅼4) Report details page layout on tablet (🅼4) Nov 26, 2021
@schlos
Copy link
Member Author

schlos commented Nov 26, 2021

How to contribute?
Read our contribution guidelines at https://codeforcroatia.org/popravito-contribution-guidelines.

@schlos
Copy link
Member Author

schlos commented Nov 26, 2021

Example of current PROD page with the problem:
https://popravi.to/report/12

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🕷️ bug Something isn't working help wanted Extra attention is needed html/css
Projects
None yet
Development

No branches or pull requests

1 participant