-
Notifications
You must be signed in to change notification settings - Fork 6
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
[6669] apps/contrib/item-detail: restyle moderator feedback, move status dis… #4818
Conversation
88581ea
to
5595e49
Compare
222d688
to
d9dbac8
Compare
@fuzzylogic2000 there is one task I am not following. So I kept the wording for the headlines for now.
|
@khamui I think, we can leave that out for now. I will add a task! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
NIce, looks good! Just one nit pick.
@khamui I guess with this not being your branch, you didn't see the answer. :) |
d9dbac8
to
eabb5cf
Compare
…ename moderator-info scss class to more general detail-info
eabb5cf
to
98a1607
Compare
@khamui Because of all the changing back and forth, I added everything in one commit. I think, it is fine like that! |
…play away from badges
This updates the labels, so that they are still all in the list, but the feedback one not in the detail view. It does also do the permissions and displays the box as we decided (only if there's content and the upper bit always to moderators). A proper styling is still missing, though:
think about the headline and its a11y for the actual feedback (we didn't want it twice, so what we call it in the form and so on is the headline to the new box and the organisation name and date are in the place of the headline now)I will keep this WIP and hope @khamui or @phillimorland can take over?