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

showTreeRowHeader does not work for empty data #5430

Closed
dhairya-avantsoft opened this issue May 24, 2016 · 3 comments · Fixed by #6694
Closed

showTreeRowHeader does not work for empty data #5430

dhairya-avantsoft opened this issue May 24, 2016 · 3 comments · Fixed by #6694
Assignees

Comments

@dhairya-avantsoft
Copy link

http://plnkr.co/edit/I6Fwu0xKEBEVJqkyUk6y?p=preview

I've just copied the plunkr for sample treegrid and update 2 lines:
showTreeRowHeader: false
and
$scope.gridOptions.data = [];

Note that showTreeRowHeader is false and still row header for tree nodes is still visible

@bjaraujo
Copy link
Contributor

I guess you have to remove ui-grid-tree-view.
But I agree it should work with settingshowTreeRowHeader: false so maybe it is a bug.

@r0ss0neri
Copy link

r0ss0neri commented Feb 27, 2018

Still reproducible, is this a bug?

@mportuga
Copy link
Member

yes, definitely a bug.

@mportuga mportuga self-assigned this Apr 21, 2018
mportuga pushed a commit that referenced this issue Apr 24, 2018
Ensure row header visiblity is respected when no data is present.

fix #5430
mportuga pushed a commit that referenced this issue Apr 24, 2018
Ensure row header visiblity is respected when no data is present.

fix #5430
m4m4m4 pushed a commit to m4m4m4/ui-grid that referenced this issue Apr 24, 2018
Ensure row header visiblity is respected when no data is present.

fix angular-ui#5430
defields923 pushed a commit to defields923/ui-grid that referenced this issue Oct 30, 2018
Ensure row header visiblity is respected when no data is present.

fix angular-ui#5430
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.

4 participants