Easier browsing of Hacker News...for me

https://news.ycombinator.com/item?id=216446

by nirmal • 18 years ago

57 36 18 years ago

news.ycombinator.com

HN SplitView is a bookmarklet that adds a link labelled "SPLIT VIEW" to each post on the Hacker News website. This link allows the user to open the post and the discussion page side by side in frames. Simply drag the link for HN SplitView to your toolbar or add it as a favorite.

I've been using this and it makes it easier for me to browse. I hope that other members of the HN community find it useful. I've tested it under Safari and FF. Let me know if you guys have any problems.

EDIT: It doesn't add the link for posts like this one or those of the "Ask HN:" variety.

mwmanning• 18 years ago

This is brilliant.

richesh• 18 years ago

very nice, and very useful...can you make it a one click thing? right now I have to click on the bookmarklet and then click on split view. If it was just one link that would be awesome. I wonder if this can be tweaked to work from within Google Reader, hmmm..I'll tinker with that a bit.

thanks again!

nirmal10• 18 years ago

I think that would require the script running at the loading of the page. This could probably be done with a Greasemonkey script (Creammonkey for Safari).

Oompa3• 18 years ago

I turned this into a Greasemonkey script for any Firefox users.

http://userscripts.org/scripts/show/28371

If the author doesn't like it, PM me, and I'll delete this post and the Userscripts.org post.

nirmal10• 18 years ago

Quite the contrary, thanks for doing it! I used to be worried by the constant security issues when Greasemonkey came out (back in the day). I never got into it and now I use Safari. Do you know if this works under Creammonkey for Safari? I may go install it if it does.

nirmal10• 18 years ago

http://userscripts.org/scripts/show/28372

I just modified it so that it would prompt for install when you clicked Install Script. GreaseKit for Safari doesn't have an Install User Script option...work in progress.

tokipin• 18 years ago

i was thinking about proposing this as a feature. because when dealing with submissions, the constant is that i look at both the article and the comments

kajecounterhack• 18 years ago

wow this is so useful. kudos =]

My first bookmarklet too, really.

vipulsolanki• 18 years ago

Wah NirmalBhai, wah.

Btw, There s one minor bug. If I click the bookmarklet more than once, it adds "|SPLIT VIEW |" more than once.

nirmal10• 18 years ago

Fixed.

agentbleu• 18 years ago

10 /10

btw0• 18 years ago

Simple, but effective. Awesome.

ScottWhigham2• 18 years ago

I love you, nirmal.

listic• 18 years ago

On Opera, it says "Unsupported browser". Why not?

nirmal10• 18 years ago

I had modified the bookmarklet to allow opera but it doesn't like the code on my splitting page. I'm not sure why though. It's pretty tame JS. Opera also doesn't show me any errors in the Error Console.

Xichekolas7• 18 years ago

So I went to add this splitview thing to my Greasemonkey script for Hacker News, only I did the same thing without hitting your site for the frameset (completely clientside js). It works quite nicely, but for some reason I had to resort to something I consider quite ugly.

Can someone please look at the code below (and read my comments in it) and tell me why the three addEventListeners that I have commented out won't work? I'd really really like to know why I am being retarded and how to fix it, as the solution I ended up with seems unnecessarily ugly.

View code here: http://userscripts.org/scripts/review/25039

Or if you don't care about it being ugly and just want to use it: http://userscripts.org/scripts/show/25039

But I'd really appreciate help with figuring out the issue with the addEventListeners.

jkkramer• 18 years ago

For one thing, the function returned by createFramesetReal() seems to be using the undefined variable 'b'.

Xichekolas7• 18 years ago

Op, I had that in a previous version, musta accidently cut that out when cleaning it up. It didn't have an effect though.

Went ahead and set it to what it should be... body.

nirmal10• 18 years ago

I've installed the script under Safari and FF2 and I don't notice anything happening. Is it overwriting the existing links? I didn't do that because sometimes I may only want to quickly see the comments or main article.

Firebug doesn't seem to be showing me any errors either.

BTW: I like the title of your blog, "Artfully Underemployed" :)

Xichekolas7• 18 years ago

If you installed it for the first time, make sure you turn the feature on. Look for the link called "blacklist" in the top bar of HN, which will pull up the options for the script. Check the box next to split view.

