SerenityOS / serenity

The Serenity Operating System 🐞
https://serenityos.org
BSD 2-Clause "Simplified" License
30.58k stars 3.19k forks source link

LibWeb: Something is wrong with height calculation inside flex item #21054

Closed kalenikaliaksandr closed 5 months ago

kalenikaliaksandr commented 1 year ago

Reduction from https://linktr.ee/katalinkult:

<!DOCTYPE html><style type="text/css">
  * {
    border: 1px solid black;
  }
  .flex-item {
    flex-grow: 1;
  }
  .box {
    height: 100%;
  }
  .flex-container {
    display: flex;
    min-height: 100px;
    flex-direction: column;
  }
</style><div class="flex-container"><div class="flex-item"><div class="box">I make music you can dream to.</div><div>My SoundCloud</div></div></div>

Funding

Fund with Polar

tcl3 commented 5 months ago

The reduction now renders correctly for me.

The commit that fixed this appears to be f963bb4f3632358a79fc3744ee7c8eb141e1662b.

Does that mean this issue can be closed?

awesomekling commented 5 months ago

Oh hey, nice that we got it fixed. Yes, then we can close it! :)