I'm not sure if this module is still being maintained(?), but at this point I think it's the only 6.x option for roles access control of content.

I've been testing it out under 6.1 and it seems to work fine, except when you are creating/editing a particular node, the Add/Edit Permission form table is not visible with IE7. It is with Firefox.

I'm not sure what IE is being picky about. Maybe the sticky header table that's in a collapsed fieldset initially?

Comments

Moonshine’s picture

Moonshine commented

Just as a followup, this IE7 display problem only seems to happen when add/editing a node as the administrator for some reason. What's more odd is that I can see the table code in the HTML and it seems well formed at first glance.

Moonshine’s picture

Moonshine commented
Status: Active » Closed (won't fix)

Well it turns out that the new Drupal 6 "sticky table headers" are the problem with the form. If that is in a collapsed field set initially then the table will not display in IE7. For the time being, the fix it to make the fieldset expanded by default or take it out of a table. (as stick tables look to be the default in Drupal 6)

deekayen’s picture

deekayen commented
Status: Closed (won't fix) » Active

I guess I could make it expanded by default. I'd be nice if that didn't have to be the case. Perhaps there's an option to turn off scrollable headings on tables.

Moonshine’s picture

Moonshine commented

yeah... I agree it's sort of lame to have it expanded by default, but that's about the only work-around short of not making it a table (which I like personally) or overiding theme_table(). :-O I am suprised that turning off sticky headers isn't a option for theme_table though. I guess there could be some way to disable it via Javascript? But that's pretty hackish also.

Anyways... i did submit an issue against core about this, as it really would affect any form with a table in a fieldset.

#251400: Collapsed fieldsets containing sticky header tables won't display table when expanded in IE7

Moonshine’s picture

Moonshine commented

Actually, you could make your own table theme fuction to call that's just theme_table with the few lines of sticky header code removed I guess. Not ideal, but it would give a clean normal drupal table.

deekayen’s picture

deekayen commented
Version: 6.x-1.0 » 6.x-1.1
Status: Active » Needs review

I committed #2. Someone please review. Mark fixed if it works and I'll make a release.

http://cvs.drupal.org/viewvc.py/drupal/contributions/modules/node_privac...

andrewbenkard’s picture

andrewbenkard commented

Works for me. Yay!
The table is displayed as open at first. Closing and opening the table by clicking the header works.
FWIW there is a trivial CSS/ie rendering issue; when opening the table, its column headings make a very brief appearance at the top of the window.
Thanks deekayen.

ie v7.0.5730.13

deekayen’s picture

deekayen commented
Status: Needs review » Fixed

there's only so much with this that I can say is a npbr bug. the css rendering i'm pointing at drupal core's table code or ie7. since it's tested by more than just me, i'll be making a release shortly.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.