-
Notifications
You must be signed in to change notification settings - Fork 36
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Would you like quite a lot of changes? #22
base: master
Are you sure you want to change the base?
Would you like quite a lot of changes? #22
Conversation
makelinks.bat had several issues (bashisms with #!/bin/sh, incorrect handling of .user.js files etc.) Rather than keep a second version in sync with the .bat file that seems better-maintained, this commit rewrites the whole thing to read the .bat and mimic its behaviour.
Note: this replaces makelinks.sh
addons.mozilla.org will reject packages uploaded using the git version.
Confuses the Chrome store
Disallowed by the Chrome store
Modern versions of Opera use Blink, so are similar enouh to Chrome not to need special treatment. Older versions of Opera had very primitive extension support. Removing support for older versions of Opera allows us to support more functionality with less work.
…nd ...remove() The existing behaviour is very confusing to a newbie, as console.log() doesn't usually go anywhere useful, so you just see a function return without doing what you expect. Throwing an error is more useful for get(), because callbacks are really required there. Inserting a no-op callback is more useful for set() and remove(), where you usually don't care.
This library allows us to style Chrome settings like native settings
The latter is supported by all browsers now old Opera is gone
Added ‘env’ to build scripts so OS X can find phantom.js, should work across all platforms.
Do you think this will ever be merged? |
@voltagex - have you had the chance to try it out and/or review? I have been insanely busy and while I'm excited to look at these changes, my free time has sadly been very very limited. This is a monstrously large PR and so I believe it really needs some peer review that I've not been able to commit yet. I'm definitely open to merging it after some conversation and some clarity. |
Just at a glance it needs to be split up.
I won't have a chance to test it properly for a little while.
|
I'd sincerely like to get some or all of it merged, it's just so monumental it's hard to review. A split would help immensely so we could at least test/merge bits of it piecemeal and decide if anything is/isn't right. For example, I'm not initially sure if I agree with the inclusion of the CSS files, etc - but I haven't had the opportunity to fully review. |
Hi,
I've used BabelExt as a framework for building forum extensions (a bit like RES, but on a much smaller scale). I've added quite a bit of functionality as part of that work. Would you be interested in taking some or all of it upstream? Major changes include:
I realise there's a lot of stuff here so I don't expect you to accept the request wholesale. But please let me know what if anything would interest you in principle, and what if anything I can do to put it in a format you'd be happy to pull.