anushkagupta200615-jpg opened a new pull request, #68914:
URL: https://github.com/apache/airflow/pull/68914

    <!-- SPDX-License-Identifier: Apache-2.0
         https://www.apache.org/licenses/LICENSE-2.0 -->
   
   <!--
   Thank you for contributing!
   
   Please provide above a brief description of the changes made in this pull 
request.
   Write a good git commit message following this guide: 
http://chris.beams.io/posts/git-commit/
   
   Please make sure that your code changes are covered with tests.
   And in case of new features or big changes remember to adjust the 
documentation.
   
   For user-facing UI changes, please attach before/after screenshots (or a 
short
   screen recording) so reviewers can assess the visual impact.
   
   Feel free to ping (in general) for the review if you do not see reaction for 
a few days
   (72 Hours is the minimum reaction time you can expect from volunteers) - we 
sometimes miss notifications.
   
   In case of an existing issue, reference it using one of the following:
   
   * closes: #ISSUE
   * related: #ISSUE
   -->
   
   ---
   This PR addresses the user feedback regarding the layout in Airflow 3 where 
the Grid and Graph views were mutually exclusive in the left panel, causing 
users to lose context when navigating between them. 
   To improve the UX and keep the Grid visible at all times, we have 
implemented **Option A**: The Graph view has been decoupled from the left panel 
and moved to the right details panel as a permanent tab.
   **Changes made:**
   * Added a new `Graph` tab and routing for both the DAG-level 
(`dags/:dagId/graph`) and DAG Run-level (`dags/:dagId/runs/:runId/graph`).
   * Removed `Graph` from the left panel toggle (`dagViewOptions`) inside 
`PanelButtons.tsx`, meaning the main panel is now strictly reserved for Grid 
and Gantt views.
   * Removed all dead conditional rendering logic tied to `dagView === "graph"`.
   * Gracefully reset user's local storage `dagView` to `grid` if they 
previously had `graph` saved.
   * Extracted Graph-specific settings (Dependencies and Direction) from 
`PanelButtons.tsx` into a new `<GraphSettings />` popover, which is now 
rendered directly over the React Flow map in the `Graph` tab alongside 
`<GraphTaskFilters />`.
   closes: #67472
   
   
   ##### Was generative AI tooling used to co-author this PR?
   
   <!--
   If generative AI tooling has been used in the process of authoring this PR, 
please
   change below checkbox to `[X]` followed by the name of the tool, uncomment 
the "Generated-by".
   -->
   
   - [x] Yes (please specify the tool below)
   
   <!--
   Generated-by: [Tool Name] following [the 
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
   -->
   
   ---
   


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