Register a SA Forums Account here!
JOINING THE SA FORUMS WILL REMOVE THIS BIG AD, THE ANNOYING UNDERLINED ADS, AND STUPID INTERSTITIAL ADS!!!

You can: log in, read the tech support FAQ, or request your lost password. This dumb message (and those ads) will appear on every screen until you register! Get rid of this crap by registering your own SA Forums Account and joining roughly 150,000 Goons, for the one-time price of $9.95! We charge money because it costs us money per month for bills, and since we don't believe in showing ads to our users, we try to make the money back through forum registrations.
 
  • Locked thread
astral
Apr 26, 2004

Nice work with the port! This is exactly what was needed to tide people over until XenForo 2.

Is there a most-updated repo for the Firefox changes, or just the zips Serfer has posted? I can help with getting it submitted / setting up an auto-update from the legacy Firefox version when it's ready.

e: can help with stuff like data translation too so people can keep their notes and relevant settings from their legacy firefox install

astral fucked around with this message at 03:57 on Dec 4, 2017

Adbot
ADBOT LOVES YOU

astral
Apr 26, 2004

Serfer posted:

I see there's no longer any conversion capability built in, so it'll probably have to be an outside process to convert the notes from the sqlite file.

Side note, does anyone have a screenshot of what user notes looked like? They should be easy enough to add, I think.

Sorry if I haven't done much, it's a busy time, and while this is useful, I have it working "well enough" for myself. I took a look at integrating the button into the main extension and it looks easy enough, and if I can do that and add user notes, I'll see about submitting it, assuming I can transfer it to whoever wants to maintain it

No worries - just point me at the latest version (do you have a repository setup or just the zips?) and I can take care of the signing part. I don't mind maintaining any firefox-specific changes that can't be merged back into the main one. As for the user notes, it's just a question of converting from the sqlite like you said. What I was referring to though is I can do the data migration with the extension itself as long as the user is running a Firefox version less than 57 (for example, I and many others have been using the ESR).

Master_Odin already mentioned they planned to integrate the extension+button extension as well, so make sure you guys aren't duplicating each others' work. :)

astral
Apr 26, 2004

I've had a few spare minutes here and there tonight so I used them to get a small head start on things. Integrated the 100% compatible stuff from the posted zip with a few slight modifications into my fork repository so far. Need to work out a good way to support the manifest for cross-browser compatibility, since chrome will give a warning with the `applications` flag we will need for SALR for Firefox (because the extension uses certain APIs that require it) - haven't changed that part in the repository yet. Also need to integrate the other stuff from Serfer's zip (such as the change to the behavior of the down arrow in the page nav) as optional instead of replacing SALR Chrome's behavior wholesale, though, and I'll have to ask 'em about the (WIP?) changes to youtube URL matching. Initial testing seems pretty good, though.

Thanks to Serfer for getting things started!

e: also may have to figure out version numbering, since the legacy Firefox version is a bit higher (4.1.3), but it might not be a big problem if the manifest ends up having to be different anyway

astral fucked around with this message at 09:21 on Dec 29, 2017

astral
Apr 26, 2004

Master_Odin posted:

So correct me if I'm wrong but `applications` flag is used to give a minimum version of Firefox that'll run the app right? It looks like it was made non-mandatory since FIrefox 48 and then since this is a web extension API, it self limits itself to FF57+? Or without it the extension store or whatever would happily try and install the web extension version to old versions of FF that don't support the API?

It's mostly used to set the extension's ID for Firefox. My research the other night turned up this note:

quote:

Some WebExtension APIs use the add-on ID and expect it to be the same from one browser session to the next. If you use these APIs, then you must set the ID explicitly using the applications key. This applies to the following APIs:
storage.managed
storage.sync
identity.getRedirectURL
Native messaging
pkcs11
runtime.onMessageExternal
runtime.onConnectExternal
https://developer.mozilla.org/en-US/Add-ons/WebExtensions/WebExtensions_and_the_Add-on_ID#When_do_you_need_an_add-on_ID

which at a quick glance at least two are being used, so it'll probably be necessary.

astral
Apr 26, 2004

The last post button on the page navigator doesn't seem to serve any purpose outside of threads in Chrome SALR - I'd like to change it so that button doesn't appear outside of threads, then adjust the navigator width to compensate. Sound good?

Also, there's a difference in behavior for that button in threads between Chrome and Firefox SALR - the chrome version seems to just scroll to the first unread post, if available, while the Firefox version performed the equivalent of clicking the 'Jump to last read post' button from your bookmark list or from a forum, which people like to use to keep up with fast-moving threads.

