search

basicallydan / forkability

:fork_and_knife: A linter for your repository.
https://basicallydan.github.io/forkability
MIT License
103 stars 19 forks source link

More Suggestions and Explanations #31

Open ethel-dev opened 10 years ago

ethel-dev commented 10 years ago

I think you should add more explanations as to what a Test Suite is?

basicallydan commented 10 years ago

I agree, I'll do that! The explanations are a bit sparse in general.

ethel-dev commented 10 years ago

Great!

basicallydan commented 10 years ago

I'm gonna keep this open actually, just as a reminder! :)

hpoom commented 9 years ago

Can I work on the Suggestions section?

I think it should be changed from a popup to expanding out the :warning: icons when clicked/touched on. At the moment it appears explanation fires up a popup but that can probably be done better with inline suggestions.

Maybe I should make another issue for the display of the suggestions as it detracts from this issue. Also another for expanding out Licence document which I am more than happy to write and do a PR for.

basicallydan commented 9 years ago

Can I work on the Suggestions section?

Please do and thank you for offering! I'm glad to see you found some time @hpoom! I still haven't since I made this repo sadly :(

I agree that the popup isn't sustainable. It will quickly get massive and annoying. I'll talk more about that in #32