1. 30 May, 2016 1 commit
    • Robert Knight's avatar
      Fix client on pages that define a variable called `global` · f732879a
      Robert Knight authored
      Modules which assume the existence of a global variable called "global"
      (which exists in Node) are wrapped by Browserify during the build which
      normally results in "global" being aliased to "self" or "window" in a
      browser environment.
      
      If code on the page into which H is loaded defines a global variable
      called "global" however, that gets used instead and this can break such
      modules. In the case of https://www.civilsprep.com, "global" is a
      reference to a DOM node for example.
      
      This commit fixes the issue by only aliasing "global" (as seen by the
      module using it) to either "window" or "self" and not to any existing
      variable called "global".
      
      Fixes #2723
      f732879a
  2. 27 May, 2016 1 commit
  3. 26 May, 2016 8 commits
  4. 25 May, 2016 15 commits
    • Robert Knight's avatar
      Remove an unused CSS class · 669d7c80
      Robert Knight authored
      Usage of this was removed by ba02aee28
      669d7c80
    • Robert Knight's avatar
      Improve logic for scrolling sidebar to annotation with a given ID (#3360) · 66a5da9a
      Robert Knight authored
      The previous logic contained several errors and omissions which could
      result in the card not appearing at the top of the viewport after
      scrolling.
      
       1. If the set of visible threads changed multiple times before the timeout
          that triggered re-measuring of visible thread heights changed, then
          a measured height could be replaced with 0.
      
          Add checks to catch this.
      
       2. Account for the target scroll offset changing as a result of
          the document height increasing after the actual heights of
          cards become known and consequently the maximum possible
          scroll offset changes.
      
       3. Correct calculation of the position that we need to scroll the
          window to in order to have a card positioned at the top
          of the window. For the first card in the list, the position
          should be 0 (as returned by visibleThreads.yOffsetOf(...)),
          and for the Nth card it should be the total height of the N-1
          previous cards.
      66a5da9a
    • Robert Knight's avatar
      Make adder enter/exit animation quicker · 55ad6a05
      Robert Knight authored
      The previous delay made the adder feel laggy. A shorter delay gives a
      snappier feel.
      55ad6a05
    • Robert Knight's avatar
      Add enter/exit animation when adder is shown · 68fcda35
      Robert Knight authored
      Add an entry animation for the adder which makes it 'pop up' or 'pop
      down', depending on which direction the arrow is pointing.
      
      Additionally, animate the color transitions when hovering over the
      toolbar buttons to soften the changes.
      68fcda35
    • Robert Knight's avatar
      Remove a remaining unused piece of the old threading implementation (#3359) · 4f194af8
      Robert Knight authored
      This was missed in 6c0e21a5df.
      4f194af8
    • Robert Knight's avatar
      Fix strict-mode violations in streamer.js (#3358) · afc5812a
      Robert Knight authored
      Fix several missing 'var' declarations in streamer.js
      afc5812a
    • Nick Stenning's avatar
      Merge pull request #3285 from hypothesis/virtualize-thread-list · 536ba5e0
      Nick Stenning authored
      New threading 4/N - Virtualize the thread list
      536ba5e0
    • Sean Hammond's avatar
      Fix some broken links in the docs · eb55d28c
      Sean Hammond authored
      eb55d28c
    • Robert Knight's avatar
      Scroll new annotations into view · d0876bbd
      Robert Knight authored
      Re-implement the behavior to scroll newly created annotations into view
      in the sidebar so that the user can start editing them immediately.
      
      A complication with doing this given that the annotation list is
      virtualized is that if we want to scroll to the Nth visible annotation,
      we may not know the actual heights of the N-1 annotations above it.
      
      As a result of scrolling the list, some of the heights of those N-1
      annotations may be updated from an estimate to an actual measured
      height, changing the Y offset that we want to scroll to in order
      to make the Nth annotation visible.
      
      There are a few strategies that we could use for dealing with this:
      
       1. Forcibly render all N-1 annotations and measure their height.
          This is prohibitively expensive for long annotation lists.
      
       2. Use a cheaper method to calculate the exact height of an annotation.
          Unfortunately we don't have a way to do this at the moment.
      
       3. Use an iterative approach:
      
          a. Estimate the target position based on current known/guessed
             heights.
      
          b. Scroll to the estimated position, which will trigger re-rendering
             and possibly alter the known thread heights.
      
          c. Re-estimate the target position and if it changed, go to step b.
      
      This commit implements the third method.
      d0876bbd
    • Robert Knight's avatar
      Remove margins for spacer elements in thread list · 488ebdc1
      Robert Knight authored
      These elements are used purely to reserve space for offscreen threads
      which are not actually present in the DOM, so they should not occupy any
      space beyond that assigned to their 'height' inline style properties.
      488ebdc1
    • Robert Knight's avatar
      Take into account top and bottom margins when computing annotation card heights · b2faeb98
      Robert Knight authored
      Annotation cards have a small (~8px) margin at the bottom which needs to
      be taken into account when computing the height of each thread in order
      to accurately size the spacer element which reserves space for
      off-screen threads.
      b2faeb98
    • Robert Knight's avatar
      Improve code comments for VirtualThreadList · fdd58e8d
      Robert Knight authored
      Expand the class description to better explain how virtualization
      improves UI performance.
      fdd58e8d
    • Robert Knight's avatar
      Save annotation state to drafts whenever annotation component is destroyed · 645858cb
      Robert Knight authored
      Prior to this unsaved changes were lost when an annotation being edited
      was scrolled off-screen and the annotation component was destroyed.
      
      There was already a mechanism in place for persisting unsaved changes
      to a temporary store when switching accounts and when switching groups
      but this was only triggered when the signed in user changed or the
      focused group changed.
      
      This commit unifies the handling of all three cases by listening
      for the '$destroy' event which is fired whenever an <annotation>
      is about to be destroyed.
      645858cb
    • Robert Knight's avatar
      Fix top bar juddering when scrolling window · 617148e1
      Robert Knight authored
      Use transform3d() to force the top bar onto a new compositor layer so
      that it does not judder around when scrolling the window.
      
      The better fix here will be to make the body scroll rather than the
      whole window.
      617148e1
    • Robert Knight's avatar
      Virtualize the annotation thread list · 0e67b614
      Robert Knight authored
      Make the sidebar handle "large" (ie. 50+) numbers of annotation threads
      much better by virtualizing the thread list. Virtualizing means that we
      only instantiate <annotation-thread> components for threads which are
      either in or 'near' (above/below) the viewport. For the remaining
      offscreen threads space is reserved by a couple of <li> elements above
      and below the visible threads.
      0e67b614
  5. 23 May, 2016 2 commits
  6. 13 May, 2016 10 commits
  7. 12 May, 2016 3 commits