Serfer replaced it with the Firefox behavior, but for the sake of integration and in case people were relying on the Chrome SALR behavior, I've gone ahead and set it up as a setting instead which defaults to the existing Chrome SALR behavior; the Firefox release itself can then adjust the default setting to switch to the old Firefox behavior.

astral
Apr 26, 2004

I've hidden the last post arrow for pages where it did nothing and also went ahead and cleaned up the styling a bit like we discussed. I further set the text-decoration to none on the navigator buttons to remove the tiny little underline between them:

Before (screenshot from Chrome in user control panel):



After: (screenshot from Firefox, so some of the font/popup menu stuff looks different, but you get the idea)


Will package this set of stuff as another pull request a little later.

astral
Apr 26, 2004

Progress update:
Got emotes working in Firefox, and about 1/3 done redoing that disabled/removed imgur thing so it'll work.

astral
Apr 26, 2004

Malachite_Dragon posted:

I've been having trouble getting the live preview thing to work, but that's more a QoL thing that can wait on the back burner.

I couldn't reproduce this problem, but I was testing in Nightly so that may have been something that was fixed in a later Firefox, or I might have incidentally fixed it for Firefox in the rest of the quick reply fixes/updates I was working on (on that front, imgur is just about done, just need to do some cleanup/polish on it tonight).

iospace posted:

Question, will you be attempting to get the old pop-up style quick reply window working? It was great for pulling quotes off of other pages or other threads. Obviously it's not a "immediate fix" thing.

It would be nice, yes, but not an immediate thing as you say.

astral
Apr 26, 2004

datajosh posted:

Probably a good idea, this one is over 8 years old now

I propose: SALR Redux - More browsers, even better taste!

tithin posted:

Maybe once the official Mozilla addon is available?

Technically it is, though it's set as Experimental until it's less buggy and closer to being ready: https://addons.mozilla.org/en-US/firefox/addon/salr-redux/

Temporary extension download for the button extension that's compatible: https://bintray.com/salr/salr-firefox/download_file?file_path=salr-chrome-button.zip

astral fucked around with this message at 01:03 on Jan 3, 2018

astral
Apr 26, 2004

Malachite_Dragon posted:

I should have specified what the problem is; it will open just fine, but it won't actually, y'know, live-preview the post I'm working on unless I un-check and then re-check the box.


Like so. But like I said, it's a quality of life thing that can wait until all the big bugs have been fixed.

I appreciate the clarification; the live preview updating part was working for me too, so just let me know if the changes in the next for-firefox release don't fix it for you.

astral
Apr 26, 2004

Master_Odin posted:

Oh sweet, I was waiting for you to post this link and then I was going to start up a new thread. I'll do that now and it'll also feature working pictures of features not pointing to waffleimages!

Sounds good. I'm mostly done with imgur uploading (as in it works nicely; I just need to do some final cleanup), but I've been sidetracked by some user bug reports. It turns out that Firefox has some very strange behavior where if third-party cookies are disabled, XHRs from content scripts to even the domain of the content the content script is running on top of get sent without credentials or origin information. This, combined with jQuery being naughty about parsing responses, led to the hilariously bad result of the unregistered boogeyman video playing on every forum page the user browsed if quick reply was enabled!

I'm rewriting those calls to use the Fetch API instead, which shows as supported by all browsers chrome SALR can run on (including Safari 10.1+). I'm also feature detecting the presence of content.fetch to use instead if available - it's present in Firefox 58+ and sends the request as if it came from the content, enabling quick reply to work properly even if third-party cookies are disabled. Further, I'm adjusting how the responses are parsed to protect people until then.

astral fucked around with this message at 23:12 on Jan 3, 2018

Adbot
ADBOT LOVES YOU

astral
Apr 26, 2004

ArtVandelay posted:

Apologies for the stupid question, but how do I install this?

In Firefox you can go to about :debugging and click "Load temporary addon"

e: Progress update, pushed some commits into the pull request bucket including a fix for quick quote not working in single post pages and some compatibility updates. Also fixed an old bug where successful imgur uploads would get reported to whatever your active tab was instead of the one you were uploading in, so if you were uploading a big image and navigated to a different tab, you'd never get the image link added to the quick reply box.

Still need to finish cleaning up/packaging up the rest of the quick reply fixes, including the emotes for Firefox, imgur uploading, and what I'll call the anti-boogeyman Firefox changes.

astral fucked around with this message at 21:23 on Jan 4, 2018

  • Locked thread