From beb4dfbde5c7e67a9ddfb337f9c8869d7879241b Mon Sep 17 00:00:00 2001
From: deathaxe <deathaxe82@googlemail.com>
Date: Sat, 9 Mar 2024 18:17:26 +0100
Subject: [PATCH] Add comments support to parameter lists

---
 Syntaxes/SCSS.sublime-syntax |  3 +++
 Tests/syntax_test_scss.scss  | 25 ++++++++++++++-----------
 2 files changed, 17 insertions(+), 11 deletions(-)

diff --git a/Syntaxes/SCSS.sublime-syntax b/Syntaxes/SCSS.sublime-syntax
index f496d1b5..6661b0ca 100644
--- a/Syntaxes/SCSS.sublime-syntax
+++ b/Syntaxes/SCSS.sublime-syntax
@@ -636,6 +636,7 @@ contexts:
     - match: \)
       scope: punctuation.section.group.end.scss
       pop: 1
+    - include: comments
     - include: comma-delimiters
     - include: scss-variadics
     - include: scss-parameters
@@ -679,6 +680,7 @@ contexts:
     - match: \)
       scope: punctuation.section.group.end.scss
       pop: 1
+    - include: comments
     - include: comma-delimiters
     - include: scss-variadics
     - include: scss-parameters
@@ -752,6 +754,7 @@ contexts:
     - match: \)
       scope: punctuation.section.group.end.scss
       pop: 1
+    - include: comments
     - include: comma-delimiters
     - include: scss-variadics
     - include: scss-parameters
diff --git a/Tests/syntax_test_scss.scss b/Tests/syntax_test_scss.scss
index 22ca047c..e9f6a54b 100644
--- a/Tests/syntax_test_scss.scss
+++ b/Tests/syntax_test_scss.scss
@@ -1061,13 +1061,13 @@ nav ul {
 // At-Rules: mixin arguments
 // https://sass-lang.com/documentation/at-rules/mixin/#arguments
 //=============================================================================
-@mixin rtl($property, $ltr-value, $rtl-value) {
+@mixin rtl($property, $ltr-value, /**/ $rtl-value) {
 // <- meta.at-rule.mixin.scss meta.mixin.scss keyword.control.directive.mixin.scss punctuation.definition.keyword.scss
 //^^^^^ meta.at-rule.mixin.scss meta.mixin.scss
 //     ^^^ meta.at-rule.mixin.scss meta.mixin.identifier.scss
-//        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ meta.at-rule.mixin.scss meta.mixin.parameters.scss meta.group.scss
-//                                           ^ meta.at-rule.mixin.scss - meta.block
-//                                            ^^ meta.at-rule.mixin.scss meta.property-list.css meta.block.css
+//        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ meta.at-rule.mixin.scss meta.mixin.parameters.scss meta.group.scss
+//                                                ^ meta.at-rule.mixin.scss - meta.block
+//                                                 ^^ meta.at-rule.mixin.scss meta.property-list.css meta.block.css
 //^^^^ keyword.control.directive.mixin.scss
 //     ^^^ entity.name.mixin.scss
 //        ^ punctuation.section.group.begin.scss
@@ -1075,9 +1075,10 @@ nav ul {
 //                  ^ punctuation.separator.sequence.css
 //                    ^^^^^^^^^^ variable.parameter.scss
 //                              ^ punctuation.separator.sequence.css
-//                                ^^^^^^^^^^ variable.parameter.scss
-//                                          ^ punctuation.section.group.end.scss
-//                                            ^ punctuation.section.block.begin.css
+//                                ^^^^ comment.block.css
+//                                     ^^^^^^^^^^ variable.parameter.scss
+//                                               ^ punctuation.section.group.end.scss
+//                                                 ^ punctuation.section.block.begin.css
   #{$property}: $ltr-value;
 // <- meta.at-rule.mixin.scss meta.property-list.css meta.block.css
 //^^^^^^^^^^^^^^^^^^^^^^^^^^ meta.at-rule.mixin.scss meta.property-list.css meta.block.css
@@ -1114,11 +1115,11 @@ nav ul {
 }
 
 .sidebar {
-  @include rtl(float, left, right);
+  @include rtl(float, left, /* in */ right);
 //^^^^^^^^^ meta.at-rule.include.scss - meta.mixin
 //         ^^^ meta.at-rule.include.scss meta.mixin.scss
-//            ^^^^^^^^^^^^^^^^^^^^ meta.at-rule.include.scss meta.mixin.arguments.scss meta.group.scss
-//                                ^ - meta.at-rule - meta.mixin - meta.group
+//            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ meta.at-rule.include.scss meta.mixin.arguments.scss meta.group.scss
+//                                         ^ - meta.at-rule - meta.mixin - meta.group
 //^ punctuation.definition.keyword.scss
 //^^^^^^^^ keyword.control.directive.include.scss
 //         ^^^ variable.other.mixin.scss
@@ -1127,7 +1128,9 @@ nav ul {
 //                  ^ punctuation.separator.sequence.css
 //                    ^^^^ support.constant.property-value.css
 //                        ^ punctuation.separator.sequence.css
-//                               ^ punctuation.section.group.end.scss
+//                          ^^^^^^^^ comment.block.css
+//                                   ^^^^^ support.constant.property-value.css
+//                                        ^ punctuation.section.group.end.scss
 }
 
 //=============================================================================