Detailview Scrollbar

[Yuof]

I'm submitting a bug report

• Library Version:
  2.5.5

• Browser:
  all

Current behavior:
When using Detailview the Detailcomponent always has a Scrollbar even if its small enough to fit.

Reproduce:

1. https://ghiscoding.github.io/Angular-Slickgrid/#/rowdetail
2. Open a Detailview
3. Every Detailview has a Scrollbar

Expected/desired behavior:
No scrollbar if component is small enough.

[ghiscoding]

Ah I thought I had fixed that in the core lib, as it was discussed here, but it looks like I was waiting for an answer and push a commit without the change. I'll make another PR on the core lib soon, but it might take a few weeks before any new release is out on the core lib since it's not updated as often.

In the mean time, I'm not sure if that would work, but the problem is on the <div class="detail-container">, the min-height has to be removed. I did not tested it, but maybe trying to reset the min-height like this would work !?

.detail-container {
  min-height: initial !important;
}

or set it to 0 if that didn't work.

[Yuof]

Ah I thought I had fixed that in the core lib, as it was discussed here, but it looks like I was waiting for an answer and push a commit without the change. I'll make another PR on the core lib soon, but it might take a few weeks before any new release is out on the core lib since it's not updated as often.

In the mean time, I'm not sure if that would work, but the problem is on the <div class="detail-container">, the min-height has to be removed. I did not tested it, but maybe trying to reset the min-height like this would work !?

.detail-container {
  min-height: initial !important;
}

or set it to 0 if that didn't work.

I tried both and none worked. It still gets overwritten somewhere.

[ghiscoding]

If that didn't work, you'll have to wait for the fix to be available in the core lib (6pac/SlickGrid). Like I said, this might take few weeks though.

[Yuof]

document.getElementsByClassName("detail-container")[0].setAttribute("style", "min-height: 0;");
worked as an hack for now.

Thank you for the advice!

[ghiscoding]

PR #370 raised and merged on core lib.
We just have to wait for a new version release of the core lib, which I don't have any control over

[ghiscoding]

Now released under version 2.7.0

You can also up vote ⭐️ if you like the lib
Cheers