add TODOs for #139, #254, #233, #228, #200, #172, #215

acejump · Nov 14, 2018 · 27f2754 · 27f2754
1 parent d911528
commit 27f2754
Show file tree

Hide file tree

Showing 7 changed files with 10 additions and 1 deletion.
diff --git a/src/main/kotlin/org/acejump/config/AceConfig.kt b/src/main/kotlin/org/acejump/config/AceConfig.kt
@@ -8,6 +8,8 @@ import com.intellij.openapi.options.Configurable
 
 /* Persists the state of the AceJump IDE settings across IDE restarts.
  * https://www.jetbrains.org/intellij/sdk/docs/basics/persisting_state_of_components.html
+ *
+ * TODO: https://github.com/acejump/AceJump/issues/254
  */
 
 @State(name = "AceConfig", storages = [(Storage("AceJump.xml"))])

diff --git a/src/main/kotlin/org/acejump/config/AceSettings.kt b/src/main/kotlin/org/acejump/config/AceSettings.kt
@@ -5,6 +5,7 @@ import java.awt.Color
 import java.awt.Color.*
 import kotlin.reflect.KProperty
 
+// TODO: https://github.com/acejump/AceJump/issues/215
 data class AceSettings(var layout: KeyLayout = KeyLayout.QWERTY,
                        var allowedChars: String = layout.allChars(),
                        // These must be primitives in order to be serializable

diff --git a/src/main/kotlin/org/acejump/label/Pattern.kt b/src/main/kotlin/org/acejump/label/Pattern.kt
@@ -57,6 +57,7 @@ enum class Pattern(val string: String) {
         get() = when (this) {
           QWERTY -> "fjghdkslavncmbxzrutyeiwoqp5849673210"
           // TODO: Prioritize keys by ease of reach for other keyboards
+          // TODO: https://github.com/acejump/AceJump/issues/172
           WORKMN -> "fjghdkslavncmbxzrutyeiwoqp5849673210"
           COLEMK -> "fjghdkslavncmbxzrutyeiwoqp5849673210"
           DVORAK -> "fjghdkslavncmbxzrutyeiwoqp5849673210"

diff --git a/src/main/kotlin/org/acejump/view/Canvas.kt b/src/main/kotlin/org/acejump/view/Canvas.kt
@@ -45,6 +45,7 @@ object Canvas : JComponent(), Resettable {
 
   private fun Editor.storeBounds() {
     viewBounds = getView()
+    // TODO: Fix reference, cf. https://github.com/acejump/AceJump/issues/200
     this::offsetToLogicalPosition.let {
       logger.info("View bounds: $viewBounds (lines " +
         "${it(viewBounds.first).line}..${it(viewBounds.last).line})")

diff --git a/src/main/kotlin/org/acejump/view/Marker.kt b/src/main/kotlin/org/acejump/view/Marker.kt
@@ -132,6 +132,9 @@ class Marker : CustomHighlighterRenderer {
     drawString(tag!!.toUpperCase(), tagPosition!!.x, tagPosition.y + fontHeight)
   }
 
+  // TODO: Fix tag alignment and visibility issues
+  // https://github.com/acejump/AceJump/issues/233
+  // https://github.com/acejump/AceJump/issues/228
   private fun String.alignTag(canvas: Canvas): Point {
     val x = tagPoint.x + fontWidth
 //    val top = Point(x - fontWidth, y - fontHeight)

diff --git a/src/main/kotlin/org/acejump/view/Model.kt b/src/main/kotlin/org/acejump/view/Model.kt
@@ -16,7 +16,7 @@ import java.awt.Font.BOLD
 /**
  * Data holder for all settings and IDE components needed by AceJump.
  *
- * TODO: Integrate this class with AceSettings.
+ * TODO: Integrate this class with [org.acejump.config.AceSettings]
  */
 
 object Model {

diff --git a/src/test/kotlin/AceTest.kt b/src/test/kotlin/AceTest.kt
@@ -6,6 +6,7 @@ import com.intellij.testFramework.EditorActionTestCase
 class AceTest : EditorActionTestCase() {
   override fun getActionId() = "AceAction"
 
+  // TODO: Improve test coverage, https://github.com/acejump/AceJump/issues/139
   fun testSomething() {
     assert(true)
   }