xuang7 opened a new pull request, #3818:
URL: https://github.com/apache/texera/pull/3818
### **Purpose**
This PR fixes #3804 that the upload status panel behaved unexpectedly: when
there were no queued/active uploads, the UI still rendered empty panels, which
was confusing. This PR hides empty panels and restores the clear empty state.
### **Changes**
- Introduce a flag ` hasAnyActivity = queuedCount > 0 || activeCount > 0 ||
pendingChangesCount > 0 `
- Conditionally render status panels:
- Pending only when queuedCount > 0
- Uploading only when activeCount > 0
- Finished only when hasAnyActivity (and default-expand when
pendingChangesCount > 0)
- Restore the empty state: when no activity, render
`<texera-dataset-staged-objects-list>` outside the collapse so “No pending
changes” is visible
- Add a bottom divider beneath the staged list to improve visual separation
(the previous `[nzBorder]` was removed to avoid overlapping with the vertical
divider)
### **Demonstration**
**Datasets page:**
<img width="1315" height="870" alt="main"
src="https://github.com/user-attachments/assets/9112999c-24bc-4076-b139-eb3b405c2288";
/>
**Finished panel:**
| Collapsed | Expanded (delete) | Expanded (adds) |
|---|---|---|
| <img width="260"
src="https://github.com/user-attachments/assets/58fe8ba4-2b13-45a6-b049-318b288ec37e";
alt="collapsed" /> | <img width="260"
src="https://github.com/user-attachments/assets/21dada5c-0e39-4a9f-b1ee-ab0ede345aca";
alt="delete" /> | <img width="260"
src="https://github.com/user-attachments/assets/d3a25bea-bee3-4bb7-8023-b407741b01fe";
alt="expand" /> |
**Uploading files:**
https://github.com/user-attachments/assets/413f9480-330e-456a-8a0a-7f87511fbf13
**Remove files:**
https://github.com/user-attachments/assets/04fc0dc0-4f59-4e18-bb1d-72b0132ac943
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
