DmitriK / darkContrast

Firefox addon that fixes low contrast text when using dark desktop theme
MIT License
63 stars 3 forks source link

"To" and "Subject" fields for Google Inbox mail (not Gmail) are wonky #27

Closed jaybosamiya closed 6 years ago

jaybosamiya commented 7 years ago

With both the extension enabled as well as the extension disabled, the To and Subject fields look wonky:

screenshot from 2017-09-19 15-54-42

Steps to reproduce:

Please do let me know if I can provide any other useful information, or what I might be able to do to fix this. Thanks a lot :) Also, I love that this project has basically fixed almost every other problem I had on most websites. It's only this minor annoyance now (it is not unreadable, but just is wrong; no such issue on Gmail's compose window).

DmitriK commented 7 years ago

tl:dr; The force-standard colors feature should be able fix this (via menu, and via the override lists to make it persist)

It looks like both of those inputs do not have any fore- or background colors defined by google's CSS, so it's using the native (Arc-Dark in this case) input styling. Because readability is good, the extensions treats it as OK and does nothing. This is by design, as I made this extension with the express desire for native input elements to remain native as long as contrast was OK, as opposed to the user-script solutions that just force dark-on-light text for all inputs.

That said, there are cases such as this where things don't look so hot, so I ended up adding the 'force standard colors' option to emulate the aforementioned user script solution. It should passively set the inputs to be black foreground, white background (which may not match the site colors exactly, but the extension is in no way context-aware enough to guess the right colors), but it should look better than it does now.

jaybosamiya commented 7 years ago

Force-standard-colors feature still is unable to fix it. In fact, it stays the same, despite having the standard icon show up and it tread it as "standard".

Running an "Inspect Element" on each of the different parts, it seems that the issue happens because both the "To" and "Subject" fields are <input> tags but the "From" is a <button> and the "say something" is a <div>.

I can now understand why any kind of fix on these would not work unless it were to be made either very context aware, or had some kind of hardcoding for the website, so feel free to close this issue if you too are convinced that this can't be fixed in the extension.

DmitriK commented 7 years ago

Not only is from a button, but it has a color set from the site CSS (q6 class, RGB(33,33,33) in my view), so the extension fixes it. It's the lack of styling for the input fields that is the root of the problem. Looking at my code again I see that 'force standard colors' does not force input elements to standard colors; it only forces regular elements for those who are running with 'use system colors' on. Thinking about it a little bit now, I'm not sure if I actually want that to be the case, so I'm going to leave this open for now until I can decide what the correct solution will be.

DmitriK commented 6 years ago

I've release a new version that appears to fix the issue on my-end. Can you test if this is still a problem.

jaybosamiya commented 6 years ago

Partially, yes. :) Partially, no. :(

It works great if I manually click on the "Force standard colors" button, but if I add it into the options for the same, then it doesn't seem to work all that well (i.e., it shows "std" in the icon, but it still has the issue). I did try increasing delay, but still it doesn't seem to be fixed even with a longer delay.

DmitriK commented 6 years ago

Herp derp, my mistake, I was loading the standard gmail interface in my hurry to check which bugs the new version fixed. Now that I've pulled the right site up, I do see the issue with the list. I'll keep at it.

jaybosamiya commented 6 years ago

No problemo :) I love the effort you've put into this extension and I can work with manually clicking to standard until you are able to fix it. Thanks for all the work you do to make our machines look nicer :)

On Mon, Dec 18, 2017 at 11:03 PM Dmitri Kourennyi notifications@github.com wrote:

Herp derp, my mistake, I was loading the standard gmail interface in my hurry to check which bugs the new version fixed. Now that I've pulled the right site up, I do see the issue with the list. I'll keep at it.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/DmitriK/darkContrast/issues/27#issuecomment-352633754, or mute the thread https://github.com/notifications/unsubscribe-auth/AFa5fj0RzicgwvL3Ut4XEvkhzblu4BcZks5tBzWagaJpZM4Pc742 .

--

Sent from my iPhone

jaybosamiya commented 6 years ago

Hi, not sure what you updated, but it suddenly seems to work perfectly in Inbox, using the "Force standard colors" override (and I no longer need to click "Force standard colors". Thanks a lot for looking into this and fixing it :)

Feel free to close this if you think the changes made actually were supposed to affect this and have fixed it :)

DmitriK commented 6 years ago

Yes, I made sure this got fixed in the latest overhaul. I generally wait a few days to make sure extension updates have gone out on AMO before asking people to test bugs. It looks like I forgot to respond to this one. Thanks for confirming it works.