mitro-co / mitro

Repository for all Mitro client & server code
GNU General Public License v3.0
1.64k stars 242 forks source link

Firefox 31-33 in Linux mint: font/login problem #40

Open denadai2 opened 10 years ago

denadai2 commented 10 years ago

I have this strange error in the Firefox extension. First of all the font is wrong, it seems that the extension content is not loaded and I can't see my secrets inside it.

ad

I have Linux Mint 17 "Qiana" - Xfce (64-bit) and Firefox 31

denadai2 commented 10 years ago

TypeError: document.getElementById(...) is null Stack trace: updateLoginState/</</</<@resource://gre/modules/addons/XPIProvider.jsm -> jar:file:///home/marco/.mozilla/firefox/mwad0hks.default/extensions/mitro-login-manager@jetpack.xpi!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/loader/sandbox.js -> resource://mitro-login-manager-at-jetpack/mitro-login-manager/data/js/popup.js:4:335 Client.prototype.setMethod/this[a]</<@resource://gre/modules/addons/XPIProvider.jsm -> jar:file:///home/marco/.mozilla/firefox/mwad0hks.default/extensions/mitro-login-manager@jetpack.xpi!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/loader/sandbox.js -> resource://mitro-login-manager-at-jetpack/mitro-login-manager/data/js/client.js:13:299 Client/this.processIncoming@resource://gre/modules/addons/XPIProvider.jsm -> jar:file:///home/marco/.mozilla/firefox/mwad0hks.default/extensions/mitro-login-manager@jetpack.xpi!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/loader/sandbox.js -> resource://mitro-login-manager-at-jetpack/mitro-login-manager/data/js/client.js:2:226 ExtensionHelper/this.bindClient/<@resource://gre/modules/addons/XPIProvider.jsm -> jar:file:///home/marco/.mozilla/firefox/mwad0hks.default/extensions/mitro-login-manager@jetpack.xpi!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/loader/sandbox.js -> resource://mitro-login-manager-at-jetpack/mitro-login-manager/data/js/helpers.js:7:411 onEvent@resource://gre/modules/addons/XPIProvider.jsm -> jar:file:///home/marco/.mozilla/firefox/mwad0hks.default/extensions/mitro-login-manager@jetpack.xpi!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/loader/sandbox.js -> resource://gre/modules/commonjs/sdk/content/content-worker.js:45:9 onChromeEvent@resource://gre/modules/addons/XPIProvider.jsm -> jar:file:///home/marco/.mozilla/firefox/mwad0hks.default/extensions/mitro-login-manager@jetpack.xpi!/bootstrap.js -> resource://gre/modules/commonjs/toolkit/loader.js -> resource://gre/modules/commonjs/sdk/loader/sandbox.js -> resource://gre/modules/commonjs/sdk/content/content-worker.js:96:9 timers.js:43

jerbob92 commented 10 years ago

Duplicate of #4

tssm commented 10 years ago

I have the same problem on Arch Linux, Firefox 31, but on Mac OS X it works well.

evanj commented 10 years ago

Oh, is this UI issue the same issue as #44? That issue doesn't have any screenshots, and I can't tell from the description. If this is the same, this could be an excellent clue for how to reproduce this, which I have been unable to do so far.

cincodenada commented 10 years ago

This does seem to be the same issue as #4 - I just ran into this myself, and my stack trace is identical to @denadai2's (I ran a diff, and literally the only differences were our usernames and randomly-generated bits in the profile directories). It's also very similar to @diracdeltas' stack trace in #4 - again, diffed them, and the only additional difference is that it references line 307 in popup.js instead of 335 - presumably due to some unrelated code changes above it between then and now.

There is some strange flashing of content, it looks like it attempts to resize the popup to the correct format, and then fails and falls back to just embedding the full page. I'm not sure if it is helpful at all, but just in case I made a quick (low-quality so it could handle the framerate) video/GIF of what it looks like: femininecompetentindianpangolin

It's unfortunately difficult to see, but it initially shows what was previously there, flashes a smaller window with a tiled background that seems to fit, then expands back out to the full page. I took a few screenshots (scaled down to fit here better) to demonstrate:

Initially loads full pageFlashes a well-scaled background for a momentThen this before going back like to the intial opening state

denadai2 commented 10 years ago

exactly @cincodenada

denadai2 commented 10 years ago

@evanj you can reproduce the error with this live cd: http://www.linuxmint.com/edition.php?id=158

ghost commented 10 years ago

Firefox 31.0 under windows 7 suffers the same issue too.

denadai2 commented 10 years ago

Update: same problem with Firefox 33 and Ubuntu 14.04. I can't use Mitro because it doesn't work.

Arduinology commented 10 years ago

Hard to recommend Mitro when it doesn't work with firefox.

Tested with v33.0 on Ubuntu 14.01 and experiencing this issue.

I have also tested with Windows 7, not sure what version of Firefox but probably the latest there as well.

jcosentino commented 10 years ago

I have this problem in Arch Linux, Firefox 33.1.1 I wanted to point out that I use hidpi scaling Under about:config, I set this: layout.css.devPixelsPerPx to 1.3, which scales to ~130%

julianlam commented 9 years ago

:+1: Seems to be a fairly serious issue (though I imagine it works on non-linux systems)?

Chrome plugin works fine.

denadai2 commented 9 years ago

windows 7 has the same error in some enviroments

gregswift commented 9 years ago

Can confirm this issue in FF 36.0.1 on Windows 8

Jarco commented 9 years ago

The problem still occures on firefox 37.0.1 on Manjaro.

DessertArbiter commented 9 years ago

The problem still occurs on Firefox 38.0.5 on Windows 7 SP1.