-
-
Notifications
You must be signed in to change notification settings - Fork 7
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
Add missing report metadata on Report Details screen #179
Comments
Temporary added. |
Pozdrav, ovo preuzimam na sebe @schlos . |
Odlično, hvala! Piši ako trebaš dodatno informacija. |
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 #193). Fix for this issue is done - design is applied. Bug found during the testing that needs fixing:
|
@znmustic no need for you to do anything. I'll take a look how to apply rules for user anonymity. |
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 |
Describe the bug
Missing report metadata on Report Details screen as per screenshots.
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Add missing report metadata on Report Details screen.
Screenshots
Current:
Comparison with a different theme:
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
The text was updated successfully, but these errors were encountered: