Conversation
d2e6e30
to
428b8e6
Compare
src/index.scss
Outdated
@@ -30,7 +30,7 @@ $backgroundColor: #fdfdfd; | |||
@import 'components/error/error'; | |||
@import 'components/import/import-api'; | |||
@import 'components/navbar/navbar'; | |||
@import 'components/newsletter-subcription/newsletter-subscription'; | |||
@import 'src/components/newsletter-subcription/newsletter-reminder'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it's strange to add 'src' ...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
flex-direction: column; | ||
align-items: center; | ||
justify-content: center; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think that's not necessary, should be part by component
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
resolved in pair programming !
|
||
this.newsletterPage = document.querySelector('gv-newsletter-subscription'); | ||
this.newsletterPage.addEventListener('gv-newsletter-subscription:subscribe', this.onSubscribe.bind(this)); | ||
this.newsletterPage.addEventListener('gv-newsletter-subscription:skip', this.onSkip.bind(this)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you want, the good syntax to add listeners in template is :
ng-on-gv-newsletter:subscribe="$ctrl.onSubscribe($event)"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
as discussed, we will keep as is since this component is simple and we set a property directly here
428b8e6
to
82ea1d5
Compare
Fixes gravitee-io/issues#4692