github/atom
1
1
Fork 0
mirror of https://github.com/atom/atom.git synced 2026-01-15 01:48:15 -05:00
Code Issues Packages Projects Releases Wiki Activity

Store load settings in URL fragment, not query string

Browse Source
This commit is contained in:
Max Brunsfeld
2015-03-04 16:03:00 -08:00
parent 8796c284be
commit 9bb4968e6f
3 changed files with 3 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/atom.coffee
View File

@@ -118,7 +118,7 @@ class Atom extends Model
# Returns the load settings hash associated with the current window.
@getLoadSettings: ->
@loadSettings ?= JSON.parse(decodeURIComponent(location.search.substr(14)))
@loadSettings ?= JSON.parse(decodeURIComponent(location.hash.substr(1)))
cloned = _.deepClone(@loadSettings)
# The loadSettings.windowState could be large, request it only when needed.
cloned.__defineGetter__ 'windowState', =>

2
src/browser/atom-window.coffee
View File

@@ -86,7 +86,7 @@ class AtomWindow
protocol: 'file'
pathname: "#{@resourcePath}/static/index.html"
slashes: true
query: {loadSettings: JSON.stringify(loadSettings)}
hash: encodeURIComponent(JSON.stringify(loadSettings))
hasProjectPath: -> @projectPaths?.length > 0

3
static/index.js
View File

@@ -14,8 +14,7 @@ window.onload = function() {
cacheDir = path.join(cacheDir, 'root');
}
// Skip "?loadSettings=".
var rawLoadSettings = decodeURIComponent(location.search.substr(14));
var rawLoadSettings = decodeURIComponent(location.hash.substr(1));
var loadSettings;
try {
loadSettings = JSON.parse(rawLoadSettings);