• Robert Knight's avatar
    Extract side-by-side functionality out of PdfSidebar class · 793d289c
    Robert Knight authored
    Move the functionality that PdfSidebar implements elsewhere and remove
    the class. This mainly consists of side-by-side mode, but there is also
    the logic to get the scrollable document container for use with the
    bucket bar. The PDF-specific logic has been moved to the `PDF` class in
    `src/annotator/plugin/pdf.js` and the generic logic has been moved to
    the `Sidebar` or `Guest` classes.
    
    This change is a step towards creating an common interface between the
    Guest/Sidebar and the document-type/viewer specific functionality. In
    future the `PDF` class will evolve into the implementation of this
    interface for PDFs. For HTML documents the logic is currently contained
    directly in the `Guest` class and this will be extracted out into a
    class that implements the same interface.
    
     - Move the PDF-specific parts of side-by-side mode into
       `PDF#fitSideBySide`
     - Move the generic parts of side-by-side mode into the `Sidebar` class.
       This will allow it to be reused for HTML documents or other document
       types in future
     - Move the logic for determining the scrollable container into
       `PDF#contentContainer`
     - Change the annotator setup code in `annotator/index.js` to always
       instantiate the `Sidebar` class, regardless of document type
    793d289c
Name
Last commit
Last update
..
integration Loading commit data...
adder-test.js Loading commit data...
annotation-counts-test.js Loading commit data...
annotation-sync-test.js Loading commit data...
bucket-bar-test.js Loading commit data...
empty.html Loading commit data...
features-test.js Loading commit data...
guest-test.js Loading commit data...
highlighter-test.js Loading commit data...
notebook-test.js Loading commit data...
range-util-test.js Loading commit data...
selection-observer-test.js Loading commit data...
sidebar-test.js Loading commit data...
sidebar-trigger-test.js Loading commit data...
toolbar-test.js Loading commit data...