Skip to content

Conversation

@kaxil
Copy link
Member

@kaxil kaxil commented Oct 7, 2021

Before:

image

After:
image

This got broken in #18715


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code change, 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 UPDATING.md.

@kaxil kaxil requested review from ashb and bbovenzi October 7, 2021 09:17
@kaxil kaxil requested a review from ryanahamilton as a code owner October 7, 2021 09:17
@boring-cyborg boring-cyborg bot added area:UI Related to UI/UX. For Frontend Developers. area:webserver Webserver related Issues labels Oct 7, 2021
@kaxil kaxil added this to the Airflow 2.2.0 milestone Oct 7, 2021
@ashb
Copy link
Member

ashb commented Oct 7, 2021

This was changed yesterday in #18715 🤔

@kaxil
Copy link
Member Author

kaxil commented Oct 7, 2021

This was changed yesterday in #18715 🤔

Yup, the QA team were using the image containing changes in that PR

@ashb
Copy link
Member

ashb commented Oct 7, 2021

Which page is your screenshot from? Cos the original PR talks about:

Removed btn-sm class that made the buttons next to each DAG on DAGs list not centered perfectly.

@kaxil
Copy link
Member Author

kaxil commented Oct 7, 2021

Which page is your screenshot from? Cos the original PR talks about:

Home Page (DAG View)

image

PS: Here I had manually updated the CSS style for one of the element to test the fix

@ashb
Copy link
Member

ashb commented Oct 7, 2021

Curious. The first row there is fine, but the second two aren't.

@kaxil
Copy link
Member Author

kaxil commented Oct 7, 2021

Curious. The first row there is fine, but the second two aren't.

I had manually updated the CSS style for one of the element (first row) to test the fix

@kaxil
Copy link
Member Author

kaxil commented Oct 7, 2021

With this PR:

image

Without this PR:

image

@kaxil kaxil merged commit cfa8fe2 into apache:main Oct 7, 2021
@kaxil kaxil deleted the fix-alignment-action-button branch October 7, 2021 10:15
@uranusjr
Copy link
Member

uranusjr commented Oct 7, 2021

[css-family-guy.gif]

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants