Skip to content

Commit 23d7bfe

Browse files
committed
Improve highlighting of names in given declarations
Upper cased names are higlighted as types like with module declarations. This also aligns with the explicit use sites of given declarations.
1 parent bca5480 commit 23d7bfe

File tree

2 files changed

+35
-2
lines changed

2 files changed

+35
-2
lines changed

src/typescript/Scala.tmLanguage.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -708,11 +708,12 @@ export const scalaTmLanguage: TmLanguage = {
708708
name: 'meta.package.scala'
709709
},
710710
{
711-
match: `\\b(given)\\b\\s*(?:\\b(as)\\b|(${backQuotedId}|(?!//|/\\*)${plainid})?)`,
711+
match: `\\b(given)\\b\\s*(?:\\b(as)\\b|(${idUpper})|(${backQuotedId}|(?!//|/\\*)${plainid})?)`,
712712
captures: {
713713
'1': { name: 'keyword.declaration.scala' },
714714
'2': { name: 'keyword.declaration.scala' },
715-
'3': { name: 'entity.name.declaration' }
715+
'3': { name: 'entity.name.type.declaration' },
716+
'4': { name: 'entity.name.declaration' },
716717
}
717718
}
718719
]

tests/unit/given.test.scala

Lines changed: 32 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,32 @@
1+
// SYNTAX TEST "source.scala"
2+
3+
4+
given Foo = ???
5+
// ^^^^^ keyword.declaration.scala
6+
// ^^^ entity.name.type.declaration
7+
8+
given foo = ???
9+
// ^^^^^ keyword.declaration.scala
10+
// ^^^ entity.name.declaration
11+
12+
given as Foo = ???
13+
// ^^^^^ keyword.declaration.scala
14+
// ^^ keyword.declaration.scala
15+
// ^^^ entity.name.class
16+
17+
given as foo = ???
18+
// ^^^^^ keyword.declaration.scala
19+
// ^^ keyword.declaration.scala
20+
// ^^^ source.scala
21+
22+
given bar as foo = ???
23+
// ^^^^^ keyword.declaration.scala
24+
// ^^^ entity.name.declaration
25+
// ^^ keyword.declaration.scala
26+
// ^^^ source.scala
27+
28+
given Foo as foo = ???
29+
// ^^^^^ keyword.declaration.scala
30+
// ^^^ entity.name.type.declaration
31+
// ^^ keyword.declaration.scala
32+
// ^^^ source.scala

0 commit comments

Comments
 (0)