Joel

Respond to viewport size

1

A good next step might be to add responsive behavior so that your game works better on devices with smaller viewports. You could accomplish this with a couple media queries...

It would be nice to order by recently active, so you could quickly jump back and forth between tabs with a lot of open tabs.

Nested list elements don't convert properly

1

Nested list should convert to --, ---, etc.

Publish to npm registry

10

Since you have a package.json, you should probably publish your package to the registry, so browserify users can `npm install` your package and use it properly. You should add a...

Make random option more random

1

The sort method doesn't do a good job of randomizing the order of the slides. With one iteration, elements only have a 50% chance of being swapped with their neighbor....

Have you tested in Opera 11? For me, the behavior is not consistent with Safari, Chrome, Firefox. Basically, the bottoms aren't equalized in Opera 11 for some reason ;)

Use array literal for defining a new array

1

In your code I see a lot of: ``` js var myarray = Array(); ``` type statements. This seems like a mistake. Perhaps you meant `new Array()`?. But, the recommended...

Whitespace inconsistent

1

Minor issue — your js code contains mixed tabs and spaces, and some trailing spaces. It's best to use only tabs or spaces for indentation, so that your code lines...

In v0.3.2, `` tag worked, as described in the [README](https://github.com/chyingp/grunt-inline#inline-tag). However, v0.3.3+ introduced a regression, and nothing is replaced.