remove all ids and replace with classes.

Major refactoring of code required!!

for example core modules

Commenting is hard to read on routes.js file, putting the comments on multiple short lines will help readability.

• on doubleclick no zoom (as node highlighting occurs on dbclick)
• slider zoom bar
• for node highlighting add a thick border around the selected nodes and links

remove anythign with name_modules from the tree

nodes now an object with properties as unique file paths
and links objects which have properties source and target with value those node properties(file path)

couldn't find link to site to eneter repo details to get markdown for bandge. Also could add a copy button for getting the markdown code.

Now, Code Crystal looks awesome! But I don't think it gives a lot of information (particularly for highly modularised code). One idea for making it work better is setting the strength of links based on the number of links between nodes. Any others?

has README spelt READme? :)

Could we just check that with the package.json? (Thanks @besarthoxhaj)

The tests are all gone, and someone ran a git filter on our commit history to get rid of them!

• User should type in user/repo/branch and we generate the crystal and save to the database and create a web-hook.
• User can copy the link and paste in READme
• remove user authentication and retrieval of all repos

Instead of node server.js use foreman to import your env variables

Just a note about customer service, guys - if you have an email address, you need someone to respond to email queries. Gitter is no different - if you set up a gitter channel, you need to make sure someone monitors it or you'll be giving your users a poor service experience from the outset.

Variables like '__dirname' can appear within require() so need to parse for these and evaluate

exclude pics, maybe unnecessary files (i.e. unconnected)?

after 100 lines links to file on github (or comment saying download our nom module and clone repo and view in crystal ball folder!!!!!!!!!!!!)

• legend size
• background colours
• graph size based on repo size

When I click to generate the code crystal, my immediate reaction upon seeing the blue square with the markdown that appears is to think that it has generated a not very user friendly error!
I'm pretty sure it's because the markdown starts with an exclamation mark !. It might be worth reviewing the UI for how this is displayed 😊

❤️ code crystal, it's so great, thanks for making it!

can hack it but not good enough

the right part of the app is under your graph instead of being aligned

• show the code instead of a preview of the rendered html file

Thank you, noble @nikhilaravi!

website first then npm module

It would help if you had an image of what it looks like once your app has executed and created a visual representation. It would really help me to understand what this app does.

Would be a nice addition

Ignores es6 files

in your readme the name of your env variables must be in capital letters

• neaten up code
• functions take single repo as argument instead of an array
  @jrans

Refreshing the page leads to extra connections being created between the files. Restarting the server resets the connections.

How can we 🌟 your repo otherwise? 💚

some of your lines are very long and require much scrolling - you might want to consider splitting them up.

Comments too

To make it clearer to read the text on top of the map

only connected nodes' text should be displayed

To make the back end graph and file dependencies clearer

code snippets in the READme that have 'require' cause incorrect connections.

Love the new site! Freaking awesome guys. I know this is ultimately more of a desktop thing, but it would be cool if it worked on mobile. I actually opened it first on mobile. The graph might be harder to make responsive, but at the very least a couple of media queries on padding etc. would make the landing page viewable on mobile.

It would be awsome to have a short line under the name of the file to describe what is the purpose of the file

Stretch

Test with the canary app. Create folders of nodes to reorganize your nodes or don't load the files from the other packages (don't parse the node_module folder)

https://github.com/Crystal-Clear/codecrystal/blob/master/d3/parser.js#L2-L4

not a huge fan of this function - seems like a lot of work for something quite simple - might want to tryn
something more like this:

function str(c) {
    return "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz1234567890".indexOf(c) >= 0;
}

I tested this and it takes 0.20ms compared to your 0.27ms