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

Fix Overflow in Interrogation Services Page #21043

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

bupd
Copy link
Contributor

@bupd bupd commented Oct 14, 2024

Thank you for contributing to Harbor!

Comprehensive Summary of your change

fixes overflow in Interrogation Services -> Vulnerability Page.
The issue has been successfully resolved for screen resolutions of 1280x720 and higher.

Screenshots

Before

on screen 1280x720
before vuln ui fix

After

on screen 1280x720
after vulnerability fix

Issue being fixed

Fixes #20818

Please indicate you've done the following:

  • Well Written Title and Summary of the PR
  • Label the PR as needed. "release-note/ignore-for-release, release-note/new-feature, release-note/update, release-note/enhancement, release-note/community, release-note/breaking-change, release-note/docs, release-note/infra, release-note/deprecation"
  • Accepted the DCO. Commits without the DCO will delay acceptance.
  • Made sure tests are passing and test coverage is added if needed.
  • Considered the docs impact and opened a new docs issue or PR with docs changes if needed in website repository.

@bupd bupd requested a review from a team as a code owner October 14, 2024 17:40
Copy link

codecov bot commented Oct 15, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 66.09%. Comparing base (c8c11b4) to head (6f4796a).
Report is 299 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##             main   #21043       +/-   ##
===========================================
+ Coverage   45.36%   66.09%   +20.72%     
===========================================
  Files         244     1049      +805     
  Lines       13333   114645   +101312     
  Branches     2719     2867      +148     
===========================================
+ Hits         6049    75773    +69724     
- Misses       6983    34733    +27750     
- Partials      301     4139     +3838     
Flag Coverage Δ
unittests 66.09% <ø> (+20.72%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 1288 files with indirect coverage changes

@Vad1mo Vad1mo self-requested a review October 15, 2024 12:16
Copy link
Member

@Vad1mo Vad1mo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While you are already on it @bupd

having a fully vertical alignment (instead of a mix between both) is probably a better option.

Take a look 👇 what do you think?
376339491-628a4a3e-6837-41a0-858b-5b541e12c87c

@Vad1mo Vad1mo added the release-note/update Update or Fix label Oct 15, 2024
@bupd bupd force-pushed the vulnerability-ui-fix branch 2 times, most recently from 7e586cb to 1e2a9c8 Compare October 16, 2024 13:34
@Vad1mo
Copy link
Member

Vad1mo commented Oct 17, 2024

@bupd can you add yet another screenshot

@bupd
Copy link
Contributor Author

bupd commented Oct 17, 2024

Sure, @Vad1mo

Screenshot

fix ui 21043

* adds padding in between scan now and results of the scan

Signed-off-by: bupd <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Stop scan all UI issue
6 participants