jeromegn / Backbone.localStorage

A localStorage adapter for Backbone.js
MIT License
1.9k stars 681 forks source link

Rename to BackboneStorage #191

Closed hugohenrique closed 7 years ago

hugohenrique commented 8 years ago

Hi! What do you think is to rename StorageBackbone? I believe it can become more semantic, if you need to use LocalStorage or SessionStorage.


Backbone.Storage = function () {
  // current implementation
};

Backbone.LocalStorage = Backbone.Storage;
Backbone.SessionStorage = Backbone.Storage;

Backbone.LocalStorage.prototype.storage = function () {
  return window.localStorage;
};

Backbone.SessionStorage.prototype.storage = function () {
  return window.sessionStorage;
};
scott-w commented 7 years ago

Hi @hugohenrique, while I like the suggestion I think keeping this focused on the localStorage implementation is the right way forward for now, given the limited resources available to the project.