will work on GreaseKit sometime?
in
22 posts,
4 voices
pista85
User
|
hey there, love this one so much I sticked with firefox until now. Safari looks much better faster and stable, but I wouldmiss this script so much, I'm hovering images to see them bigger even not on facebook hehe. It's just, are you planning to port it on GreaseKit for Firefox, almost everyscript doesn't work here, would be lovely... thanks for your work mate! |
zertox
User
|
One more vote for safari support! I use Firefox 3.5 beta as well but I still prefer Safari over it |
|
zertox
User
|
I got it working by installing the gm_scripts userscript and by adding a GM_registerMenuCommand function to that script that returns null. |
Vaughan Chan...
Script's Author
|
The next version of the script (which I'm hoping to release either tomorrow or the day after) has some changes which might allow it to run in Safari. I'm running Linux so I cannot test this myself, but I have gotten the script to run in Opera (without the ability to use the configuration screen). |
|
Vaughan Chan...
Script's Author
|
The new version has been released. I have tested it in Opera 9.6 and at a quick glance everything seems to work except for saving settings, obviously because of the missing GM_getValue and GM_setValue functions.
|
sizzlemctwizzle
Scriptwright
|
There's no way to bring up the configuration screen since GreaseKit doesn't support GM_registerMenuCommand. Maybe add a link to the home page? |
|
Vaughan Chan...
Script's Author
|
You can bring up the configuration screen by pressing C (shift + c) or from the Settings link in Facebook's top menu. Unfortunately the settings won't save cause there's no GM_setValue, so right now its just to test that all the features that are enabled by default are working. Or rather to test that the script doesn't just fail completely. Any idea if GreaseKit provides an alternative to GM_setValue? I'm kinda hoping I don't have to resort to saving the settings as cookies. |
|
zertox
User
|
Its not working anymore with the new version. I get the error " TypeError: Result of expression 'lang[language]' [undefined] is not an object "
|
|
zertox
User
|
Maybe you can use the Safari 4 client side database features instead of cookies? |
|
Vaughan Chan...
Script's Author
|
I want the next version of the script to work properly in Safari (as well as Opera and Chrome). I can't do this on my own cause I don't have a Mac and as far as I know GreaseKit only works on OS X. If anybody is willing to help me test in Safari on OS X, please send me a message with an email address I can use to mail you test versions of the script. Also, please let me know which version(s) of Safari you can test with. Correct me if I'm wrong, but it looks like the Safari 4 client side database is one of the new HTML 5 features. If so I think I'd rather use cookies for now, at least until more browsers support HTML 5. |
|
pista85
User
|
hey there, ah sht I didn't notice any email notifs you replied on my thread. Sure I will test it. I was big fan of Firefox until those flash video issues appeared (was using beta for a long time). my email pista.kralovic at gmail dot com |
|
pista85
User
|
well still not working on it! |
|
Vaughan Chan...
Script's Author
|
Hi pista85, I have not had time to work on it yet. I'll probably email you the test version sometime this weekend. |
|
Vaughan Chan...
Script's Author
|
Hi pista85, you should have the test version now. |
Vaughan Chan...
Script's Author
  |
I've gotten more than one response for testing, so I've made the script available at http://www.code-poet.net/test/facebook_fixer.us... for anybody who wants to help out. pista85, thanks for your response. I don't know why you were getting the messages you did. Do any Safari users recognize this type of message:
|
zertox
User
 |
I'm getting this error 2x: Error: NO_MODIFICATION_ALLOWED_ERR: DOM Exception 7 |
zertox
User
 |
The error comes from function addStyle. If you change style.innerHTML to style.innerText it will work! |
|
Vaughan Chan...
Script's Author
|
Thanks a lot zertox, I'll try to update the script tonight. |
|
Vaughan Chan...
Script's Author
|
I have uploaded the modified version of the script, please let me know how it works. Currently checking for updates will not work in Safari as I do not have a replacement for GM_xmlhttpRequest as yet. All other features should hopefully work. Also, Safari users should read the Customization section of the home page. |
|
sizzlemctwizzle
Scriptwright
|
Yes it works. Thank you. |
|
sizzlemctwizzle
Scriptwright
|
My app blocking code is not working in Safari. You need to change line 1090-91 to:
function blockStream(appId) {
if(typeof unsafeWindow==='undefined') unsafeWindow = window;
var xmlhttp = new XMLHttpRequest();I've been using your script while I've been repairing my own. Very nice work.
|
|
Vaughan Chan...
Script's Author
|
Thanks again for the code sizzlemctwizzle, and for the compliment :) |
