Great Script Thank you
|
|
oh my god!
all the shortcut can work rightly now. thank you! |
|||||||||||||||
|
|
My listener is attached to the actual textarea and I used preventDefault to disable the native shortcuts when you are focused on the textarea. Works rather nicely. I should have used it before. |
|||||||||||||||
|
|
I made one change to my script at line 100: If you don't add new line's between the code and the |
|||||||||||||||
|
|
Tim Smart wrote: Yeah I noticed that. You get some strange indentation. I've been doing that manually. I'll add it to the script. |
|||||||||||||||
|
|
Added the one-click spam reporting feature from this script: http://userscripts.org/scripts/show/47097 |
|||||||||||||||
|
|
ok .. i know you've been through all this.. but maybe u can set it to SHIFT + ALT... that shourtcut is rarley used.. |
|||||||||||||||
|
|
DMaster wrote: I don't know... are you having problems with the current shortcuts? |
|||||||||||||||
|
|
sizzlemctwizzle wrote:Maybe having an issue with understanding how your script functions?... took me a while to figure out not to use the hot keys outside of the reply to topic box. This is why I made this:
|
|||||||||||||||
|
|
Marti wrote: why would you use the hot keys outside of the reply box? Auto updater included triggers GM access violation by use of GM_getValue in unsafeWindow (Probably GM_setValue too) I'm not accessing GM_getValue or GM_setValue with unsafeWindow. |
|||||||||||||||
|
|
sizzlemctwizzle wrote:For Alt-Q with selection highlighting... that's what I initially thought your hot keys were, but they are local to the reply to topic and not the entire DOM. sizzlemctwizzle wrote:I can see that in your code, but I'm guessing that an event listener is in a different namespace and triggering it in the default namespace authority. This could be a Firefox related issue too. Your script is the only one I'm using that has unsafeWindow and the vast majority of time I selectively disable DOM JavaScript. It is pretty intermittent so it's hard to pin down what triggers it here. Present in Mozilla/5.0 (X11; U; Linux i686 (x86_64); en-US; rv:1.9.0.9) Gecko/2009040820 Firefox/3.0.9 |
|||||||||||||||
|
|
Marti wrote: My update code isn't triggered by an event. I have this update code in another script that uses unsafeWindow and it has never produced that error either. Marti wrote: Well I might actually be able to add that shortcut if you want. Just uploaded my latest version. This version includes some UI changes, the dynamic page lengthening feature, and added a @namespace. |
|||||||||||||||
|
|
sizzlemctwizzle wrote:Well it triggered it yesterday and the day before I actually figured out a way to trigger it WITH a line number... Like I said it might be a Firefox issue... double checked to see if I had the latest and greatest FF, and I didn't, but now I do. This might correct it, but I'll have to rehack your script that I'm running to see. Mozilla/5.0 (X11; U; Linux i686 (x86_64); en-US; rv:1.9.0.9) Gecko/2009040820 Firefox/3.0.9 sizzlemctwizzle wrote:You might be running into some issues with KDE like we discussed on the forum but it's up to you. The way it stands now it's more compatible with uso integration. If you add a DOM wide hot key might be detrimental to that goal. sizzlemctwizzle wrote:Kewl... I look forward to testing it. :) |
|||||||||||||||
|
|
Marti wrote: Yeah, you're right. That could cause so many issue. I'd rather not mess with it. Pressing Cancel on comment of reporting spam still sends message to spam post... Is this "by design"??? Yes that is by design. Pressing the cancel just means that you don't want to add a comment to the report. Marti wrote: And I look forward to you catching some of my mistakes lol. I've already caught two myself(hence why there is two previous uploads of this version). |
|||||||||||||||
|
|
sizzlemctwizzle wrote:hehe I just corrected my script today too... been buggin me that I've had the errata since the day after I last posted it LOL but finally got around to tweaking it a bit. Kewl CSS styling btw :) will load up a scums script and see if it works there too ;) *GRIN* Looks good for just yours... multiple pages were hiding the prev/next on prior install of your script with the static page lengthening... but appears to be well now. WOOT WOOT! :) |
|||||||||||||||
|
|
Marti wrote: That took a lot of work lol. But the page lengthening was the most difficult. |
|||||||||||||||
|
|
You da man!!! Even works with a scum's script enabled too... here's Joe's script test too: OMG! I think you can all play nicely together now. hehe nice job! Now if I could just get Joe to change his script like I suggested him to. sorta works with a scum's script but only when I use my full single screen rez... still MUCH better. |
|||||||||||||||
|
|
Marti wrote: That's nice to know. I'd use Joe's script if there was some special syntax for replacements so I didn't have to type all the words exactly for them to be replaced. Marti wrote: Can't win 'em all ;) |
|||||||||||||||
|
|
sizzlemctwizzle wrote:That was my suggestion to him... an alias or "shortcut" I'd like to type GMW for the wiki. He's more or less currently using JSON structure... which is quite limited. sizzlemctwizzle wrote:LOL well I won't be using it very much unless I'm super coding here. (unlikely) Resolver test: <hi there>bold bold strong underline italic underline ins bold bold strong underline italic underline ins Table test:
|
|||||||||||||||
|
|
Marti wrote:sizzlemctwizzle wrote:That was my suggestion to him... an alias or "shortcut" I'd like to type GMW for the GM wiki You're lazy like me(or at least we have lazy hands) lol Marti wrote: That's why I only include the most commonly used tags. My script is just for your average everyday sort of use. |
|||||||||||||||
|
|
sizzlemctwizzle wrote:LOL sometimes. I'm still getting the GM access violations on GM_getValue too... and a new one in the error console. Error: document.evaluate("//a[@class='utility']/child::text()[.='Reply to topic']", document, null, 9, null).singleNodeValue is null
Source file: file:///~/.mozilla/firefox/{randomseed}.default/gm_scripts/userscripts_comments_fix/userscripts_comments_fix.user.js
Line: 241
|
|||||||||||||||
|
|
Yeah I'm getting the same null error but still not the access violations. Fixed and uploaded. Going to bed now. Gotta get up early tomorrow. |
|||||||||||||||
|
|
sizzlemctwizzle wrote:Get it constantly now with Reply to topic. Error: Greasemonkey access violation: unsafeWindow cannot call GM_getValue. No line number so it's out of GM's normal "scope"... which typically means it's the DOM or the browser. wait a second... this particular one isn't just your script...*thwaps someone* however lemme retest how I did it in your script and see if it's still present. Can't reproduce the original access violation that had a line number a few days back... so I guess all is well with yours there. Thankie. Scratch that... it only happens when Joe's script is enabled along with yours. He doesn't use unsafeWindow at all but does have event listeners. I'm going to try commenting out your autoupdater and see if it comes back. When I comment out your autoupdater in this script, the Access Violation goes away... Steps to reproduce... have his script running and yours... click Reply to topic.... produces violation... disable his script... goes away... reenable his... comment out your updater... still stays away... but comes back when I reenable the code for the autoupdater. If I comment out Joes two paraphrased lines:
...
pb.addEventListener("focus", function(e){ fix(e); }, false);
...
epb.addEventListener("focus", function(e){ fix(e); }, false);
...
then I don't get the Access Violation... but it still only shows up when I have your script enabled. I do see you use .focus() methods in One-click quoting.
If I comment out just your entire autoupdater code and leave Joe's untouched then I also don't get the Access Violation. I would make a partial guess that your script is triggering his focus event listener where he set's GM_getValue and that trigger is where the Access Violation comes into play. Any other ideas why it would be doing this?
Still present in 1.5.5 I'm de-minifying your script and normalizing it as we speak as well as looking for inconsistencies that may pertain, but this is going to take quite a bit of time. I already did Joe's yesterday and he's not doing anything unusual other than overkill on the event listeners ;) |
|||||||||||||||
|
|
No clue. That error has be completely puzzled. I installed Joe's script and I'm getting it too when I click "Reply to Topic". |
|||||||||||||||
|
|
sizzlemctwizzle wrote:Have some new information. Modified GM to actually tell me the source of the violation (This should be in GM but I can only guess advertising the source of a violation hasn't been addressed? Ticket #254 entered) The actual violation is coming from multiple locations depending on context. Click Quote anchor with no selection highlighting OR Reply to topic anchor
Error: Greasemonkey access violation: unsafeWindow cannot call GM_getValue. Source: http://static.userscripts.org/javascripts/all.js?1240391508 Select text to quote, then click Quote anchor
Error: Greasemonkey access violation: unsafeWindow cannot call GM_getValue.
Source: javascript:$("post_body").focus();
Some of this may pertain to this post in that topic about Prototype... but your script is definitely triggering it as well. @sizzlemctwizzle Is there any way you can retool your portion of your code?
I've modified Joes script to test out the other non-API Leak checked functions and sure enough they run from the ProtoType scope as well as Joe's scope. |
|||||||||||||||
|
|
Marti wrote: Yes I could. |
|||||||||||||||