Seems to no longer mark messages as read.

[kris-redfall]

Not sure if its just me but no longer seems to mark messages as read when reading them.

[Goctionni]

Hey there,

Could you tell me whether this is in Chrome or Firefox?

[ralyodio]

i noticed this as well in chrome. Is this addon availalbe for firefox? I couldn't find it.

[ralyodio]

Uncaught TypeError: Cannot read property 'innerText' of null notification_icons.js:93 Uncaught TypeError: Cannot read property 'innerText' of null

i get this sometime before/during or after replying to an unread message.

[Goctionni]

Bah, unfortunately that is an entirely different / unrelated error. I was hoping you would get an error in console that was specific to this.

I am not sure what could be causing this at the moment :/
For what it's worth, the next version has a lot of structural changes in how things are saved; it might fix this issue. (Cant say for certain because I do not know what is causing this)

[ralyodio]

I have not tested this but I think that actually is the error:

replace:

        var tbMailCount = document.querySelector('#tb-mailCount');

with:

        var tbMailCount = document.querySelectorAll('.new-message')[1];

[Goctionni]

The problem there was that I assumed Mod toolbox was fully initialized when it adds its class to , which turns out is not the case. I have already got a fix for this locally, just haven't committed to git in a while.

However, that is a separate issue from RIR not marking messages as read :s

[ralyodio]

but if you look at that block it relies on that number being set.

[Goctionni]

I'm not sure what block you're referring to. I know there are errors that are caused by that snippet of code (the bug happened as a result of a Mod Toolbox update that changed how that extension works).

However it is unrelated to this particular issue (ie: it is unrelated to Inbox Revamp not marking messages as read). For what it's worth though, I've just committed a new version that has the latest changes, unfortunately this version is not yet ready for actual release. It is missing features, some things are broken, etc. But this particular console error is fixed.