show else prefix for instance chain items in docs#3715
show else prefix for instance chain items in docs#3715csicar wants to merge 7 commits intopurescript:masterfrom
else prefix for instance chain items in docs#3715Conversation
Yes please, good call. |
|
The solution in bc3cc4e is not very elegant, but works. Basically the problem is, that rendering of the ChildDeclaration needs to know if the parent declaration is a TypeClass. Alternatively, I could pass the Parent declaration to |
|
This is actually quite a bit more subtle than I first expected. I think what we have here is still not quite right, for a couple of reasons:
I'm trying to come up with some convincing examples. I think the correct approach here may be to represent the entire chain as a single |
bc3cc4e to
7943881
Compare
|
New version is improved in pretty much every way:
TODO:
|
c00e543 to
b8805c8
Compare


Fix for Issue #3605
example:

Changes
Boolto ChildInstance Constructorchainedto JSON representation, with default valueFalsefor backwards-compatabilityelsekeywordNotes
Displaying the instances of a type can look weird because the

elseprefix has no meaning there. I would suggest hiding theelsekeyword when displaying anywhere except a type class context