[GitHub] [airflow] bbovenzi edited a comment on pull request #22272: Add map_index support to al task instance-related views

Wed, 16 Mar 2022 10:16:37 -0700 


bbovenzi edited a comment on pull request #22272:
URL: https://github.com/apache/airflow/pull/22272#issuecomment-1069354329


   <img width="563" alt="Screen Shot 2022-03-16 at 12 57 03 PM" 
src="https://user-images.githubusercontent.com/4600967/158645377-3d3502d8-8f8a-4532-b72e-240542aa1bde.png";>
   
   I think it would be useful to show the map index in the header when it isn't 
`-1`
   ```
     // task_instance.html
       {% if map_index and map_index != -1 %}
         <span class="text-muted">Map Index:</span> <span>{{ map_index }}</span>
       {% endif %}
   ```
   
     or the task instance modal (I think this only applies to a modal triggered 
from the gantt view right now):
     ```
     // dag.html
     <span id="show_map_index"><br><span class="text-muted">Map Index:</span> 
<span id="map_index"></span></span>
     ```
     and 
   ```
   // dag.js
     if (mi >= 0) {
       $('#show_map_index').show();
       $('#map_index').text(mi);
     } else {
       $('#show_map_index').hide();
     }
   ```


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