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

Grid: Virtual scrolling - Part of scrollbar not visible (Edge) #1556

Closed
2 of 3 tasks
joerneu opened this issue Nov 1, 2018 · 5 comments · Fixed by #1612
Closed
2 of 3 tasks

Grid: Virtual scrolling - Part of scrollbar not visible (Edge) #1556

joerneu opened this issue Nov 1, 2018 · 5 comments · Fixed by #1612
Assignees
Labels

Comments

@joerneu
Copy link

joerneu commented Nov 1, 2018

  • I have searched this repository's issues and believe that this is not a duplicate.

I'm using ...

  • React Grid
  • React Chart

Current Behaviour

With Edge part of the vertical scrollbar is not visible and cannot be used:

reactgridedgescrollbar

Expected Behaviour

Same layout and behavior as other browsers.

Steps to Reproduce (for Bugs)

Virtual scrolling demo

  • Select "Bootstrap 4"

Environment

  • browser: Edge
  • bootstrap: 4
@joerneu
Copy link
Author

joerneu commented Nov 1, 2018

Found another instance where part of a scrollbar is not visible: (No virtual scrolling)

reactgridedgescrollbar2

Data editing demo

(I found so many issues just looking at the Demos that I am wondering if Edge is missing from your browser tests?)

@ushkal
Copy link
Contributor

ushkal commented Nov 8, 2018

Hi,

We found that this issue is caused by the following bug in the Edge browser: Sticky element overlaps a scrollbar of a container element We won't fix this problem as it is related to the browser.

@ushkal ushkal added the wontfix label Nov 8, 2018
@ushkal ushkal closed this as completed Nov 8, 2018
@joerneu
Copy link
Author

joerneu commented Nov 13, 2018

How about a workaround?

Not fixing it or implementing a workaround makes this feature useless on Edge.

The workaround could probably be as simple as removing:

-ms-overflow-style: -ms-autohiding-scrollbar

At least according to the issue that you linked to.

If you are not going to fix it, you should document that Edge is not fully supported and has issues.

(I believe that it is a mistake to not support IE11 and Edge fully. Those two browsers are often the only browsers allowed in corporate environments.)

@ushkal
Copy link
Contributor

ushkal commented Nov 14, 2018

We discussed the issue again and decided to fix it on our side. In the meantime, you can use the following workaround:

const Container = ({ style, ...restProps }) => (
  <VirtualTable.Container 
    style={{...style, msOverflowStyle: 'auto'}} 
    {...restProps}
  />
);

<Grid>
  ...
  <VirtualTable containerComponent={Container} />
  ...
</Grid>

@lock
Copy link

lock bot commented Jun 15, 2019

This thread has been automatically locked since it is closed and there has not been any recent activity. Please open a new issue for related bugs or feature requests.

@lock lock bot locked as resolved and limited conversation to collaborators Jun 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants