1. 24 Sep, 2014 4 commits
    • Aron Carroll's avatar
      Merge pull request #1524 from hypothesis/isolate-bootstrap-js · 13e5cdcd
      Aron Carroll authored
      Rework application bootstrap.
      13e5cdcd
    • Aron Carroll's avatar
      8466608e
    • Randall Leeds's avatar
      Rework application bootstrap · 9253b420
      Randall Leeds authored
      The embed code has for some time been the place where our application
      is bootstrapped. When we did the epub.js integration, we added query
      parameters to the embed.js view to control which plugins were loaded
      by each instance of the application (a widget host for the reader frame
      and a slimmer guest annotator for the chapter frames).
      
      Change the injection bundle to instead include a separate bootstrap
      script. This has the nice benefit of ensuring that while we still use
      yepnope or a similar technique for script injection that our bootstrap
      code runs in the same global context as the application dependencies
      whether or not the embed script is run in a normal context or a Chrome
      extension content script context. The need for an inline script created
      by the embed code is no longer, since it is a separate script fetched
      from the server (it's bundled in production, inside the monolithic
      hypothesis-assets.js).
      
      Instead of passing the URI of the sidebar iframe by templating the
      bootstrap script, the embed script adds a link tag to the page with
      ``rel=sidebar`` and ``type=application/annotator+html``, which the
      bootstrap script looks for. This pattern fits nicely with the existing
      role of ``rel=sidebar`` in Firefox and Opera and nicely decouples the
      embeding and the bootstrapping.
      
      Further customization that used to take place in the query parameters
      is now done through ``window.hypothesisRole``, which can specify an
      alternate constructor, and ``window.hypothesisConfig``, which can be
      a function to configure the options passed to the constructor. This
      function will be useful for Single Sign On options, too, as laid out
      in #1437.
      
      The destroy script from the Chrome extension is put alongside the new
      bootstrap script, as they complement one another and are not extension
      specific.
      9253b420
    • Aron Carroll's avatar
      Merge pull request #1479 from hypothesis/markdown · 4abf518a
      Aron Carroll authored
      Introduce zero-dependency Markdown Editor.
      4abf518a
  2. 23 Sep, 2014 2 commits
    • RawKStar77's avatar
      dbd633b6
    • RawKStar77's avatar
      Remove $window dependency in markdown directive · f4ceb8d0
      RawKStar77 authored
      Covert px values in markdown-editor.scss to use em. Note: I did this for all values except min-height on .markdown-preview which for some strange reason caused a weird flicker in Firefox.
      
      Add 'markdown-' namespace to .preview and .preview-toggle classes.
      
      Unnested CSS in markdown-editor.scss. Everything is well namespaced now so this is no longer necessary.
      
      Apply consistent sentence case to the title attributes of the markdown tools.
      f4ceb8d0
  3. 22 Sep, 2014 14 commits
  4. 19 Sep, 2014 6 commits
  5. 18 Sep, 2014 10 commits
  6. 17 Sep, 2014 4 commits