fallback colours when settings not available in Fonts and Colors

Author: tonyhallett

SharedProject/Impl/CoverageColorProvider.cs

@@ -14,29 +14,37 @@ namespace FineCodeCoverage.Impl
     internal class CoverageColorProvider : ICoverageColoursProvider, ICoverageColours
     {
         private readonly IVsFontAndColorStorage fontAndColorStorage;
-        private Guid categoryWithCoverage = Guid.Parse("ff349800-ea43-46c1-8c98-878e78f46501");
+        private readonly ILogger logger;
         private readonly uint storeFlags = (uint)(__FCSTORAGEFLAGS.FCSF_READONLY | __FCSTORAGEFLAGS.FCSF_LOADDEFAULTS | __FCSTORAGEFLAGS.FCSF_NOAUTOCOLORS | __FCSTORAGEFLAGS.FCSF_PROPAGATECHANGES);
-        private System.Windows.Media.Color defaultCoverageTouchedArea = System.Windows.Media.Colors.Green;
-        private System.Windows.Media.Color defaultCoverageNotTouchedArea = System.Windows.Media.Colors.Red;
-        private System.Windows.Media.Color defaultCoveragePartiallyTouchedArea = System.Windows.Media.Color.FromRgb(255, 165, 0);
+        private readonly System.Windows.Media.Color defaultCoverageTouchedArea = System.Windows.Media.Colors.Green;
+        private readonly System.Windows.Media.Color defaultCoverageNotTouchedArea = System.Windows.Media.Colors.Red;
+        private readonly System.Windows.Media.Color defaultCoveragePartiallyTouchedArea = System.Windows.Media.Color.FromRgb(255, 165, 0);
+        private Guid categoryWithCoverage = Guid.Parse("ff349800-ea43-46c1-8c98-878e78f46501");
+        private bool coverageColoursFromFontsAndColours;
+        private bool dirty = true;
+        private bool canUseFontsAndColours = true;
         public System.Windows.Media.Color CoverageTouchedArea { get; set; }
 
         public System.Windows.Media.Color CoverageNotTouchedArea { get; set; }
 
         public System.Windows.Media.Color CoveragePartiallyTouchedArea { get; set; }
 
-        private bool coverageColoursFromFontsAndColours;
-        private bool dirty = true;
+        
 
         [ImportingConstructor]
-        public CoverageColorProvider([Import(typeof(SVsServiceProvider))] IServiceProvider serviceProvider, IAppOptionsProvider appOptionsProvider)
+        public CoverageColorProvider(
+            [Import(typeof(SVsServiceProvider))] IServiceProvider serviceProvider, 
+            IAppOptionsProvider appOptionsProvider,
+            ILogger logger
+        )
         {
             ThreadHelper.ThrowIfNotOnUIThread();
             fontAndColorStorage = (IVsFontAndColorStorage)serviceProvider.GetService(typeof(IVsFontAndColorStorage));
             Assumes.Present(fontAndColorStorage);
             coverageColoursFromFontsAndColours = appOptionsProvider.Get().CoverageColoursFromFontsAndColours;
             UseDefaultColoursIfNotFontsAndColours();
             appOptionsProvider.OptionsChanged += AppOptionsProvider_OptionsChanged;
+            this.logger = logger;
         }
 
         private void AppOptionsProvider_OptionsChanged(IAppOptions appOptions)
@@ -50,15 +58,20 @@ private void UseDefaultColoursIfNotFontsAndColours()
         {
             if (!coverageColoursFromFontsAndColours)
             {
-                CoverageTouchedArea = defaultCoverageTouchedArea;
-                CoverageNotTouchedArea = defaultCoverageNotTouchedArea;
-                CoveragePartiallyTouchedArea = defaultCoveragePartiallyTouchedArea;
+                UseDefaultColours();
             }
         }
 
+        private void UseDefaultColours()
+        {
+            CoverageTouchedArea = defaultCoverageTouchedArea;
+            CoverageNotTouchedArea = defaultCoverageNotTouchedArea;
+            CoveragePartiallyTouchedArea = defaultCoveragePartiallyTouchedArea;
+        }
+
         public async Task PrepareAsync()
         {
-            if (coverageColoursFromFontsAndColours && dirty)
+            if (coverageColoursFromFontsAndColours && canUseFontsAndColours && dirty)
             {
                 await UpdateColoursFromFontsAndColorsAsync();
             }
@@ -80,18 +93,22 @@ System.Windows.Media.Color GetColor(string displayName)
                     {
                         return ParseColor(touchAreaInfo[0].crBackground);
                     }
-                    throw new Exception($"Failed to get color - error {getItemSuccess}");
+                    throw new NotSupportedException($"{getItemSuccess}");
+                }
+                try
+                {
+                    // https://developercommunity.visualstudio.com/t/fonts-and-colors-coverage-settings-available-in-vs/1683898
+                    CoverageTouchedArea = GetColor("Coverage Touched Area");
+                    CoverageNotTouchedArea = GetColor("Coverage Not Touched Area");
+                    CoveragePartiallyTouchedArea = GetColor("Coverage Partially Touched Area");
+                }catch(NotSupportedException)
+                {
+                    logger.Log("No coverage settings available from Fonts and Colors");
                 }
-
-                CoverageTouchedArea = GetColor("Coverage Touched Area");
-                CoverageNotTouchedArea = GetColor("Coverage Not Touched Area");
-                CoveragePartiallyTouchedArea = GetColor("Coverage Partially Touched Area");
-            }
-            else
-            {
-                //throw ?
             }
-            fontAndColorStorage.CloseCategory(); // only for success ?
+            canUseFontsAndColours = false;
+            UseDefaultColours();
+            fontAndColorStorage.CloseCategory();
         }
 
         private System.Windows.Media.Color ParseColor(uint color)