The future: Divide and rule
in
10 posts,
5 voices
timendum
Script's Author
  |
I already splitted the script in 2 pieces: the "core" with all the functions, and this script for with the extraction rule for every site supported. Next i'm going to split again this script: i'm going to keep here only "hosting" sites like imageshack.
I think this is the only way to make everyone happy, so you have to install only the script you need, and other sites don't get "ShowJustImaged". Also, if you want to make your own version of SJI, you have to include the "core" and keep only the site logic in your script, so there is no code "fork" ;) |
Barthez
User
  |
Since I'm Opera user it will only make updating this script harder. The //@exclude * results in script not working, so i have to either remove it or copy-paste the //@include from supported sites script. First option could result in some possible conflicts (I'm really guessing here), second one forces me to do manual script alteration every single time you will update any script. I think i will stay with 2.5.3. |
Marti
Scriptwright
|
timendum wrote:An option that you may not have considered is to utilize GM_config(GMC) like I've done with uso - Count Issues Options header. Then just use a single // @include http://* and have a bunch of conditionals that parse window.location along with the GMC checkbox values.
With this method you could have individual check boxes for each site that the end user can toggle on or off on this scripts homepage. Anyhow... it's just a thought. :) Barthez wrote:Yet another Opera incompatibility LOL. ;) Good luck in trying to get them to fix it since Opera is closed source and their engineers don't listen. |
|
timendum
Script's Author
|
@Barthez sorry for the Opera incompatibility, but Opera isn't "officially" supported by this script; i try to make it simpler possible, but everything i do, someone complains... @Marti: i considered your solution, but the script would be slower, because getting/setting configuration is slow, and a include * is tooo much for a simple script!
|
|
Barthez
User
|
timendum, thanks for replay. I know Opera isn't supported but your script is really handy and happens to work in Opera. Since you were honest enough to give us insight on things you're planning to do, I thought that giving some feedback about my experience on different browser (which you might not have installed) would be somehow beneficial. I wouldn't be surprised if it turns out that I'm the only Opera user using this script ;) So don't overwork yourself. |
riego
User
 |
Well done, timendum!
Thanks |
|
Marti
Scriptwright
|
timendum wrote:Well the choice is yours... in my current/local copy of Count Issues I have over 100 conditionals (some are in for loops with regular expressions) and it loads instantly with no delay. Querying would depend on what window.location a user is on so in effect it's test for that, then check for enabled... so two conditionals otherwise exit. ;) Maybe I'll give it a whirl locally just for real-time timing tests on an older version. Just to be clear I don't have any qualms with having it split because I have libraries everywhere too. :) |
|
Marti
Scriptwright
|
Barthez wrote:I usually don't debug Opera but in a test script it seems to be working for me just fine in: Opera/9.80 (X11; Linux x86_64; U; en) Presto/2.8.131 Version/11.11 |
Alexo
User
|
Please don't split, it will make it harder to manage.
|
|
Marti
Scriptwright
|
Alexo wrote:This won't change that. |
