Skip to content

Conversation

guan404ming
Copy link
Contributor

Why

There are some small issues about layout.

How

  • fix ui error gap
  • make timezone-selection text have gap
  • fix AssetEvent and Dashboard layout

before (fix ui error gap)
image
after (fix ui error gap)
image

before (fix AssetEvent and Dashboard layout)
image
after (fix AssetEvent and Dashboard layout)
image

before
image
after
image


^ 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 airflow-core/newsfragments.

@boring-cyborg boring-cyborg bot added the area:UI Related to UI/UX. For Frontend Developers. label May 11, 2025
@guan404ming
Copy link
Contributor Author

Question: should we need to let the ui error show across pages instead of making it only appear in homepage?

@bbovenzi bbovenzi added this to the Airflow 3.0.2 milestone May 12, 2025
Copy link
Contributor

@bbovenzi bbovenzi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great eye for detail!

@bbovenzi
Copy link
Contributor

Question: should we need to let the ui error show across pages instead of making it only appear in homepage?

Oh probably the Dags list too at least, maybe the Dag details page too?

@bbovenzi bbovenzi merged commit d73d5fa into apache:main May 12, 2025
43 checks passed
@guan404ming guan404ming deleted the fix-timezone-selection-ui branch May 13, 2025 14:34
@guan404ming
Copy link
Contributor Author

Thanks, I would try implement it.

kaxil pushed a commit that referenced this pull request Jun 3, 2025
kaxil pushed a commit that referenced this pull request Jun 3, 2025
sanederchik pushed a commit to sanederchik/airflow that referenced this pull request Jun 7, 2025
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants