Is it possible to add support for hashes?

Such as node/2343/#foo

Comments

Anonymous’s picture

Anonymous (not verified) commented
Status: Active » Postponed (maintainer needs more info)

Some very quick testing shows that hashes can be used. Please provide details about where you are trying to use this at so that I can do further testing.

Flying Drupalist’s picture

Flying Drupalist commented

I have a custom content type from which I alter visible content with js and subsequently mark the user 'location' with hashes. I can load where I was just by loading the url with the hash. But this module would be perfect for saving the location to profile as well.

Thanks.

Anonymous’s picture

Anonymous (not verified) commented
Status: Postponed (maintainer needs more info) » Closed (won't fix)

I'm going to make some assumptions here.

It appears that what you want is for the 'Add This Page' link to support the hash correct?

The problem is, the hash is not passed to the server by the browser. It just isn't. This means that the Add This Page link won't support the hash because it is not possible to determine what the hash is. I've searched to try and see if this has been solved elsewhere but couldn't find anything. Feel free to correct me if I'm wrong.

I'm marking as won't fix for now.

Flying Drupalist’s picture

Flying Drupalist commented
Version: 6.x-2.1 » 6.x-2.3
Status: Closed (won't fix) » Active

Well yes, that's because the URL is currently determined by server side code. This would be possible if the url was determined by javascript, which it could be and would work in this scenario.

And getting the URL by javascript is pretty simple, just window.location.

I really really need this feature, and thanks a lot for the great module.

Anonymous’s picture

Anonymous (not verified) commented
Status: Active » Postponed

This would be a progressive enhancement. Postponing for the moment until I've got a few free hours to think about it.

Anonymous’s picture

Anonymous (not verified) commented
Status: Postponed » Closed (won't fix)

This is not as simple as doing a window.location query. At this time I won't be implementing it. Feel free to submit a patch otherwise it will remain and continue to be won't fixed.