Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wrong syntax highlighting in Scala case class #956

Closed
olafleur opened this issue Oct 15, 2015 · 8 comments
Closed

Wrong syntax highlighting in Scala case class #956

olafleur opened this issue Oct 15, 2015 · 8 comments
Assignees
Labels
Milestone

Comments

@olafleur
Copy link

@olafleur olafleur commented Oct 15, 2015

screen shot 2015-10-14 at 7 59 06 pm

First thing, the parentheses should be of the same color.
Second thing, the class after extends does not have the same color every time. Maybe it's intended, I'm not sure.

@Sannis Sannis added the bug label Oct 15, 2015
@Sannis Sannis added this to the 8.9 milestone Oct 15, 2015
@Sannis
Copy link
Collaborator

@Sannis Sannis commented Oct 19, 2015

abstract class Vertical extends CaseJeu
case class Haut(name: String) extends Vertical
case class Bas(name: String) extends Vertical

http://www.scala-lang.org/old/node/107

@olafleur
Copy link
Author

@olafleur olafleur commented Oct 19, 2015

Not sure to understand your comment.

Is it because you cannot reproduce the bug or because it was fixed?

@Sannis
Copy link
Collaborator

@Sannis Sannis commented Oct 19, 2015

It's because I would not like to type this example every time when testing :-/

@olafleur
Copy link
Author

@olafleur olafleur commented Oct 19, 2015

Ok. ;)

@Sannis
Copy link
Collaborator

@Sannis Sannis commented Oct 19, 2015

screen shot 2015-10-20 at 00 25 39

@Sannis Sannis self-assigned this Oct 19, 2015
@Sannis
Copy link
Collaborator

@Sannis Sannis commented Oct 19, 2015

At least it now looks consistent.

@olafleur
Copy link
Author

@olafleur olafleur commented Oct 19, 2015

yes. :)

@Sannis Sannis closed this in 2974ead Oct 19, 2015
@olafleur
Copy link
Author

@olafleur olafleur commented Oct 19, 2015

Thanks !

Sannis added a commit to Sannis/highlight.js that referenced this issue Oct 19, 2015
Sannis added a commit to Sannis/highlight.js that referenced this issue Oct 19, 2015
@Sannis Sannis reopened this Oct 19, 2015
@Sannis Sannis added this to the 8.10 milestone Oct 19, 2015
@Sannis Sannis removed this from the 8.9 milestone Oct 19, 2015
@Sannis Sannis closed this in 6993a49 Oct 21, 2015
@isagalaev isagalaev removed this from the 8.10 milestone Oct 27, 2015
@isagalaev isagalaev added this to the 9.0 milestone Oct 27, 2015
@isagalaev isagalaev added this to the 9.0 milestone Oct 27, 2015
@isagalaev isagalaev removed this from the 8.10 milestone Oct 27, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants