konveyor / tackle2-ui

Tackle (2nd generation) UI component.
Apache License 2.0
8 stars 43 forks source link

Add the ability to individually view all files attached to a task on the Analysis details view #1929

Closed sjd78 closed 5 months ago

sjd78 commented 6 months ago

Summary

For an application, the activity of the current / most recently run analysis task:

The enhancement will make a few changes:

  1. The activity log view will allow viewing each attached file individually
  2. The activity log view will be moved a full page view

Enhancements reference:

Requirements

Viewing each file individually

Existing functionality to keep:

Allow viewing all of the task's attached files by adding a select box to the top of the viewer. The select box should:

Additional functionality:

View moved to a full page

The current view opens in a modal window. In order to have more screen space to view the logs and provide selection of attached files, move the view from the modal to a full page.

Some potential examples of the bread crumbs (including some potential future uses):

Screen shots and mockups

Current State

Open the view from the kebab menu: screenshot-localhost_9000-2024 05 31-10_04_57

Open the view from the application details drawer report tab: screenshot-localhost_9000-2024 05 31-10_06_28

View an analysis details (task activity log): screenshot-localhost_9000-2024 05 31-10_06_03

Future State Mockups

Attached file list box and on page view: analysis-details-on-page

Attached file list box opened to select a different file to view: analysis-details-select-expanded

Viewing an attached file: analysis-details-log

konveyor-ci-bot[bot] commented 6 months ago

This issue is currently awaiting triage. If contributors determine this is a relevant issue, they will accept it by applying the triage/accepted label and provide further guidance. The triage/accepted label can be added by org members.

sjd78 commented 6 months ago

@dymurray, @JustinXHale, @rromannissen -- for your consideration

JustinXHale commented 5 months ago

Full page looks great.

sjd78 commented 5 months ago

@jortel, @mansam -- also for your consideration (sorry about the late tags)

rszwajko commented 5 months ago

@sjd78 Regarding bread crumbs: note that according to PF guidelines the last segment (current page) is considered active and not styled as a link.