Skip to content
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

Airflow Logo hides half of the page #11025

Closed
kaxil opened this issue Sep 19, 2020 · 5 comments · Fixed by #11028
Closed

Airflow Logo hides half of the page #11025

kaxil opened this issue Sep 19, 2020 · 5 comments · Fixed by #11028
Labels
area:webserver Webserver related Issues kind:bug This is a clearly a bug

Comments

@kaxil
Copy link
Member

kaxil commented Sep 19, 2020

Looks like a side-effect of #11018

Airflow Logo almost takes up half of the page on current master:

image

cc @ryanahamilton

@kaxil kaxil added the kind:bug This is a clearly a bug label Sep 19, 2020
@ryanahamilton
Copy link
Contributor

This would surely be a side-effect of #11018, but I'm unable to reproduce in Chrome, Safari, or Firefox on Mac. I can certainly provide a simple solution for this if it is indeed a problem.

@yuqian90
Copy link
Contributor

I could reproduce this in Chrome on Mac.

@ryanahamilton I think there are two issues at least.

  1. For the problem @kaxil reported, I think it's caused by the javascript code in graph.html, tree.html, etc that does this. These code picks up the svg tag in the header as well and transform it. That's why it became distorted. If you could identify all these code and change them to select something more specific that should fix it, e.g. d3.select("#svg_container svg") or something along that line. It's not clear to me why you had issues reproducing this though.
  d3.select("svg")
  .insert("g")
...
  1. Unrelated to 1, even in the home page, the alignment of the logo seems off. Looks like this for me. I believe this is a simpler problem than 1.

image

@turbaszek
Copy link
Member

image

Go big or go home 😸

@turbaszek turbaszek added the area:webserver Webserver related Issues label Sep 19, 2020
@ryanahamilton
Copy link
Contributor

ryanahamilton commented Sep 19, 2020

I can see it now—I didn't know it was view-specific and hadn't ventured off of the home page in my testing.

@yuqian90 yes, that is indeed the issue. Generic selectors that assumed another SVG element would never be added to the page. I'll fix this.

As for (2), you. probably just need to run airflow/www/compile_assets.sh—there is a CSS adjustment of the padding around the logo in main.css that you don't appear to have.

@ryanahamilton
Copy link
Contributor

Graph view is also drawing teensy little charts on top of the logo 🧐

Image 2020-09-19 at 8 05 41 AM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:webserver Webserver related Issues kind:bug This is a clearly a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants