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.

Sign up for GitHub

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

Jump to bottom

Big white space in the modal of content report #4802

Closed
fcoveram opened this issue Aug 22, 2024 · 0 comments · Fixed by #4800
Closed

Big white space in the modal of content report #4802

fcoveram opened this issue Aug 22, 2024 · 0 comments · Fixed by #4800
Assignees
@obulat
Labels
💻 aspect: code Concerns the software code in the repository 🕹 aspect: interface Concerns end-users' experience with the software 🛠 goal: fix Bug fix 🟨 priority: medium Not blocking but should be addressed soon 🧱 stack: frontend Related to the Nuxt frontend

Comments

@fcoveram
Copy link
Contributor

Description

There is an unnecessary white space in the top side of the content area in the recent implementation of the report content as modal (#4710 to solve #3582).

Solution: the spacing should be zero.

Screenshots

Page Page with inspector
Screenshot of content report modal Screenshot of content report modal with arrow pointing the padding top of content area
@fcoveram fcoveram added 🚦 status: awaiting triage Has not been triaged & therefore, not ready for work 🛠 goal: fix Bug fix 🧱 stack: frontend Related to the Nuxt frontend labels Aug 22, 2024
@dhruvkb dhruvkb added help wanted Open to participation from the community 🟨 priority: medium Not blocking but should be addressed soon 💻 aspect: code Concerns the software code in the repository 🕹 aspect: interface Concerns end-users' experience with the software and removed 🚦 status: awaiting triage Has not been triaged & therefore, not ready for work labels Aug 23, 2024
@obulat obulat removed the help wanted Open to participation from the community label Aug 23, 2024
@obulat obulat self-assigned this Aug 23, 2024
@obulat obulat linked a pull request Aug 23, 2024 that will close this issue
Use a modal for license explanation #4800
Merged
8 tasks
@obulat obulat mentioned this issue Aug 23, 2024
Merged
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@obulat obulat

Labels
💻 aspect: code Concerns the software code in the repository 🕹 aspect: interface Concerns end-users' experience with the software 🛠 goal: fix Bug fix 🟨 priority: medium Not blocking but should be addressed soon 🧱 stack: frontend Related to the Nuxt frontend
Projects
Openverse Backlog
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use a modal for license explanation WordPress/openverse
3 participants
@fcoveram @obulat @dhruvkb