Since this is becoming more than just the simple blacklist script it originally was, I suppose I should rename it or something to make that more clear. ;)

Oh and I almost forgot, thanks for the brilliant idea!

marrone• 18 years ago

I didnt look to closely, but it looks like you are creating a closure around the comments var for your event listener function. The problem is that you are doing this within a loop, meaning the comments var changes after each iteraion, probably ending up in something you hadnt intended

Xichekolas7• 18 years ago

But even if I create an anonymous function that just calls alert(), the click event never fires. i.e. ->

  byId(thisnode.id).addEventListener('click', function(ev) { alert('test'); }, false);

Won't fire either.

thaumaturgy2• 18 years ago

Disclaimer: I've never used Greasemonkey (now an Opera user) and have only spent a few minutes trying to figure out how it interfaces with the browser. I know my way around JS though.

That said, after doing some testing I think your issue is that the page isn't completely loaded before the addEventListener is called. Changing the addEventListener line to a simple element.onclick = function(alert) triggers an error in the browser; so does trying to ask about the element's onclick function. Putting in an alert box in place of the addEventListener line pretty clearly demonstrates that the page is still loading when the code is being called on the links. Firefox is a little touchy about that.

I'm still working on it, lemme see what happens if I wrap up your script's initialization section into a document.onload, or something similar.

Edit: Also, elements are being returned as XPCNativeWrapper objects [http://developer.mozilla.org/en/docs/XPCNativeWrapper]. I'm totally unfamiliar with those; maybe that's interfering with the event attachment somehow?

Edit: Yeah, looks like it. I wasn't sure, so I had to test, but references to elements from directly inside the code body get the XPCNativeWrapper objects, which provide protection for some properties. But, code called from a setTimeout doesn't get those objects, and gets direct access to the elements. Not sure how to fix that right since I've never encountered that before.

Xichekolas7• 18 years ago

Yeah I learned about the XPCNativeWrapper objects last night too. Basically GM has access to every object (from window/document on down) as one of these for security purposes (so the target page can't redefine standard methods). Due to the vagaries of the way they are implemented, you can't call the onclick/onkeypress/etc methods directly. It actually states in the documentation that addEventListener is the preferred (and only working) method.

You can supposedly get access to the underlying JS object from the XPCNativeWrapper object with the wrappedJSObject property and then act on it, but it's a big security no-no. I tried it anyway to no avail.

Your thought on the page not being loaded is kind of where I ended up too. However, my understanding was that GM ran user scripts after the window.onload finished, but I could be completely wrong here. If I assign all the links id attributes in the GM script and then manually add event listeners in my Firebug console, they work fine. Also, you'll notice that later in my script I add click listeners on other items (buttons and links) that work swimmingly, so it's either entirely arbitrary or I'm missing something very obvious.

Thanks for looking at this, I'll look forward to any other thoughts you have.

EDIT: Looks like I was very wrong about the GM script executing after the window load event. I have been playing with wrapping code in a load event listener and have indeed got it working that way. Thanks for the suggestion! I'll hopefully have something polished and posted soon.

Xichekolas7• 18 years ago

Just an update to this. I have the problem fixed now, so thanks all that looked at it. (And thaumaturgy, I owe you a beer.)

New version is on the userscripts page for anyone that wants it: http://userscripts.org/scripts/show/25039

nirmal10• 18 years ago

UPDATES: For greasemonkey/kit users. Clicking Split View links in the right frame will now not cause internal splitting.

UPDATES: I can't edit the main post anymore so I thought I would try and summarize the updates here.

Oompa posted a Greasemonkey script here: http://userscripts.org/scripts/show/28371

I modified to work on multiple pages under hacker news, you can find it here: http://userscripts.org/scripts/show/28372

The bookmarklet still works and is located at the URL in the post.

The bookmarklet will not continue adding SPLIT VIEW links after it has been used once.

xccx• 18 years ago

i'm too retarded to install this but not too proud to admit it.. it sounds like it scrolls 2+ windows with one control:

xcroll

christefano• 18 years ago

This script assumes that I visit news.ycombinator.com to read news. I'd prefer a Firefox extension that compares the addresses of web pages I visit with links in the news.ycombinator.com RSS feed. That way I'd be notified that there's a discussion thread about the page I'm currently viewing.

Easier browsing of Hacker News...for me

Related Stories