From 91651e0a3b4a857a316a64f95f4428368ef10454 Mon Sep 17 00:00:00 2001
From: overlookmotel <557937+overlookmotel@users.noreply.github.com>
Date: Tue, 22 Oct 2024 10:38:15 +0000
Subject: [PATCH] docs(ast): fix comment for `ClassElement::r#static` (#6771)

Follow-up after #6731. Make a long comment to remove ambiguity about what this method does for once and for all!
---
 crates/oxc_ast/src/ast_impl/js.rs | 18 ++++++++++++++++--
 1 file changed, 16 insertions(+), 2 deletions(-)

diff --git a/crates/oxc_ast/src/ast_impl/js.rs b/crates/oxc_ast/src/ast_impl/js.rs
index 892d30d48..ce685ee66 100644
--- a/crates/oxc_ast/src/ast_impl/js.rs
+++ b/crates/oxc_ast/src/ast_impl/js.rs
@@ -1344,8 +1344,22 @@ impl<'a> ClassElement<'a> {
         matches!(self, Self::StaticBlock(_))
     }
 
-    /// Returns `true` if this [`ClassElement`] is a property and has a
-    /// static modifier.
+    /// Returns `true` if this [`ClassElement`] has a static modifier.
+    ///
+    /// Note: Class static blocks do not have a "modifier", as there is no non-static equivalent.
+    /// Therefore, returns `false` for static blocks.
+    ///
+    /// The following all return `true`:
+    /// ```ts
+    /// class {
+    ///   static prop = 1;
+    ///   static method() {}
+    ///   static #private = 2;
+    ///   static #privateMethod() {}
+    ///   static accessor accessorProp = 3;
+    ///   static accessor #privateAccessorProp = 4;
+    /// }
+    /// ```
     pub fn r#static(&self) -> bool {
         match self {
             Self::TSIndexSignature(_) | Self::StaticBlock(_) => false,