Fixed log scroll view not snapping to bottom intuitively.

Author: DavidF-Dev

CHANGELOG.md

@@ -17,6 +17,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Changed DevConsole.InvokeCoroutine method to return the Coroutine instance.
 - Fixed Unity logs from other threads not showing in the developer console log.
 - Fixed generic or nullable parameter types not having their type displayed nicely.
+- Fixed log scroll view not snapping to bottom intuitively.
 
 ## [0.2.1-alpha] - 2021-08-09
 - Added documentation.

Runtime/DevConsoleMono.cs

@@ -155,6 +155,7 @@ internal sealed class DevConsoleMono : MonoBehaviour
         private readonly TextGenerator _textGenerator = new TextGenerator();
         private int _vertexCount = 0;
         private int _initLogTextSize = 0;
+        private bool _pretendScrollAtBottom = false;
 
         #endregion
 
@@ -382,6 +383,7 @@ internal void ClearConsole()
             ClearLogFields();
             _vertexCount = 0;
             StoredLogText = ClearLogText;
+            _pretendScrollAtBottom = true;
         }
 
         internal void ResetConsole()
@@ -397,7 +399,7 @@ internal void SubmitInput()
         {
             if (!string.IsNullOrWhiteSpace(InputText) && RunCommand(InputText))
             {
-                ScrollToBottomAtEndOfFrame();
+                ScrollTo(0.0f);
             }
 
             InputText = string.Empty;
@@ -887,9 +889,12 @@ private void LateUpdate()
             // Process the stored logs, displaying them to the console
             if (StoredLogText != string.Empty)
             {
+                bool scrollToBottom = _logScrollView.verticalNormalizedPosition < 0f || Mathf.Approximately(_logScrollView.verticalNormalizedPosition, 0f);
+                float scrollTo = scrollToBottom || _pretendScrollAtBottom ? 0.0f : _logScrollView.verticalNormalizedPosition;
                 string logText = string.Copy(StoredLogText);
                 StoredLogText = string.Empty;
                 ProcessLogText(logText);
+                ScrollTo(scrollTo);
             }
 
             // Check if the developer console toggle key was pressed
@@ -2635,17 +2640,21 @@ private void RebuildLayout()
             LayoutRebuilder.ForceRebuildLayoutImmediate(_logContentTransform);
         }
 
-        private void ScrollToBottomAtEndOfFrame()
+        private void ScrollTo(float vertical)
         {
-            IEnumerator ScrollToBottomCoroutine()
+            IEnumerator ScrollTo()
             {
                 yield return new WaitForEndOfFrame();
-                _logScrollView.verticalNormalizedPosition = 0f;
-                _logScrollView.CalculateLayoutInputVertical();
+                _logScrollView.verticalNormalizedPosition = vertical;
+                LayoutRebuilder.ForceRebuildLayoutImmediate((RectTransform)_logScrollView.transform);
+                if (_pretendScrollAtBottom && (_logScrollView.verticalNormalizedPosition < 0f || Mathf.Approximately(_logScrollView.verticalNormalizedPosition, 0f)))
+                {
+                    _pretendScrollAtBottom = false;
+                }
             }
 
             // Start the coroutine that snaps the scroll view at the end of the frame
-            StartCoroutine(ScrollToBottomCoroutine());
+            StartCoroutine(ScrollTo());
         }
 
         #endregion