microsoft / pxt-neopixel

A Neo-Pixel package for pxt-microbit
https://makecode.microbit.org/pkg/microsoft/pxt-neopixel
MIT License
58 stars 141 forks source link

documentation is minimal #47

Open whaleygeek opened 5 years ago

whaleygeek commented 5 years ago

The documentation for this package is not complete - right-clicking on 'help' for any of the NeoPixel package docs takes you to this very basic docs page, and many of the blocks have no help at all.

https://makecode.microbit.org/pkg/microsoft/pxt-neopixel

Screen Shot 2019-10-22 at 13 15 52

Given that this is an approved package, it must have gone through an approval process. Does the approval process check correctness and accuracy of the documentation?

This has surfaced directly into makecode.microbit.org so presumably it has passed the micro:bit Foundation's quality review process??

Could be ask the authors to complete the documentation for this package please?

Thanks!

microbit-mark commented 5 years ago

@pelikhan @ganicke could you review the docs here?

whaleygeek commented 5 years ago

I noted the following public methods, all of which have a hint in the blocks, but clicking on 'help' takes to the same page that does not offer detailed help about the methods their behaviours, their parameters, and an example of what it does and why that would be useful.

create
rgb
hsl
HueInterpolationDirection
colours
neopixel mode
showColor
showRainbow
showBarGraph
setPixelColor
setMatrixWidth
setMatrixColor
setPixelWhiteLED
show
clear
length
setBrightness
easeBrightness
range
shift
rotate
setPin
power
ganicke commented 5 years ago

Adding reference for the blocks would be for en only as this extension (package) is not currently part of our docs translation path.

microbit-mark commented 4 years ago

Even if its en only, I think this would be a good start @ganicke

microbit-mark commented 4 years ago

Also, now you can do block embedding in GH pages, this might be a good candidate to have it's own docs site as an example to others. There is a Jekyll theme for JTD that would work nicely.