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

Filter's UI dropdown is hidden under the bottom level of the grid #2928

Closed
npaunov opened this issue Nov 5, 2018 · 4 comments · Fixed by #3118
Closed

Filter's UI dropdown is hidden under the bottom level of the grid #2928

npaunov opened this issue Nov 5, 2018 · 4 comments · Fixed by #3118

Comments

@npaunov
Copy link
Contributor

npaunov commented Nov 5, 2018

Description

Filter dropdown is hidden under the bottom level of the grid

  • igniteui-angular version: all
  • browser: 6.2.x

Steps to reproduce

  1. Go to dev samples - "Grid Column Moving" , "Grid Summary"
  2. Open the Filtering dropDown

Result

The end of operands list is not visible.

Expected result

The end of operands list is should be visible.

Attachments

image

@bkulov
Copy link
Contributor

bkulov commented Nov 13, 2018

Filtering conditions dropdown now have a limited height (4 items and a scrollbar) so please take a look at the case where it goes under the summaries.

@desig9stein
Copy link
Contributor

desig9stein commented Nov 14, 2018

We can increase the z-index of .igx-grid__outlet from 2 to 1003 in order to make it bigger than the z-index of the footer, I test that and it fixes the issue @simeonoff are you ok with that?

@desig9stein
Copy link
Contributor

Here is a table with all z-index added form SCSS
screenshot 2018-11-14 at 14 33 50

@simeonoff
Copy link
Collaborator

simeonoff commented Nov 19, 2018

@desig9stein thanks for the z-index mapping, however, without markup context it's still difficult to grasp the z-index structure. Also, I don't think this table takes into account all dynamically assigned z-index values.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants