0.0.5: Fixed spacing bug

Author: sequenter

.idea/modules.xml

@@ -130,16 +130,20 @@ private void handleAttributes(Context context, AttributeSet attrs){
     }
 
     private void handleHelperVisibility(CharSequence text){
-        if(text.length() == 0 && !_keepHelperSpacing && _helperView.getVisibility() != View.GONE){
-            _helperView.setVisibility(View.GONE);
-        } else if (_helperView.getVisibility() != View.VISIBLE){
+        if(text.length() == 0 && !_keepHelperSpacing){
+            if(_helperView.getVisibility() != View.GONE){
+                _helperView.setVisibility(View.GONE);
+            }
+        } else if (_helperView.getVisibility() != View.VISIBLE) {
             _helperView.setVisibility(View.VISIBLE);
         }
     }
 
     private void handleLabelVisibility(){
-        if(_labelText.length() == 0 && !_keepLabelSpacing && _labelView.getVisibility() != View.GONE){
-            _labelView.setVisibility(View.GONE);
+        if(_labelText.length() == 0 && !_keepLabelSpacing){
+            if(_labelView.getVisibility() != View.GONE) {
+                _labelView.setVisibility(View.GONE);
+            }
         } else if(_labelView.getVisibility() != View.VISIBLE){
             _labelView.setVisibility(View.VISIBLE);
         }
@@ -172,6 +176,7 @@ public void setError(@Nullable CharSequence error){
             _isErrord = false;
 
             _labelView.setTextColor(_labelTextColour);
+            _contentView.setTextColor(_contentTextColour);
             _helperView.setTextColor(_helperTextColour);
             _helperView.setText(_helperText);
 
@@ -181,6 +186,7 @@ public void setError(@Nullable CharSequence error){
             _isErrord = true;
 
             _labelView.setTextColor(_errorTextColour);
+            _contentView.setTextColor(_errorTextColour);
             _helperView.setTextColor(_errorTextColour);
             _helperView.setText(error);
 
@@ -329,4 +335,7 @@ public CharSequence getHelperText(){
 
     @SuppressWarnings("unused")
     public Boolean getSingleLine(){ return _singleLine; }
+
+    @SuppressWarnings("unused")
+    public Boolean getError(){ return _isErrord; }
 }

materialtextview/src/main/java/uk/co/onemandan/materialtextview/MaterialTextView.java

@@ -9,26 +9,34 @@
 
 public class Sample extends AppCompatActivity {
 
-    private boolean _error = false;
-
     @Override
     protected void onCreate(Bundle savedInstanceState) {
         super.onCreate(savedInstanceState);
         setContentView(R.layout.activity_sample);
 
         final MaterialTextView materialTextView = findViewById(R.id.mtv_sample);
+        final MaterialTextView materialTextView2 = findViewById(R.id.mtv_sample2);
 
         materialTextView.setOnClickListener(new View.OnClickListener() {
             @Override
             public void onClick(View v) {
-                _error = !_error;
-
-                if(_error){
+                if(!materialTextView.getError()){
                     materialTextView.setError("Error!");
                 } else {
                     materialTextView.setError(null);
                 }
             }
         });
+
+        materialTextView2.setOnClickListener(new View.OnClickListener() {
+            @Override
+            public void onClick(View v) {
+                if(!materialTextView2.getError()){
+                    materialTextView2.setError("");
+                } else {
+                    materialTextView2.setError(null);
+                }
+            }
+        });
     }
 }

sample/src/main/java/uk/co/onemandan/materialtextviewsample/Sample.java

@@ -54,14 +54,14 @@
         android:orientation="horizontal">
 
         <uk.co.onemandan.materialtextview.MaterialTextView
+            android:id="@+id/mtv_sample2"
             android:layout_width="0dp"
             android:layout_height="wrap_content"
             android:layout_weight="1"
             android:layout_marginEnd="8dp"
             android:layout_marginRight="8dp"
             app:mtv_labelText="Label Left"
-            app:mtv_contentText="Content Leftttttttt"
-            app:mtv_singleLine="true"/>
+            app:mtv_contentText="Content Left" />
 
         <uk.co.onemandan.materialtextview.MaterialTextView
             android:layout_width="0dp"
@@ -74,4 +74,16 @@
 
     </LinearLayout>
 
+    <uk.co.onemandan.materialtextview.MaterialTextView
+        android:layout_width="match_parent"
+        android:layout_height="wrap_content"
+        android:layout_marginStart="@dimen/margin_large"
+        android:layout_marginLeft="@dimen/margin_large"
+        android:layout_marginEnd="@dimen/margin_large"
+        android:layout_marginRight="@dimen/margin_large"
+        android:layout_marginTop="@dimen/margin_large"
+        app:mtv_labelText="Single Line"
+        app:mtv_contentText="The Quick Brown Fox Jumps OverThe Lazy Dog"
+        app:mtv_singleLine="true"/>
+
 </LinearLayout>