-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Issue with vertical scroll bar hidding the column data #3157
Comments
I do not think WHEN_NEEDED is implemented though. Last time I checked the source code it was not used anywhere. |
@AdityaJitu Please update to the latest RC. This has been fixed in versions newer than the one you're using. |
Well, I'm using the latest one and the issue is still there. I do not think it is an issue actually but by design. Unless somebody thinks it is a bug. |
@yurigenin The issue should be fixed as of this commit: cf86090 Can you reproduce the problem in a plunker? |
Even your tutorials do not work now and the same plunker in the original case has the same behavior. The plunker is using unstable as all your tutorials... From: Brian Hann notifications@github.com @yurigenin The issue should be fixed as of this commit: cf86090 Can you reproduce the problem in a plunker? — |
@yurigenin What are you talking about? What plunker? There is no plunker linked in this issue. What tutorial are you referring to? |
#3138. The original one you closed. There is a plunker there. Go to any tutorial like this: http://ui-grid.info/docs/#/tutorial/210_selection And try to use it. Do you see a selection header? Do you see any menus? That's what I'm talking about. Thanks, Yuri From: Brian Hann notifications@github.com @yurigenin What are you talking about? What plunker? There is no plunker linked in this issue. What tutorial are you referring to? — |
@yurigenin That's a completely separate issue from this one. This issue is about the scrollbar hiding cell contents. Your issue is about icons not being visible on the tutorials. You already filed an issue (four separate issues in fact) for that. |
You are right, it is a separate issue. The issue I was talking about (#3138) was closed before I could test. I added several comments to it asking to reopen but nobody replied. No, my issue is not about icons. My issue is this: Filter widgets (controls) do not show up if filtering is enabled dynamically #3138 Since nobody replied to my requests to reopen I had to open other ones. Anyway, if you are saying the issue #3138 is resolved, then everything is honky dorky. Except it is not resolved, thus, this thread. @yurigenin That's a completely separate issue from this one. This issue is about the scrollbar hiding cell contents. Your issue is about icons not being visible on the tutorials. You already filed an issue (four separate issues in fact) for that. — |
Hi @c0bra , @yurigenin , Thank you for your comments. I will use the latest release & verify it again. With Regards, |
@yurigenin @AdityaJitu this should be fixed now. Broken tests were preventing the build from completing. |
Hi @c0bra , It is working fine now. The issue arises rarely. I got this issue when I try to resize the browser by dragging the browser edges or by clicking the "restore button" of the browser. It does not happen frequently though. |
Sounds like this is resolved. Closing. |
Hi,
Initially when the grid is loaded then the rightmost column is displayed properly. But when I resize the grid & then the scroll-bar overlaps the contents of the right-most(last) column.
I am attaching the screenshot of the issue that I encountered.
As of now I am using this version.
ui-grid - v3.0.0-RC.18-92a8836 - 2014-12-25
I am highlighting the issue.
We are showing the scrollbars conditionally using
enableHorizontalScrollbar:uiGridConstants.scrollbars.WHEN_NEEDED,
enableVerticalScrollbar:uiGridConstants.scrollbars.WHEN_NEEDED
The text was updated successfully, but these errors were encountered: