This is a great module, but a sticky nav is often not desirable for admins and perhaps other roles using a toolbar. They end up with both a fixed and a sticky menu.

I thought that it might possible to disable stickynav for certain roles with context_module, but this serves only to enable or disable a module completely.

Please provide a option to control which roles see a sticky nav.

Comments

skein’s picture

skein commented

Hello

I have added a new permission and an option in the admin section to turn on the sticky navigation permission restriction.

Please review.

skein’s picture

skein commented
Status: Active » Needs review
mrpauldriver’s picture

mrpauldriver commented

Sorry for the delay in getting back to you, I've been pretty busy on another project.

I can see the permission setting that you have introduced and note that it is called 'Administer Sticky Nav - Permission to activate and manage Sticky Nav. However, I am not sure how this permission would allow Sticky Nav to be disabled by role. Have I missed something?

To provide an example. Allow anonymous users to have a Sticky Nav but disable this for Admins (or other role)

skein’s picture

skein commented

Sorry for the delay as well.

Yes you are correct. I'm actually thinking of refactoring the admin interface that it's more flexible and clean.

I'll post in this issue when it's committed

skein’s picture

skein commented
Status: Needs review » Needs work
skein’s picture

skein commented

Hey,

Sorry for the long wait on this but I now replaced the permissions with exclusion roles.
Try it out and if it works I will package a stable release.

  • Commit 8c472b9 on 7.x-1.x: 
    Issue #2182045 by MrPaulDriver: replaced permissions with exclusion...
skein’s picture

skein commented
Status: Needs work » Needs review

  • 397a411 committed on 
    Issue #2182045 by MrPaulDriver: added restriction by role to use sticky...
  • 8c472b9 committed on 
    Issue #2182045 by MrPaulDriver: replaced permissions with exclusion...

  • 397a411 committed on 8.x-1.x 
    Issue #2182045 by MrPaulDriver: added restriction by role to use sticky...
  • 8c472b9 committed on 8.x-1.x
    Issue #2182045 by MrPaulDriver: replaced permissions with exclusion...
skein’s picture

skein commented

Due to no activity and feature implemented, I'm closing this issue

skein’s picture

skein commented
Status: Needs review » Closed (fixed)