nexB / vulntotal-extension

0 stars 1 forks source link

Extension frontend #5

Open michaelehab opened 2 weeks ago

michaelehab commented 2 weeks ago

Fix #2

Updates

I added Frontend files using Bulma CSS as we discussed in a previous meeting.
The frontend design is very close to the VulnerableCode website.
I added the Full-Screen button as we discussed in a previous meeting.
I extracted a real JSON response from VulnTotal, corresponding to "pkg:pypi/jinja2@2.4.1" purl, and used the response to show how the extension should behave with real data.
Screenshots

Popup

Full Screen

home page settings page results table

OmkarPh commented 1 week ago

UI is looking good !

Minor nits - Change font color on hovering links, so that it looks like

instead of

OmkarPh commented 1 week ago

On clicking settings button, it opens a new tab (regardless of whether I'm in full screen or smaller window)

If i do, settings -> back to home -> settings -> back to home This results unnecessary tabs, we should ideally open settings in the same tab?

michaelehab commented 4 days ago

I agree! I updated the hover effect for the navigation links. I also updated the links behaviors, making sure we only open a new tab if the active tab does not belong to VulnTotal.

michaelehab commented 5 hours ago

@OmkarPh Please review and accept the PR if it looks good, we need to merge it.

nexB / vulntotal-extension

Extension frontend #5

Updates

Screenshots

Popup

Full Screen