Fixed
Status Update
Comments
km...@google.com <km...@google.com>
hu...@google.com <hu...@google.com>
da...@google.com <da...@google.com> #2
Agree with changing this to be more readable.
In the mocks I've been carrying forward, it calls for something along the lines of <device name> | <Test Result in semibold and color> <string describing error in normal font>. See screenshot.
pa...@google.com <pa...@google.com>
an...@google.com <an...@google.com> #3
Thank you for your patience while our engineering team worked to resolve this issue. A fix for this issue is now available in:
- Android Studio Meerkat | 2024.3.1 Canary 1
- Android Gradle Plugin 8.9.0-alpha01
We encourage you to try the latest update.
If you notice further issues or have questions, please file a new bug report.
Thank you for taking the time to submit feedback — we really appreciate it!
Description
To me, this looks a little "noisy"; the different font sizes makes it less visually clean. If we need to emphasize this line, I think we should use something other than font size, such as perhaps a background highlight (or possibly font weight).
I found the code which does this. I then tried this patch:
```
diff --git a/android/src/com/android/tools/idea/testartifacts/instrumented/testsuite/view/DetailsViewContentView.kt b/android/src/com/android/tools/idea/testartifacts/instrumented/testsuite/view/DetailsViewContentView.kt
index e538a794dd3..abb305662dd 100644
--- a/android/src/com/android/tools/idea/testartifacts/instrumented/testsuite/view/DetailsViewContentView.kt
+++ b/android/src/com/android/tools/idea/testartifacts/instrumented/testsuite/view/DetailsViewContentView.kt
@@ -243,7 +243,7 @@ class DetailsViewContentView(parentDisposable: Disposable, private val project:
else {
myTestResultLabel.text = String.format(
Locale.US,
- "<html><font size='+1'>%s</font><br><font color='%s'>Failed</font> on %s</html>",
+ "<html><font>%s</font><br><font color='%s'>Failed</font> on %s</html>",
errorMessage.htmlEscape(),
ColorUtil.toHtmlColor(statusColor),
device.getName().htmlEscape())
```
The result is in the second screenshot. To me this is a bit better, but I guess it's subjective so thought I'd submit it as a bugreport for UX to consider rather than just uploading a patch :-)