Skip to content

Update table for new source data#2152

Open
edlu77 wants to merge 13 commits intoftu-layout-updatesfrom
source-list-table
Open

Update table for new source data#2152
edlu77 wants to merge 13 commits intoftu-layout-updatesfrom
source-list-table

Conversation

@edlu77 edlu77 self-assigned this Feb 17, 2026
@nx-cloud
Copy link

nx-cloud bot commented Feb 17, 2026

🤖 Nx Cloud AI Fix Eligible

An automatically generated fix could have helped fix failing tasks for this run, but Self-healing CI is disabled for this workspace. Visit workspace settings to enable it and get automatic fixes in future runs.

To disable these notifications, a workspace admin can disable them in workspace settings.


View your CI Pipeline Execution ↗ for commit 692fbf7

Command Status Duration Result
nx affected --targets=lint,test,compodoc --conf... ❌ Failed 57s View ↗
nx affected --target=build,build-webcomponent,b... ✅ Succeeded 1m 45s View ↗

☁️ Nx Cloud last updated this comment at 2026-02-26 00:18:22 UTC

@github-actions
Copy link

github-actions bot commented Feb 17, 2026

🚀 Preview Deploy Report

✅ Successfully deployed preview here

@edlu77 edlu77 requested review from LibbyUX and axdanbol February 20, 2026 22:23
Copy link
Contributor

@LibbyUX LibbyUX left a comment

Choose a reason for hiding this comment

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

@edlu77 - Thanks for the opportunity to review. We are getting closer! I added cosmetic app & web component feedback unrelated to the source data table here: #2155

Global

  • All menu supporting text should be bumped down to label small 12px: This is present on both the landing page, app, and small wc

App landing page preview

  • The landing page is also showing unspecified scrollbar arrows in 4 different corners that are not related to the actual page scrollbar - can these be removed?
  • Both sections are missing the background imagery
  • Is it possible for both sections to take up 100% of the viewport and to have a parallax effect on both background images?

App & small web component

  • Info indicator should be bumped down to label medium 14px
  • Is it possible to remove the arrow marks and swap out with our scrollbar components? There is an awkward space where the dividers and comonents end and this unspecified scrollbar begins
  • Can we please edit Health Status to read Health status in the table header?
Image Image Image

@LibbyUX
Copy link
Contributor

LibbyUX commented Feb 23, 2026

@edlu77 - I am reviewing on Brave to see the disabled additional scrollbar arrows on the landing page:

image

@LibbyUX LibbyUX self-requested a review February 26, 2026 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

apps: ftu-ui-small-wc Functional Tissue Unit Explorer small web component apps.humanatlas.io/ftu-explorer ftu-ui: Functional Tissue Unit Explorer

Projects

None yet

2 participants