2

My target is to set or remove class depending on component boolean with Angular 2. For example: isRed = true > add class "red", if isRed = false > remove class "red". How is that possible? Code tried:

isRed: boolean;

constructor() {
    $(document).scroll(function(){
        var scrollTop = $(this).scrollTop();
        if(window.location.hash) {

        } else{
            this.isRed = true;
        }
        if(scrollTop > 50) {
            this.isRed = true;
        }
        else  {
            this.isRed = false;
        }
    });
}

and html:

[ngClass]="{red: isRed}"
3
  • Your code should work, does it not? Commented Jan 17, 2017 at 14:34
  • Is the different variable naming in the constructor a copy/paste issue? If not isRed might be always undefined.
    – Philipp
    Commented Jan 17, 2017 at 14:35
  • Can you please share more code (full class and template) Commented Jan 17, 2017 at 14:41

2 Answers 2

8

The most concise way is IMHO

[class.red]="isRed"

update

The cause of your problem is function in

 $(document).scroll(function(){

it should use arrow function

 $(document).scroll(() => {

otherwise this within the callback won't point to the current class, but instead to the caller.

I'd suggest you try to avoid jQuery with Angular2. Use instead

class MyComponent {

  constructor(private elRef:ElementRef) {}

  isRed:boolean;

  @HostListener('document:scroll', ['$event'])
  onScroll(event:any) {
    var scrollTop = this.elRef.nativeElement.scrollTop;
    // or
    $(this.elRef.nativeElement).scrollTop();

    if(window.location.hash) {

    } else{
        this.isRed = true;
    }
    if(scrollTop > 50) {
        this.isRed = true;
    }
    else  {
        this.isRed = false;
    }
  }
}
9
  • isRed <-> isRead as mentioned by @Phillip Commented Jan 17, 2017 at 14:36
  • it is not from that Commented Jan 17, 2017 at 14:36
  • Then there is something else wrong - nothing in the code you posted. Commented Jan 17, 2017 at 14:37
  • @HostBinding('document:scroll', ['$event']) Support parameter did not match any signature of target Commented Jan 17, 2017 at 15:10
  • Sorry, should have been HostListener instead of HostBinding Commented Jan 17, 2017 at 15:12
-1

This is javascript so I would try something like:

isRed; // there's no need to initialize this variable
          since the constructor has its own scope but hey,
          do it if you wish so

Also it doesn't seem that you are working inside an object since you are using ; instead of , which means that you shouldnt use ":" but rather "="

Start asking to get answers

Find the answer to your question by asking.

Ask question

Explore related questions

See similar questions with these tags.