Fix some newline issues in enumeration

src/nl/hannahsten/texifyidea/editor/typedhandlers/LatexEnterInEnumerationHandler.kt

@@ -15,6 +15,9 @@
 import com.intellij.openapi.util.Ref
 import com.intellij.psi.PsiElement
 import com.intellij.psi.PsiFile
+import com.intellij.psi.util.parentOfType
+import com.intellij.psi.util.parentOfTypes
+import com.jetbrains.rd.util.first
 import nl.hannahsten.texifyidea.file.LatexFileType
 import nl.hannahsten.texifyidea.psi.*
 import nl.hannahsten.texifyidea.settings.TexifySettings
@@ -71,7 +74,7 @@
      */
     private fun getPreviousMarker(element: PsiElement): String? {
         val environment = element.parentOfType(LatexEnvironment::class) ?: return null
 
         // Last element in the list => Find last \item.
         val label = if (element.parent is LatexEnvironment) {
             getLastLabel(environment)
@@ -82,8 +85,11 @@
         } ?: return null // when no label could befound.
 
         // Extract optional parameters.
-        val optionals = label.childrenOfType(LatexOptionalParam::class).firstOrNull() ?: return null
-        return optionals.text
+        val paramMap = label.getOptionalParameterMap()
+        if (paramMap.isEmpty())
+            return null
+        else
+            return paramMap.first().key.text ?: return null
     }
 
     /**
@@ -141,6 +147,12 @@
 
         val isGluedToTheBeginCommand = element.hasParent(LatexBeginCommand::class)
         val isInsideAnEnumeration = element.inDirectEnvironment(EnvironmentMagic.listingEnvironments)
-        return isInsideAnEnumeration && !isGluedToTheBeginCommand
+        val environment = element.parentOfTypes(LatexEnvironment::class)
+        val isInsideRequiredParam =
+            if (environment != null)
+                element.firstParentOfType(LatexRequiredParam::class)?.isChildOf(getLastLabel(environment)) ?: false
+            else
+                false
+        return isInsideAnEnumeration && !isGluedToTheBeginCommand && !isInsideRequiredParam
     }
 }

test/nl/hannahsten/texifyidea/editor/typedhandlers/LatexEnterInEnumerationHandlerTest.kt

@@ -46,6 +46,46 @@ class LatexEnterInEnumerationHandlerTest : BasePlatformTestCase() {
         )
     }
 
+    fun testItemizeBreakLine() {
+        myFixture.configureByText(
+            LatexFileType,
+            """
+            \begin{itemize}
+                \item This sentence is <caret>broken
+            \end{itemize}
+            """.trimIndent()
+        )
+        myFixture.type("\n")
+        myFixture.checkResult(
+            """
+            \begin{itemize}
+                \item This sentence is 
+                \item <caret>broken
+            \end{itemize}
+            """.trimIndent()
+        )
+    }
+
+    fun testItemizeBreakLineNoItem() {
+        myFixture.configureByText(
+            LatexFileType,
+            """
+            \begin{itemize}
+                \item {This sentence is <caret>broken}
+            \end{itemize}
+            """.trimIndent()
+        )
+        myFixture.type("\n")
+        myFixture.checkResult(
+            """
+            \begin{itemize}
+                \item {This sentence is 
+                <caret>broken}
+            \end{itemize}
+            """.trimIndent()
+        )
+    }
+
     fun `test nested enumeration with prefix`() {
         myFixture.configureByText(
             LatexFileType,