1. 14 Sep, 2021 9 commits
    • Lyza Danger Gardner's avatar
      Remove unused forms SASS · b000b180
      Lyza Danger Gardner authored
      b000b180
    • Lyza Danger Gardner's avatar
      Replace .InputButton custom pattern with TextInputWithButton · 97e7d67a
      Lyza Danger Gardner authored
      The general pattern for an input next to an icon-only button has been
      added to the pattern library in the frontend-shared package and is
      implemented by the TextInputWithButton component. We don't need custom
      treatment in this project anymore.
      97e7d67a
    • Lyza Danger Gardner's avatar
      Simplify .PaginationPageButton · 81df42f9
      Lyza Danger Gardner authored
      Use `classes` to extend, not obliterate, component styles, obviating
      the need to use complex mixins.
      81df42f9
    • Lyza Danger Gardner's avatar
      Simplify .NonResponsiveIconButton · a81a51ac
      Lyza Danger Gardner authored
      Use `classes` to extend, not obliterate, component styles and obviate
      the need for complex mixins.
      a81a51ac
    • Lyza Danger Gardner's avatar
      Simplify PublishControlButton styling · 35a6fb4a
      Lyza Danger Gardner authored
      Use `classes` to extend, not obliterate, components styles and obviate
      the need to use complex mixins.
      35a6fb4a
    • Lyza Danger Gardner's avatar
      Simplify .InlineLinkButton · dc99cb1e
      Lyza Danger Gardner authored
      Use `classes` to extend, not obliterate, component styles, and obviate
      the need to use complex mixins.
      dc99cb1e
    • Robert Knight's avatar
      Remove obsolete `@ts-ignore` · 51155eda
      Robert Knight authored
      51155eda
    • Robert Knight's avatar
      Use `classList.add` to add tokens to the class list · 0d4b5367
      Robert Knight authored
      This is functionally the same, just a little cleaner.
      0d4b5367
    • Robert Knight's avatar
      Reduce forced reflows when drawing PDF highlights · c8307923
      Robert Knight authored
      The rendering of SVG highlights in PDFs triggered an excessive number of
      style recalculations and forced reflows due to interleaving DOM changes
      with DOM API calls that require current layout information.
      
      This commit makes several changes to reduce reflows:
      
       1. Create all the `<hypothesis-highlight>` elements in the text layer
          before creating any of the associated SVG elements.
      
          This avoids interleaving creating `<hypothesis-highlight>` elements
          with measuring them in order to position the associated SVG
          `<rect>`.
      
       2. Change `drawHighlightsAbovePdfCanvas` to accept a list of highlight
          elements instead of a single element. By assuming all highlights for
          a single annotation are on the same page, we can avoid some repeated
          work and add all the `<rect>` elements to the parent `<svg>` in one
          batch.
      
      These changes only reduce the thrash when creating highlights for a
      single annotation. To eliminate the remaining thrash we'd need to use a
      combination of optimizing DOM updates across multiple `highlightRange`
      calls and/or reducing the work for PDF pages which are rendered but not
      visible on-screen.
      c8307923
  2. 13 Sep, 2021 17 commits
  3. 09 Sep, 2021 2 commits
  4. 08 Sep, 2021 12 commits