Skip to content

add group type 'dots' so that we get the right font w/o special casing group type 'op'#2

Merged
edemaine merged 1 commit intoedemaine:masterfrom
KaTeX:dots
Aug 13, 2017
Merged

add group type 'dots' so that we get the right font w/o special casing group type 'op'#2
edemaine merged 1 commit intoedemaine:masterfrom
KaTeX:dots

Conversation

@k4b7
Copy link

@k4b7 k4b7 commented Aug 13, 2017

dots

Green is LaTeX and red is KaTeX. The last two lines use \dots whereas the first is \dotsb and the second is \dotsi. The issue with the second line is a pre-existing issue.

@edemaine
Copy link
Owner

This works, but I was vaguely hoping to do it via the macro system (so that the produced content is identical to the effect of typing \cdots or whatever). Of course, I haven't had time to do that yet, so maybe this is worth going with...

@edemaine edemaine merged commit 885a4db into edemaine:master Aug 13, 2017
@k4b7
Copy link
Author

k4b7 commented Aug 13, 2017

I don't think our support for macros is powerful enough b/c we need to know what the next token is and then somehow use that to decide which version of dots to render.

@edemaine
Copy link
Owner

Yes, but I think it'd be worth adding support for that to the macro parser. I wasn't planning on implementing \futurelet, but rather allow defineMacro to give a function which gets a context that has a nextToken equivalent of \futurelet... Of course, at that point, maybe I'm just implementing defineFunction?

@edemaine
Copy link
Owner

Maybe the dots function should recursively call the parser to get the output of \cdots and \ldots? Is that crazy?

@k4b7 k4b7 deleted the dots branch November 27, 2017 01:03
k4b7 pushed a commit that referenced this pull request Jun 3, 2018
* Support \cfrac

This was an easier PR to write than KaTeX#135 since KaTeX now has well established methods for pt-to-em conversions and nulldelimiter.

As in KaTeX#135, this does not support the LaTeX optional argument for numerator justification.  It supports a `\cfrac #1 #2 ` syntax.

* Added screenshotter files

* fix ref to context in genfrac handler
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants