1. 23 Sep, 2015 3 commits
    • Robert Knight's avatar
      Use autoprefixer to add vendor prefixes to CSS · 00c29fe7
      Robert Knight authored
      Use postcss and the autoprefixer plugin to post-process
      CSS to add vendor prefixes and remove use of compass
      mixins which did the same thing.
      
      This moves compatibility concerns out of our styling
      and down to a post-processor whose options can be adjusted
      as necessary and moves us a step towards removing the
      build dependency on compass.
      
      Looking at the generated output, it fixes several issues
      where we were inconsistent about which prefixes were used.
      
      Pyramid webassets does have an autoprefixer filter but
      it was written for an older version of autoprefixer which
      was a standalone tool rather than a postcss plugin. In future
      this also paves the way to replacing the cleancss asset filter
      with an optional step in the postcss-filter.js script.
      00c29fe7
    • Nick Stenning's avatar
      Merge pull request #2514 from robertknight/t87-group_scope_dropdown_ui · b382243d
      Nick Stenning authored
      T87 - Improve group scope dropdown ui
      b382243d
    • Robert Knight's avatar
      Add WebKit / IE prefixes for flexbox · 8699869d
      Robert Knight authored
      For IE 10 and Safari < 9 support, use prefixed flexbox.
      
      For now, this duplicates the flexbox declarations as is
      done in topbar.scss. I will look into using autoprefixer for
      this separately.
      
      Card 87
      8699869d
  2. 22 Sep, 2015 14 commits
    • Nick Stenning's avatar
      Fix annotation pages · 89bfa6b3
      Nick Stenning authored
      cf410d0 broke standalone annotation pages by changing
      `threadRoot.children` from an array into a thread container.
      Unsurprisingly, `children` is supposed to be an array.
      89bfa6b3
    • Robert Knight's avatar
      Use more consistent naming for groupList directive · 9d93a631
      Robert Knight authored
       * Use the singular 'groupList' everywhere
      
       * Simplify the logic for stopping click propagation
         when toggling the selected group
      
       * Remove underscore prefix for Controller classes as per the PR
         discussion.
      
      Card 87
      9d93a631
    • Robert Knight's avatar
      Sort groups in lexicographical order · b2a90531
      Robert Knight authored
      Card 87
      b2a90531
    • Sean Hammond's avatar
      Merge pull request #2522 from hypothesis/fix-pdf-uris · 99bd3f21
      Sean Hammond authored
      Fix detection of PDF URIs
      99bd3f21
    • Robert Knight's avatar
      Add unit tests for <group-list> directive · f49b6d3c
      Robert Knight authored
      Card 87
      f49b6d3c
    • Nick Stenning's avatar
      Only render unanchored notice if feature turned on · 5d37b53e
      Nick Stenning authored
      *N.B.* This does not fix the underlying problem, but it does make it
      possible to deploy the existing codebase without breaking everything in
      production.
      5d37b53e
    • Nick Stenning's avatar
      Fix detection of PDF URIs · 84867512
      Nick Stenning authored
      70c9b9b broke a number of features of the h client that were toggled on
      for PDF documents, by renaming the Annotator PDF plugin from
      `Annotator.Plugin.PDF` to `Annotator.Plugin.Pdf`.
      
      While it's certainly an ugly side-effect of a straightforward rename,
      the problem is that `Guest#getDocumentInfo` (to pick one example) used
      `@plugins.PDF?` to determine whether to use the PDF metadata
      (provided by the PDF.js `PDFViewerApplication`).
      
      In the absence of PDF metadata, `getDocumentInfo` falls back to using
      `window.location.href`, which explains the behaviour identified
      in #2506.
      
      Fixes #2506.
      84867512
    • Robert Knight's avatar
      Add unit test for Groups List dropdown controller · 1b25b808
      Robert Knight authored
      Add a test for the controller part of the <groups-list> directive.
      
      We have a couple of different patterns for testing directive
      controllers in the code, the approach taken here follows the privacy
      dropdown's directive in simply exporting the controller's constructor
      as a 'private' export and referencing it directly in the test.
      
       * Rename the exports in the 'directive/privacy' module for consistency
      1b25b808
    • Robert Knight's avatar
      Groups list dropdown UI tweaks · 08ced29b
      Robert Knight authored
       * Restructure the 'New group' item to make icon
         and label align with the items above it
      
       * Make the whole 'New group' section clickable
      
       * Make 'New group' icon bold for visual consistency
         with other icons
      
       * Use the prerendered group URL from the API instead
         of generating it locally
      
      Card 87
      08ced29b
    • Robert Knight's avatar
      Add inline share link to groups dropdown in top bar · 31316fac
      Robert Knight authored
       * Use an element directive for <group-list> for
         consistency with the <privacy> directive and because
         the directive has its own template.
      
       * Use flexbox for easier layout of items in the groups
         dropdown list.
      
       * Update the hover and selection states of items in the dropdown
         list as per the mocks.
      
       * Add descriptive titles to all links in the list
      
      Card 87
      31316fac
    • Robert Knight's avatar
      Update style for dropdown menus · 8b7a7887
      Robert Knight authored
       * Use the brand color for hovered items,
         including selected items.
      
      Card 87
      8b7a7887
    • Robert Knight's avatar
      Refactor groups list styling · 43085602
      Robert Knight authored
      Use nesting for readability.
      
      Card 87
      43085602
    • Robert Knight's avatar
      Adjust color of 'New Group' background as per the mocks. · 1190a4c2
      Robert Knight authored
      Use a light grey background.
      
      Card 87
      1190a4c2
    • Nick Stenning's avatar
      Merge pull request #2518 from robertknight/karma-watch-target · 87665a80
      Nick Stenning authored
      Add make target to run karma tests with auto-watch enabled
      87665a80
  3. 21 Sep, 2015 6 commits
  4. 18 Sep, 2015 1 commit
    • Randall Leeds's avatar
      Clean up socket, threading, mapper with routing · d99347a1
      Randall Leeds authored
      Move all the logic required to set up a clean environment for the
      client views into the dependency resolution mechanism of ngRoute.
      
      The annotation mapper gains an unloadAnnotations method to make the
      isolated code cleaner.
      
      Use angular-websocket to remove the need for our own socket class.
      
      The AppController becomes much more sane.
      d99347a1
  5. 17 Sep, 2015 4 commits
  6. 16 Sep, 2015 1 commit
  7. 15 Sep, 2015 8 commits
  8. 12 Sep, 2015 3 commits