From 7126d13b6f7e01052aa37cad26181693077d48b4 Mon Sep 17 00:00:00 2001 From: marhali Date: Wed, 24 Aug 2022 21:15:46 +0200 Subject: [PATCH] change color to increase contrast in light themes Resolves #157 --- CHANGELOG.md | 1 + .../java/de/marhali/easyi18n/tabs/mapper/TreeModelMapper.java | 4 ++-- .../java/de/marhali/easyi18n/tabs/renderer/TableRenderer.java | 2 +- 3 files changed, 4 insertions(+), 3 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index fc09032..9b55e4f 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -5,6 +5,7 @@ ## [Unreleased] ### Changed - Documentation provider better focuses on the actual translation part +- Color duplicate translation values orange to increase contrast in light themes ### Fixed - Unintended overwrite of existing folding regions diff --git a/src/main/java/de/marhali/easyi18n/tabs/mapper/TreeModelMapper.java b/src/main/java/de/marhali/easyi18n/tabs/mapper/TreeModelMapper.java index 1baf35d..ff74e12 100644 --- a/src/main/java/de/marhali/easyi18n/tabs/mapper/TreeModelMapper.java +++ b/src/main/java/de/marhali/easyi18n/tabs/mapper/TreeModelMapper.java @@ -78,8 +78,8 @@ public class TreeModelMapper extends DefaultTreeModel { data.setForcedTextForeground(JBColor.RED); color = JBColor.RED; } else if(TranslationUtil.hasDuplicates(new Translation(keyPath, childTranslationNode.getValue()), this.data)) { - data.setForcedTextForeground(JBColor.YELLOW); - color = JBColor.YELLOW; + data.setForcedTextForeground(JBColor.ORANGE); + color = JBColor.ORANGE; } parent.add(new DefaultMutableTreeNode(data)); diff --git a/src/main/java/de/marhali/easyi18n/tabs/renderer/TableRenderer.java b/src/main/java/de/marhali/easyi18n/tabs/renderer/TableRenderer.java index a9e2e7c..ec6bc68 100644 --- a/src/main/java/de/marhali/easyi18n/tabs/renderer/TableRenderer.java +++ b/src/main/java/de/marhali/easyi18n/tabs/renderer/TableRenderer.java @@ -28,7 +28,7 @@ public class TableRenderer extends DefaultTableCellRenderer { if(missesValues(row, table)) { component.setForeground(JBColor.RED); } else if(hasDuplicates(row, table)) { - component.setForeground(JBColor.YELLOW); + component.setForeground(JBColor.ORANGE); } return component;