From 27d9e7a8c64ea1f7725a298246a4f1a20f3f8b3a Mon Sep 17 00:00:00 2001 From: Gonzalo Rodriguez Date: Wed, 12 Oct 2011 02:57:48 -0200 Subject: [PATCH] Remove this, since selected aspects are now handled in the session hash Not only this code is unnecessary with the new way selected aspects are stored, also on some edge cases could confuse the user for example when having two tabs opened and hitting refresh in one of them. Expecting selected aspects, but the "a_ids[]"-like url could instead set the new group of selected aspects. --- lib/stream/aspect_stream.rb | 2 +- public/javascripts/widgets/aspect-navigation.js | 4 ---- 2 files changed, 1 insertion(+), 5 deletions(-) diff --git a/lib/stream/aspect_stream.rb b/lib/stream/aspect_stream.rb index d12f59d43..a0bbf90f9 100644 --- a/lib/stream/aspect_stream.rb +++ b/lib/stream/aspect_stream.rb @@ -53,7 +53,7 @@ class AspectStream < BaseStream # @return [String] URL def link(opts={}) - Rails.application.routes.url_helpers.aspects_path(opts.merge(:a_ids => aspect_ids)) + Rails.application.routes.url_helpers.aspects_path(opts) end # The first aspect in #aspects, given the stream is not for all aspects, or #aspects size is 1 diff --git a/public/javascripts/widgets/aspect-navigation.js b/public/javascripts/widgets/aspect-navigation.js index dadf06bdf..dddd0d0d3 100644 --- a/public/javascripts/widgets/aspect-navigation.js +++ b/public/javascripts/widgets/aspect-navigation.js @@ -100,10 +100,6 @@ self.fadeOut(); self.jXHR = $.getScript(newURL, function(data) { - if (typeof(history.pushState) == 'function') { - history.pushState(null, document.title, newURL); - } - var textarea = $("#publisher textarea#status_message_fake_text"), photozone = $("#photodropzone");