codeforcroatia / popravi.to

This is popular map-based reporting platform FixMyStreet for Croatia named "Popravi.to"
http://popravi.to
Other
9 stars 7 forks source link

Add missing report metadata on Report Details screen #179

Open schlos opened 2 years ago

schlos commented 2 years ago

Describe the bug

Missing report metadata on Report Details screen as per screenshots.

To Reproduce Steps to reproduce the behavior:

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

Expected behavior

Add missing report metadata on Report Details screen.

Screenshots

Current: Screenshot from 2021-12-02 08-07-50

Comparison with a different theme: photo_2021-12-02_08-06-04

Expected: Figma design screen - https://www.figma.com/proto/OrvZfg5BsfdOzeL7nhRpEI/Popravi.to?node-id=3171%3A11518&scaling=min-zoom&page-id=25%3A186&hide-ui=1

Desktop (please complete the following information):

Smartphone (please complete the following information):

Additional context

Contribution guidelines: https://codeforcroatia.org/popravito-contribution-guidelines

schlos commented 2 years ago

Temporary added. It still needs to be displayed as per Figma.

znmustic commented 1 year ago

Pozdrav, ovo preuzimam na sebe @schlos .

schlos commented 1 year ago

Odlično, hvala!

Piši ako trebaš dodatno informacija.

schlos commented 1 year ago

As discussed after change done via pull request #192, we need to display moderator view only when moderator is logged in (to be done via https://github.com/codeforcroatia/popravi.to/issues/193).

Fix for this issue is done - design is applied.

Bug found during the testing that needs fixing:

Screenshot from 2023-06-12 10-31-14

schlos commented 1 year ago

@znmustic no need for you to do anything. I'll take a look how to apply rules for user anonymity.

schlos commented 1 year ago

Noticed that Report Title is also missing (i.e. "Rupa na cesti" in the Figma design screen) and "Ažuriranja" section text.

Screenshot: https://app.snippyly.com/discuss/?q=E9rxGmdrwtoydD2348jP &EDIT=mfmrn40YrbzdFhpbNyXV