-
Notifications
You must be signed in to change notification settings - Fork 16.4k
Docs: refresh UI Guide screenshots for Airflow 3.1.x (dark/light) #55876
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Great job! There's just one screenshot that should be updated (however is easiest). |
|
@cmarteepants Fixed that last dag_list screenshot. I think its read to go now |
kaxil
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These blank lines are needed. Without it, the docs build fail:
https://github.com/apache/airflow/actions/runs/17862695877/job/50797945885?pr=55876
|
@kaxil I will handle all the above listed suggestions and recommit. |
|
@kaxil @cmarteepants I have made the suggested changes.Kindly look into them |
|
@kaxil I have replaced above listed 4 images (both light and dark). Please review it further |
|
@kaxil I replaced dag_run_graph and dag_overview_grid with better screenshots.Please review once again.Thanks! |
|
@ashb Ok , I will make the required changes and recommit |
|
@KatalKavya96 No, I'm afraid not. There are still unresolved comments (even before I just reviewed it and added some more) |
|
@KatalKavya96 Please can you go and review the side-by-side diff and see that they roughly match the old content before asking a review again. Thanks |
|
@ashb Ok , I will once go for a side by side comparison for all changed images and make sure that they are the nearest to the original image, Thanks for the Review |
providers/edge3/src/airflow/providers/edge3/plugins/www/openapi-gen/requests/types.gen.ts
Outdated
Show resolved
Hide resolved
dfd1341 to
d841edc
Compare
|
@kaxil Thanks for the merge but all the files in ui.rst were not updated rightly in the docs after i tried to rebase. I was trying to fix this issue by adding all other left images, Since this PR has been merged let me raise another PR for the files which still remains not updated, Is it fine with you? |
|
Yes, please do it |
Description
This PR updates the UI Guide documentation to reflect the refreshed Airflow 3.1.x UI.
:alt:text for accessibility and clarity.docs/ui.rst.These changes ensure that the UI Guide remains accurate and visually consistent for Airflow 3.1.x.
Related Issues
closes: #55521
Notes