fix(router): Link active directive: fix adding CSS class#2227

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

NathanWalker merged 1 commit intoNativeScript:masterfromdenlysenko:fix/ns-router-link-active

Aug 19, 2020

Merged

fix(router): Link active directive: fix adding CSS class#2227

NathanWalker merged 1 commit intoNativeScript:masterfromdenlysenko:fix/ns-router-link-active

Aug 19, 2020

Conversation

Copy link

Contributor

denlysenko commentedAug 19, 2020

fixes#2224

fix(router): link active directive

a48b162

Copy link

cla-botbot commentedAug 19, 2020

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign the CLA athttps://www.nativescript.org/cla.
CLA has not been signed by users:@denlysenko.
After signing the CLA, you can ask me to recheck this PR by posting@cla-bot check as a comment to the PR.