Skip to content

Commit e440628

Browse files
committed
Merge pull request #49 from meleyal/patch-2
use `page:change` event so components mount on cached pages
2 parents 9a76532 + dc65a90 commit e440628

File tree

2 files changed

+5
-1
lines changed

2 files changed

+5
-1
lines changed

lib/assets/javascripts/react_ujs.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@
5757
document.addEventListener(eventName, callback);
5858
}
5959
}
60-
handleEvent('page:load', mountReactComponents);
60+
handleEvent('page:change', mountReactComponents);
6161
handleEvent('page:before-change', unmountReactComponents);
6262
}
6363
})(document, window, React);

test/view_helper_test.rb

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -58,6 +58,10 @@ class ViewHelperTest < ActionDispatch::IntegrationTest
5858
page.click_link('Bob')
5959
assert page.has_content?('Hello Bob')
6060

61+
# Try going back.
62+
page.execute_script('history.back();')
63+
assert page.has_content?('Hello Alice')
64+
6165
# Try Turbolinks javascript API.
6266
page.execute_script('Turbolinks.visit("/pages/2");')
6367
assert page.has_content?('Hello Alice')

0 commit comments

Comments
 (0)