Skip to content

Instantly share code, notes, and snippets.

@maks
Last active December 17, 2015 11:59
Show Gist options
  • Save maks/5606915 to your computer and use it in GitHub Desktop.
Save maks/5606915 to your computer and use it in GitHub Desktop.
Notes on Making Brackets work as a Chrome Packaged App

Starting point

Started with: https://github.com/adobe/brackets/tree/glenn/dropbox-integration

removed dropbox int

because dropbox 3pp js script uses localstorage api

inline script elements in index.html not allowed

for now just commented out the script elements

can't use localStorage

for now just commented out usage as it was only for workaround with CEF not setting navigator.language correctly.

Problem with less 1.3.0

Can't use sync with XHR, bug: less/less.js#937 fixed in less 1.3.1 So switched to latest stable: 1.3.3 and add to crx.js:

less = { async: true, fileAsync: true }

as per less/less.js#1031

FIXED as of Sprint 27

why I hate compiled CSS

upgrading to LESS 1.3.3 uncovers error in form.less, part of ancient version 1.4 of Twitter bootstrap. Using http://less2css.org/ online demo pointed to error on line 187:

Syntax error: expected ')' got '}'
      .box-shadow(0 0 6px lighten(@borderColor, 20%);    }  }

and yes sure enough, the brackets don't balance, so easy fix in the end.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment