mozilla / voicefill

A WebExtension To Add Speech To Text Support to Web Pages
Mozilla Public License 2.0
91 stars 29 forks source link

Voice Fill icon from the toolbar is barely visible with Dark Theme enabled #229

Closed SoftVision-CarmenFat closed 5 years ago

SoftVision-CarmenFat commented 6 years ago

[Affected versions]:

[Affected Platforms]:

[Prerequisites]:

[Steps to reproduce]:

  1. Open the browser with the profile from prerequisites.
  2. Observe the "Voice Fill" toolbar button.

[Expected result]:

[Actual result]:

[Notes]:

dark theme

andrenatal commented 6 years ago

@johngruen what should we use here?

johngruen commented 6 years ago

should now be resolved

SoftVision-CarmenFat commented 6 years ago

This issue is still reproducible with the pre-release version (v.1.3.5).

andrenatal commented 6 years ago

Carmen, I could not reproduce this issue using the latest xpi. Please, check the screen cast here: https://www.dropbox.com/s/82spwwbtv6nsoyd/icon-issue.mov?dl=0

SoftVision-CarmenFat commented 6 years ago

After digging more into this issue, I found out that this is reproducible only with Windows and Ubuntu. On macOS, the icon is correctly displayed in the toolbar when Dark theme is enabled.

andrenatal commented 6 years ago

Hi Carmen, could you please test this again with 1.3.6?

SoftVision-CarmenFat commented 6 years ago

Tested this with Voice Fill v.1.3.6 and I can confirm that the issue is fixed: the icon is now correctly displayed across platforms when Dark theme is enabled.

Softvision-CristinaBadescu commented 6 years ago

Indeed, this issue has been fixed but not entirely. It seems that the default buttons color on dark theme is BEBEBE while this fix makes "Voice Fill" button D1D1D2, which is a little bit lighter. Considering this, I'm going to reopen this issue. @andrenatal can you please take a look at this? Below is a screenshot with the colors codes for the buttons: button colors