Skip to content
This repository has been archived by the owner on Apr 4, 2019. It is now read-only.

Doesn't work with FF 34.0.5 #36

Closed
davidegiannella opened this issue Jan 13, 2015 · 25 comments
Closed

Doesn't work with FF 34.0.5 #36

davidegiannella opened this issue Jan 13, 2015 · 25 comments

Comments

@davidegiannella
Copy link

After the update to FF 34.0.5 whenever I try to open a local .md file, it prompts me with what I want to open it (let's say emacs) rather than viewing it in the browser itself.

@cmtonkinson
Copy link

Or 36.0.4

@klonos
Copy link

klonos commented Mar 29, 2015

Just for the record, it works for me in fx 39.0a1 x64 latest builds. I guess you guys will have to wait until that version is released as stable 😄

@Thiht
Copy link
Owner

Thiht commented Mar 29, 2015

With the version 36 aurora, I had this problem with the built-in pdf.js (it prompted to download the pdf instead of reading it). Can you try to completely reinstall Firefox, including config files, preferences, etc. ? It worked for me with pdf.js

@fpirsch
Copy link
Contributor

fpirsch commented Mar 29, 2015

Everything fine here with FF36 Ubuntu and Windows.

@MaerF0x0
Copy link

No worky for 37.0.1 on OS X

@davidegiannella
Copy link
Author

Tried to start firefox, 37.0.1 this time, with a new profile having only MD viewer installed but still it doesn't work. Seems we'll have to wait for FF 39.

@hahcho
Copy link

hahcho commented Jun 2, 2015

The same with firefox 38.0-1 on Arch.

@dizzib
Copy link

dizzib commented Jun 22, 2015

Same problem after upgrade FF from 38.0 (where it was working nicely) to 38.0.5 as part of a full system upgrade in Arch Linux. Problem even persists after downgrade back to 38.0.

@klonos
Copy link

klonos commented Jun 22, 2015

Still works fine in 41 nightly builds. Must be some kind of regression in 38.

@fpirsch
Copy link
Contributor

fpirsch commented Jun 22, 2015

This is NOT related to FF 38. It works fine here with 38 and 38.0.5 on both Ubuntu and Windows, I'm using it every day.
This problem must be somewhere else, e.g. in the OS config.

@RGBD
Copy link

RGBD commented Jul 24, 2015

Do not work with local files.
Firefox 39.0
Both Fedora 22 and Arch Linux.

@davidegiannella
Copy link
Author

what details can I provide for better investigate and reproduce this issue? FF 39.0 and still not working.

I played around though so maybe can be something else now.

@wangjiezhe
Copy link

It doesn't work for me on firefox 39.0-1 on arch.

@Thiht
Copy link
Owner

Thiht commented Aug 7, 2015

Progress report: I was able to reproduce the "not working" bug on Fedora with FF39, so I guess there was a change in FF's API or something. I tried several minor modifications on my side but weren't able to make it work yet... So some investigations will be needed on what could be broken. As far as I can tell, it may require checking on FF's source code since the doc on streamconverters is kinda sketchy. It could take a while before it works again.

Could some of you (with the bug) try these two add-ons and tell here if they work, so I can compare what differs with markdown viewer ?

@wangjiezhe
Copy link

The first one work fine for me,

2015-08-08-110612_1920x1080_scrot

but the second doesn't with local files.

2015-08-08-105202_1920x1080_scrot
2015-08-08-105204_1920x1080_scrot

@raphael0202
Copy link

It also doesn't work for me, I get a prompt asking where to save the file. On Fedora 22, Firefox 40.0.3

@RGBD
Copy link

RGBD commented Oct 3, 2015

Not working. Firefox 41.0.1 x86_64 Arch Linux

@WnP
Copy link

WnP commented Oct 30, 2015

@Thiht both jsonview and open-in-browser works as expected here on firefox 40.0.3, but markdown-viewer doesn't render mardown at all

@WnP
Copy link

WnP commented Oct 30, 2015

setting:

<RDF:Description RDF:about="urn:mimetype:text/plain"
        NC:value="text/plain"
        NC:fileExtensions="md"
        NC:description="Text Document">
    <NC:handlerProp RDF:resource="urn:mimetype:handler:text/plain"/>
</RDF:Description>

in ~/.mozilla/firefox/*default/mimeTypes.rdf fix this issue for me

all credits are for Ryan Chapin, original article: source

@Thiht
Copy link
Owner

Thiht commented Nov 23, 2015

I think I finally found the fix, now working on my Fedora 22/Firefox 42.0. :)

Can I have some feedback before I push the changes to addons.mozilla?

@Thiht Thiht reopened this Nov 23, 2015
@Thiht
Copy link
Owner

Thiht commented Nov 25, 2015

If you want to try the fix I published a pre-release here: https://github.com/Thiht/markdown-viewer/releases/tag/v1.9-pre
Note it won't work if electrolysis is enabled (Firefox 43+ I think), see #46.

@wangjiezhe
Copy link

It now works fine for me on firefox 42.0 on Arch

@Thiht
Copy link
Owner

Thiht commented Nov 26, 2015

Cool, I'll update the AMO version later today :)

@Thiht
Copy link
Owner

Thiht commented Nov 26, 2015

I just updated the new version to AMO, it should be updated in a few days if nothing goes wrong. I suggest you switch back to the signed version when it's updated to obtain possible future updates.

@davidegiannella
Copy link
Author

Sorry for very late reply. I can confirm that the 1.9 version fix the issue. Thanks @Thiht

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests