Skip to content
This repository has been archived by the owner on Sep 28, 2023. It is now read-only.

fix: ONavMenuComponent selector #118

Closed
wants to merge 2 commits into from
Closed

fix: ONavMenuComponent selector #118

wants to merge 2 commits into from

Conversation

robertIsaac
Copy link
Contributor

fix: change ONavMenuComponent selector from lib-li[o-nav-menu] to li[o-nav-menu] to be usable
add: tslint disable component-selector rule for the selector name of ONavMenuComponent

fix #117

fix: change ONavMenuComponent selector from `lib-li[o-nav-menu]` to `li[o-nav-menu]` to be usable
add: tslint disable component-selector rule for the selector name of ONavMenuComponent

fix #117
@netlify
Copy link

netlify bot commented Oct 11, 2019

You can preview your changes

Building with commit a05d65c

https://app.netlify.com/sites/youthful-allen-8a6081/deploys/5da6cca1d21c680008c70d7f

fix: when an event trigger on OCarouselContainerComponent before swiper being initialized, it throw errors
  this commit tries to prevent that by add save checks to make sure that swiper is defined before it tries to stop/start the autoplay

fix #120
@robertIsaac
Copy link
Contributor Author

@BehnooshShiva @Lausselloic if the o-nav-menu part won't get merged i can make new PL just for the swiper check

@BehnooshShiva
Copy link
Contributor

@robertIsaac
The o-nav-menu correction is merged

@BehnooshShiva
Copy link
Contributor

pull request #119

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

o-nav-menu isn't working
2 participants