Skip to content

C
coopwire-hypothesis
Switch branch/tag
  1. 25 May, 2021 3 commits
  3. 24 May, 2021 9 commits
  5. 21 May, 2021 7 commits
  7. 20 May, 2021 2 commits
    • Eduardo Sanz García's avatar
      Display the annotation header on the top-level annotation · fcfde74a
      Eduardo Sanz García authored 
      This PR ensures that the annotation header is always shown, even if the
top-level annotation is hidden by a filter. The annotation header
contains the document target. This information is specially important in
the context of the notebook, where annotations from different documents
are displayed together.

I created a wrapper around the `AnnotationHeader` component. I couldn't
use it directly because I needed a couple of methods from the store to
mimic 100% the same behaviour as when `AnnotationHeader` is called
through `Annotation` component.
      fcfde74a
    • Eduardo Sanz García's avatar
      Remove flag icon from annotation cards for LMS users · cf22b700
      Eduardo Sanz García authored 
      1. Added an option (`allowFlagging`) in the service configuration to
   enable/disable flagging.

2. Added logic in `AnnoationActionBar` component to hide the flag icon
   if the service configuration has the option `allowFlagging`.

Closes https://github.com/hypothesis/product-backlog/issues/1126
      cf22b700
  9. 17 May, 2021 11 commits
  11. 14 May, 2021 2 commits
  13. 13 May, 2021 1 commit
    • Lyza Danger Gardner's avatar
      Add classname prefix to avoid conflicts with `frontend-shared` · 3b967dc6
      Lyza Danger Gardner authored 
      Add a temporary class name prefix to the local `Panel` component styles.

This will avoid conflicts with the styling for the new `Panel` component
in the `frontend-shared` package.

In the future:

* This local `Panel` will be supplanted by the shared variant, and
* Shared component classnames will be prefixed to avoid future conflicts
      3b967dc6
  15. 12 May, 2021 3 commits
    • Robert Knight's avatar
      Remove unused `store.getState` stubs in tests · 801b3fa8
      Robert Knight authored 
      All of the components use selector methods now instead of accessing the
store's internal state directly.
      801b3fa8
    • Robert Knight's avatar
      Use store selectors instead of `getState` in `FrameSyncService` · 0e523df3
      Robert Knight authored 
      Remove the last remaining usage of `store.getState()` outside of the
store implementation in favor of the `store.allAnnotations()` selector
method.

This allows simplifying the state structure in the fake store that the
`FrameSyncService` tests create.
      0e523df3
    • Robert Knight's avatar
      Update export style and documentation for sidebar store · 4b7c566d
      Robert Knight authored 
      Convert the `store/index.js` and `store/create-store.js` modules to use
named rather than default exports per our current conventions, and
revise the documentation.

The goal of the revised documentation is to more clearly describe what the
store is and how it is used within the sidebar/notebook app, assuming
that the reader is likely have at least some familiarity with Redux (or
can read the "Introduction" section of the linked website if not). In
particular I have tried to convey:

 - The separation between the "base" `createStore` function which is not
   application-specific and the `createSidebarStore` function and the
   modules it uses which are
 - How the store in the sidebar app differs from a standard/base Redux store
 - Best practices around using the store from other parts of the app
   (use `useStoreProxy` in UI components, use selector and action
   methods rather than `getState` and `dispatch`)
      4b7c566d
  17. 11 May, 2021 2 commits
    • Robert Knight's avatar
      Replace `document.title` as a fallback for title in PDFs · 40eaf5c9
      Robert Knight authored 
      Replace the usage of `document.title` as a way to get the document title
if the PDF has no embedded title in either its _document info
dictionary_ or _metadata stream_.

In top-level frames using `document.title` (where `document` is the
global HTML document, not the PDF) works because PDF.js sets the title
based on the first non-empty value from:

 1. The embedded title
 2. The filename from the `Content-Disposition` header
 3. The last segment of the URL's path (eg. "test.pdf" in
    "https://example.com/test.pdf")

When PDF.js is embedded in an iframe however, it does not set
`document.title` by default. As a result, documents were ending up in
Hypothesis with a generic "PDF.js viewer" title.

This commit implements (roughly) the same logic that PDF.js uses to
determine the value used to set `document.title`, in the case where the
PDF has no embedded title. This means implementing steps (2) and (3)
from the above list. The `Content-Disposition` filename is not exposed
as a public property on `PDFViewerApplication`, so
`PDFMetadata#getMetadata` was refactored to call the
`pdfDocument.getMetadata` instead.

Fixes https://github.com/hypothesis/client/issues/3372
      40eaf5c9
    • Robert Knight's avatar
      Add missing tests for various error paths · 96723e83
      Robert Knight authored 
      I ignored one path that is only used in local development.
      96723e83