opened
nqthqn
username_0
Would this work? Ripped out of https://github.com/gtap-dev/nighthawk
Adds search
issue
created
Dan9boi
username_1
Looks very good to me!
comment
created
Dan9boi
username_1
this fixes #27 btw...
comment
created
LeBenLeBen
username_2
I gave it a quick look and beyond the fact that I would love to see this feature land and therefor appreciate the contribution, I cannot merge it as is.

There are a couple of issues that should be fixed first:
* Move inline styles to https://github.com/frctl/mandelbrot/blob/develop/assets/scss/components/_navigation.scss or new component file depending on how you'll name your classes
* Move inline script to a dedicated JS file, probably in https://github.com/frctl/mandelbrot/tree/develop/assets/js/components
* Clean-up unused classes and rename the others to match the existing code base naming

Once it's clean, I'd be happy to put some more precise comments about the implementation.
comment
created
ahebrank
username_3
I've got an update for this PR that's more in line with existing standards... but please merge #80 or something like it since as noted in #77 the node deps won't currently install.

https://github.com/username_3/mandelbrot/pull/1/files
comment
created
LeBenLeBen
username_2
Closing in favor of #81
comment