Skip to content

Conversation

@bbovenzi
Copy link
Contributor

Addressing an issue raised in #30337 (comment) coming from #29971

The default zoom in was way too much and disconnected vertical elements were not visible by default.
This PR adds a max zoom and also makes sure that the default zoom stays the same when expanding/collapsing groups:

Before After
Screenshot 2023-03-29 at 9 48 52 AM Screenshot 2023-03-29 at 9 49 06 AM
before after

^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

@bbovenzi bbovenzi added the type:bug-fix Changelog: Bug Fixes label Mar 29, 2023
@bbovenzi bbovenzi added this to the Airflow 2.5.3 milestone Mar 29, 2023
@boring-cyborg boring-cyborg bot added area:UI Related to UI/UX. For Frontend Developers. area:webserver Webserver related Issues labels Mar 29, 2023
@pierrejeambrun pierrejeambrun added changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) and removed type:bug-fix Changelog: Bug Fixes labels Mar 29, 2023
@pierrejeambrun pierrejeambrun merged commit 8e0be06 into apache:main Mar 29, 2023
@bbovenzi bbovenzi deleted the fix-graph-max-zoom branch March 29, 2023 17:38
pierrejeambrun pushed a commit that referenced this pull request Mar 29, 2023
(cherry picked from commit 8e0be06)
@pierrejeambrun pierrejeambrun added type:bug-fix Changelog: Bug Fixes and removed changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) labels Mar 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:UI Related to UI/UX. For Frontend Developers. area:webserver Webserver related Issues type:bug-fix Changelog: Bug Fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants