Skip to content
This repository has been archived by the owner on Jan 12, 2023. It is now read-only.

Bug 1908745: Add Help dropdown with About modal and documentation link #484

Merged
merged 1 commit into from Mar 26, 2021

Conversation

mturley
Copy link
Collaborator

@mturley mturley commented Mar 26, 2021

https://bugzilla.redhat.com/show_bug.cgi?id=1908745

Adds a help icon to the left of the brand logo on the right side of the header. Clicking it opens a dropdown with options for "Documentation" (a link to the MTV product docs) and "About". Clicking About opens a modal showing branding and various version numbers.

The Forklift-branded version uses the full Forklift logo from https://github.com/konveyor/community/tree/main/brand/logo in place of a brand icon and text product name. The MTV-branded version uses a small Red Hat icon and the product name in text.

Forklift-branded About modal

Screen Shot 2021-03-26 at 3 46 10 PM

Screen Shot 2021-03-26 at 3 46 19 PM

Screen Shot 2021-03-26 at 3 46 27 PM

MTV-branded About modal

Screen Shot 2021-03-26 at 3 34 00 PM

Screen Shot 2021-03-26 at 3 34 07 PM

Screen Shot 2021-03-26 at 3 41 48 PM

@mturley mturley requested a review from a team March 26, 2021 19:55
@github-actions
Copy link

valid bug 1908745

@konveyor-preview-bot
Copy link

🚀 Deployed Preview: http://konveyor-forklift-ui-pr-484-preview.surge.sh

Compare with current main branch: http://konveyor-forklift-ui-preview.surge.sh

@codecov
Copy link

codecov bot commented Mar 26, 2021

Codecov Report

Merging #484 (ad80e7d) into main (ddbd8ab) will increase coverage by 0.05%.
The diff coverage is 66.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #484      +/-   ##
==========================================
+ Coverage   60.20%   60.26%   +0.05%     
==========================================
  Files         131      133       +2     
  Lines        4227     4263      +36     
  Branches     1046     1050       +4     
==========================================
+ Hits         2545     2569      +24     
- Misses       1660     1672      +12     
  Partials       22       22              
Impacted Files Coverage Δ
src/app/AppLayout/ForkliftAboutModal.tsx 56.25% <56.25%> (ø)
src/app/AppLayout/HelpDropdown.tsx 72.22% <72.22%> (ø)
src/app/AppLayout/AppLayout.tsx 95.23% <100.00%> (+0.23%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ddbd8ab...ad80e7d. Read the comment docs.

@mturley mturley requested review from a user and gildub March 26, 2021 19:59
Copy link
Contributor

@gildub gildub left a comment

Choose a reason for hiding this comment

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

LGTM

@mturley
Copy link
Collaborator Author

mturley commented Apr 5, 2021

Note: this was part 1 of #470, there will be a followup PR to close that issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants