ScrollablePane, DetailsList and Sticky Header: blank space inserted and sticky obscures 1st item

[aappddeevv]

v5.62.

If you modify the ScrollablePane DetailsList Example to remove all non DetailsList sticky headers, you find that there is inserted extra space into the ScrollPane that throws scrolling off. There is also an issue where the DetailsList sticky details list header (per the example) is sitting on top of the first DetailsList item obscuring it, but this actually only happens under some conditions I cannot determine yet. I'm not sure these two issues are related.

 return (
      <ScrollablePane>
{ false &&        <Sticky>{ selectionDetails }</Sticky> }
{ false &&
        <TextField
          label='Filter by name:'
          // tslint:disable-next-line:jsx-no-lambda
          onChanged={ text => this.setState({ items: text ? _items.filter(i => i.name.toLowerCase().indexOf(text) > -1) : _items }) }
        />
}
{ false &&
        <Sticky>
          <h1 style={ { margin: '0px' } }>Item List</h1>
        </Sticky>
}
        <MarqueeSelection selection={ this._selection }>
          <DetailsList
...

Removing these "test" stickies" resulted in a lingering div:

[obiwan007]

The problem with the first item obscuring was fixed in my case by a padding: 1px to the container div. Strange.
Another issue was that I had to add position: relative too - otherwise the sticky was hovering over other elements above the actual scrollable.

[aappddeevv]

I'll try the 1px pad. I did add position: relative as modeled after the example. Without the position: relative in the right place, my stickies jump to the top of the frame/window on scroll. It looks like the sticky uses position so it needs a proper positioned parent (my outer div) to anchor to the right place.

[aappddeevv]

I'm hoping PR #4602 fixes this.

[michaelangotti]

@aappddeevv Can you confirm if PR #4602 has fixed this? IF so, we will close.

[aappddeevv]

I will but it will take me a bit of time to circle back to it. If I don't squawk by Monday you can close it.

[leddie24]

I think this should be resolved from #4491 . Please reopen if this is not the case. Linking a video below with the changes from 4491 (and removing the non DetailsList stickies)

video