Issue #42: Update calculating the padding of line number to solve ove…

…rlap problem
AmrDeveloper · Mar 27, 2023 · d91c2ac · d91c2ac
1 parent e002f3d
commit d91c2ac
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/codeview/src/main/java/com/amrdeveloper/codeview/CodeView.java b/codeview/src/main/java/com/amrdeveloper/codeview/CodeView.java
@@ -186,8 +186,9 @@ protected void onDraw(Canvas canvas) {
                     }
                 }
 
-                // Calculate padding depending on current line number
-                final int paddingLeft = 50 + (int) Math.log10(lineCount) * 10;
+                // Calculate padding depending on number of lines and the size of line number text
+                final int lineNumberDigitCount = (int) (Math.log10(lineCount) + 1);
+                final int paddingLeft = (int) (lineNumberPaint.getTextSize() * lineNumberDigitCount);
                 setPadding(paddingLeft, getPaddingTop(), getPaddingRight(), getPaddingBottom());
             }
         }
@@ -638,7 +639,7 @@ public void setLineNumberTextColor(int color) {
 
     /**
      * Modify the line number text size
-     * @param size The new size value
+     * @param size The new text size in pixel units
      * @since 1.1.0
      */
     public void setLineNumberTextSize(float size) {