Skip to content

bugfix: Remove entity.other.inherited-class.scala #242

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

Merged
merged 1 commit into from
Sep 23, 2022

Conversation

tgodzik
Copy link
Contributor

@tgodzik tgodzik commented Sep 16, 2022

Previously, symbol in inheritance clause would be treated differently than outside it. Now, they are all treated the same.

There are three reasons for that change:

  • most themes don't have a specific color for entity.other.inherited-class.scala and it would end up the same color as string
  • there is no reason to show the same symbol with different color aside from definition/declaration
  • there is no sensible way to handle derives A, B, C or extends A, B, C and the first A would be colored differently

Fixes #239

Previously, symbol in inheritance clause would be treated differently than outside it. Now, they are all treated the same.

There are three reasons for that change:
- most themes don't have a specific color for `entity.other.inherited-class.scala` and it would end up the same color as string
- there is no reason to show the same symbol with different color aside from definition/declaration
- there is no sensible way to handle `derives A, B, C` or `extends A, B, C` and the first `A` would be colored differently

Fixes scala#239
@tgodzik tgodzik requested a review from MaximeKjaer September 22, 2022 09:57
@tgodzik tgodzik merged commit 256002e into scala:main Sep 23, 2022
@tgodzik tgodzik deleted the remove-inherited branch September 23, 2022 14:13
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.

Wrong highlighting with derives in Scala 3
1 participant