Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Update Lens Documentation #57

Open
mightybyte opened this Issue Nov 12, 2012 · 3 comments

Comments

Projects
None yet
2 participants
Owner

mightybyte commented Nov 12, 2012

The lens documentation in src/Snap/Snaplet.hs needs to be updated for the new lens library. This needs to be done before the 0.10 release.

Contributor

sopvop commented Nov 13, 2012

Also please think about limiting amount of Control.Lens exported from Snap.
edwardk suggested exporting only Control.Lens.Loupe, but I think some common things like set, over, view should also be exported. Exporting whole thing pollutes namespace.

P.S. I don't understand lens enough to write docs.

Owner

mightybyte commented Nov 13, 2012

Ok, I changed to only export Control.Lens.Loupe. Docs still need to be updated.

Contributor

sopvop commented Nov 20, 2012

I've tried to update docs, but failed to explain lens simply, because Lens type is not simple, and SnapletLens don't really get generated by makeLenses. sopvop/snap@72a8e4e Ideas, anyone?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment