include snugssi before links #177
Conversation
example is broken and elements aren't instantiating.
brandondees
left a comment
brandondees
left a comment
There was a problem hiding this comment.
Makes sense to me. @snuggs was this expected/intended to work the other way for some reason?
|
Hmmmmm @Robertchristopher it's a bug if we want to make it a feature. :-) (For now) This looks good. But TECHNICALLY a good user experience would be if where I'm taking a look as well but this looks good regardless. Nice catch by the way. I've definitely experienced this before. Now we can actually have a test for it due to work @brandondees and @tmornini and I did with test coverage. #162, #70, #62 |
|
It would be really nice if we can achieve that but until we can prove that's a safe assumption to rely on, I'd rather say don't assume it can do that and always load the dependency first. @snuggs the CI hook seems to not be responding but it passed already. Merge away. |
|
Well done @brandondees |
example is broken and elements aren't instantiating. moved snuggsi import to above the link tags. @snuggs