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

Sorting explanation appears even on dashboard without sorting function #528

Closed
ds-jleyh opened this issue Dec 21, 2023 · 2 comments · Fixed by catenax-ng/tx-traceability-foss#1024
Assignees
Labels
bug Something isn't working frontend Frontend related Issues

Comments

@ds-jleyh
Copy link
Contributor

ds-jleyh commented Dec 21, 2023

Hints / Details

Prerequisites

Most of tables in the FE are sortable but not the quality-investigations and -alerts tables on the dashboard. Mouseover on column names of sortable tables leads to appearance of an information field with descriptions for sorting. This appears even on the dashboard however no sorting functionality is given.

Steps to reproduce:

  1. Open dashboard.
  2. Scroll down to the quality-investigations or -alerts table.
  3. Hold the mouse over e.g. the column name "Description".

Current behavior:

Sorting informations appear for mouseover on column names on dashboard:
image

Expected behavior:

No information about column sorting appear if the desired column is not sortable .

Testing

Environment

Results

  • Tooltip does not appear on any of the columns of dashboard tables without sorting support
  • LGFM
@ds-jleyh ds-jleyh added the frontend Frontend related Issues label Dec 21, 2023
@github-project-automation github-project-automation bot moved this to inbox in Trace-X Dec 21, 2023
@jzbmw jzbmw changed the title [FE] Sorting explanation appears even on dashboard without sorting function Sorting explanation appears even on dashboard without sorting function Jan 8, 2024
@jzbmw jzbmw added the bug Something isn't working label Jan 8, 2024
@mkanal mkanal moved this from inbox to backlog in Trace-X Feb 27, 2024
@ds-mmaul ds-mmaul moved this from backlog to next in Trace-X Mar 1, 2024
@ds-mmaul ds-mmaul self-assigned this Mar 1, 2024
@ds-mmaul ds-mmaul moved this from next to wip in Trace-X Mar 1, 2024
@ds-mmaul ds-mmaul moved this from wip to review in Trace-X Mar 4, 2024
@ds-mmaul
Copy link
Contributor

ds-mmaul commented Mar 4, 2024

  • Disabled the tooltip on dashboard, merged in main, testable on e2e

@mkanal
Copy link
Contributor

mkanal commented Mar 4, 2024

@jzbmw @ds-mmaul
Tested LGFM ready for PO acceptance

@ds-mmaul ds-mmaul assigned jzbmw and unassigned mkanal Mar 4, 2024
@jzbmw jzbmw moved this from review to done in Trace-X Mar 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working frontend Frontend related Issues
Projects
Status: done
Development

Successfully merging a pull request may close this issue.

4 participants