Skip to content

feat: add eye icon tip to empty result messages #5061

Merged
aglinxinyuan merged 2 commits into
apache:mainfrom
xuang7:feat/add-eye-icon-tip
May 14, 2026
Merged

feat: add eye icon tip to empty result messages #5061
aglinxinyuan merged 2 commits into
apache:mainfrom
xuang7:feat/add-eye-icon-tip

Conversation

@xuang7
Copy link
Copy Markdown
Contributor

@xuang7 xuang7 commented May 14, 2026

What changes were proposed in this PR?

This PR adds a tip about the eye icon when an operator shows no results, guiding users to enable materialization and execute the workflow to view results.

tip

Any related issues, documentation, discussions?

Closes #4500

How was this PR tested?

Manually tested.

Was this PR authored or co-authored using generative AI tooling?

No

@github-actions github-actions Bot added feature frontend Changes related to the frontend GUI labels May 14, 2026
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 42.84%. Comparing base (09828ea) to head (ca79422).

Files with missing lines Patch % Lines
...component/result-panel/result-panel.component.html 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##               main    #5061   +/-   ##
=========================================
  Coverage     42.84%   42.84%           
  Complexity     2204     2204           
=========================================
  Files          1045     1045           
  Lines         40144    40146    +2     
  Branches       4240     4240           
=========================================
+ Hits          17198    17199    +1     
- Misses        21879    21880    +1     
  Partials       1067     1067           
Flag Coverage Δ *Carryforward flag
access-control-service 39.53% <ø> (ø) Carriedforward from 09828ea
agent-service 33.72% <ø> (ø) Carriedforward from 09828ea
amber 43.71% <ø> (ø) Carriedforward from 09828ea
computing-unit-managing-service 0.00% <ø> (ø) Carriedforward from 09828ea
config-service 0.00% <ø> (ø) Carriedforward from 09828ea
file-service 32.18% <ø> (ø) Carriedforward from 09828ea
frontend 33.93% <50.00%> (+<0.01%) ⬆️
python 88.99% <ø> (ø) Carriedforward from 09828ea
workflow-compiling-service 47.72% <ø> (ø) Carriedforward from 09828ea

*This pull request uses carry forward flags. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@aglinxinyuan aglinxinyuan merged commit 6538e20 into apache:main May 14, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature frontend Changes related to the frontend GUI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improve result panel behavior for intermediate operators

3 participants