-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Dark icon blends with background in Dark Mode #4
Comments
Hi @akaustav , thanks for opening this issue and the pictures. Yes, there should be a setting to change the toolbar icon color in URLI's Options page. As far as I know, Chrome doesn't offer an API to do this beyond the toolbar icon unfortunately, so the extension view is stuck with the default. In the next update, the default icon will be a lot darker to help make it stand out against the more common dark gray themes, though it won't be a perfect solution. I tried adding some color but just didn't like the way it looked. Thanks for the feedback! |
Thanks @roysix for the quick response. I have not authored a Google Chrome extension yet. So, pardon my ignorance if the below suggestions are invalid. There seems to be a way using CSS to detect whether the user is in dark mode or not. These SO answers might be helpful: |
Hi @akaustav thank you for taking the time. I wasn't expecting a pull request! Unfortunately, URLI is a no-permission extension by default, so requiring <all_urls> will upset a lot of users who are using it permission-free. It would also definitely cause extra difficulty and wait times in getting approval from the Chrome Web Store review team. Reviews are already taking upwards of 3+ weeks due to the current situation. Adding a content script that loads on every page request to do the check also seems inefficient. So it's a deal breaker unfortunately. :( However, it looks like we can use the Thanks again for bringing up this issue! |
Hi @roysix, I have been developing JavaScript applications for the better part of a decade now. So, having absolutely zero knowledge around Chrome extension authoring before this, and with some "Google-Fu", I could understand most of the SO answer and your codebase. Hence the PR. Meanwhile, I understand your dilemma about the time vs effort vs complexity vs worthiness of this issue. It would help older users of your extension. Take my scenario, for instance. I had installed your extension in my Google Chrome around 10 years ago, I think. At that moment, I might have received the options popup/window upon installation (hard to recall - If I saw the options popup, I must have accepted the default settings). Then I turned on auto-sync in Google Chrome. Since then, all my Chrome extensions (including yours) sync across every device (which allows Chrome extensions). Dark mode was enabled in late 2019 (in Windows 10 September, 2019 release and supported by Chrome v76, I think). I turned on Dark mode on Windows 10 around that time. And since then your extension had disappeared in my Chrome. I thought I must have uninstalled or disabled it. It was until recently, that I needed your extension and found I still have it installed and enabled, just not visible because the icon blended in perfectly with the background. Hence, I had opened this issue. I respect your decision on which direction you want your extension to go. Unrelated: While going thru your codebase I see a blend of old ES5 ( |
Wow! Thanks for having URLI such a long while ago! Thank you also for the detailed reply and bringing up your situation about having the extension installed already. I hadn't considered that scenario. Maybe the simplest solution is to just have one icon that works across all themes? If we stick with the multiple icons, I will definitely at least use your Thanks for taking a good look at URLI's code! You're right. I've been trying to switch over to using promises and Right now, the focus is on getting a new extension called Infy Scroll out before the end of this month, which is the successor to URLI in a way, but uses infinite scrolling instead of tab updating for incrementing. Since you know JavaScript, if you're interested, you can install any unpacked extension and test your code changes. For example, you could clone/download the URLI zip or Infy Scroll zip here on github and load the Thanks again @akaustav! |
The URLI extension is super helpful. I just wish the icon on the URLI extension button (next to address bar) did not blend in with the background when using Dark mode in Google Chrome. Here is how it looks like in Microsoft Windows 10:
Non-Hover state:
Hover state:
Google Chrome Extensions view (
chrome://extensions/
):The text was updated successfully, but these errors were encountered: