rachanadutta commented on PR #60591:
URL: https://github.com/apache/airflow/pull/60591#issuecomment-3756082020

   
   <img width="1280" height="822" alt="Screenshot 2026-01-15 at 6 08 28 PM" 
src="https://github.com/user-attachments/assets/8fbd3ce3-6d5a-4962-b054-b48b5776e92f";
 />
   
   Hi @VedantMadane , thanks for the PR — the implementation looks solid 👍
   
   I was actually working on this issue as well (I’m the assignee), and my 
approach was slightly different on the UI side.
   
   Instead of placing the button in the mapped task header, I was adding the 
“Clear all mapped task instances” button next to the Filter button in the Task 
Instances section, to keep it consistent with other bulk actions on that page.
   
   I’ve attached a screenshot below showing the placement I was testing locally.
   
   Functionality-wise, your approach of passing only task_id (without 
map_index) matches what I had in mind, and we could reuse the same dialog logic.
   
   @maintainers — would you prefer the button in the header (as in this PR), or 
alongside the filter in the Task Instances section?
   
   Based on your guidance, I’m happy to either:
   
   follow up with a small PR adjusting the placement, or
   
   help refine this PR instead.
   
   Thanks!


-- 
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]

Reply via email to