Skip to content

header element not yet available when using amd and angular #5

Description

@micahblu

The header is queried before angular appends the directive html to the dom, this could be circumvented with a quick check of the header var in the page scroll function and a re-query if necessary. like so:

header = header || document.querySelector( '.cbp-af-header' );

Not sure if this issue is common enough for you guys to make the change but if you'd like I could submit a pr with the fix, it would help me not have to re write the fix when I update :)

Activity

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions