- 18 Jan, 2021 10 commits
-
-
Robert Knight authored
This reverts commit 5c00dcac.
-
dependabot-preview[bot] authored
Bumps [sass](https://github.com/sass/dart-sass) from 1.32.2 to 1.32.4. - [Release notes](https://github.com/sass/dart-sass/releases) - [Changelog](https://github.com/sass/dart-sass/blob/master/CHANGELOG.md) - [Commits](https://github.com/sass/dart-sass/compare/1.32.2...1.32.4) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
dependabot-preview[bot] authored
Bumps [preact](https://github.com/preactjs/preact) from 10.5.9 to 10.5.10. - [Release notes](https://github.com/preactjs/preact/releases) - [Commits](https://github.com/preactjs/preact/compare/10.5.9...10.5.10) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
dependabot-preview[bot] authored
Bumps [@sentry/browser](https://github.com/getsentry/sentry-javascript) from 5.29.2 to 5.30.0. - [Release notes](https://github.com/getsentry/sentry-javascript/releases) - [Changelog](https://github.com/getsentry/sentry-javascript/blob/master/CHANGELOG.md) - [Commits](https://github.com/getsentry/sentry-javascript/compare/5.29.2...5.30.0) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
dependabot-preview[bot] authored
Bumps [commander](https://github.com/tj/commander.js) from 6.2.1 to 7.0.0. - [Release notes](https://github.com/tj/commander.js/releases) - [Changelog](https://github.com/tj/commander.js/blob/master/CHANGELOG.md) - [Commits](https://github.com/tj/commander.js/compare/v6.2.1...v7.0.0) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
dependabot-preview[bot] authored
Bumps [aws-sdk](https://github.com/aws/aws-sdk-js) from 2.824.0 to 2.828.0. - [Release notes](https://github.com/aws/aws-sdk-js/releases) - [Changelog](https://github.com/aws/aws-sdk-js/blob/master/CHANGELOG.md) - [Commits](https://github.com/aws/aws-sdk-js/compare/v2.824.0...v2.828.0) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
dependabot-preview[bot] authored
Bumps [eslint](https://github.com/eslint/eslint) from 7.17.0 to 7.18.0. - [Release notes](https://github.com/eslint/eslint/releases) - [Changelog](https://github.com/eslint/eslint/blob/master/CHANGELOG.md) - [Commits](https://github.com/eslint/eslint/compare/v7.17.0...v7.18.0) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
Robert Knight authored
Previous commits updated the router service to enable it to correctly set the route from the initial path in the browser extension. However some parts of the client still behaved as if they were the sidebar in the notebook view because there was a second path => route mapping in `src/sidebar/index.js`. This commit replaces that with `store.route()` everywhere which is the route determined by the router service. This fixes an issue where launching the notebook app in the browser extension resulted in a "second Discovery server" error in the console.
-
Robert Knight authored
When the app launches the `router` service sets the initial route based on the path. When the notebook view is loaded from the extension the path is `/client/notebook.html` rather than `/notebook`. Fixes https://github.com/hypothesis/client/issues/2866
-
Robert Knight authored
Using a generic `<div>` can result in the notebook being positioned incorrectly or otherwise receiving undesirable styling on pages with very generic selectors. For example https://example.com. Using a custom tag name reduces the risk and we already do this for various other UI elements in the annotator part of the app. We can protect the contents of the outermost element using Shadow DOM, but the outermost element itself will always be subject to the page's styles.
-
- 15 Jan, 2021 1 commit
-
-
Eduardo Sanz García authored
Highlights contains two pseudo-elements (`::before` and `::after`) with text for assistive technologies. Because these elements have dimensions, on RTL languages a highlight near the right border of the document, makes the `::after` pseudo-element to overflow the page to the right. Based on similar classes from other popular CSS frameworks, I made these pseudo-elements to have 1 x 1px dimensions. This does not affect negatively the screen readers. Example: - Bootstrap: https://github.com/twbs/bootstrap/blob/c2f67d72e8010307f657b2a55ae93fc5c9cab2a5/scss/mixins/_visually-hidden.scss#L9-L17 - Foundation: https://github.com/foundation/foundation-sites/blob/258d9eb3432d7e497f41b4554f4116d5c59c92a8/scss/util/_mixins.scss#L217-L224 Closes #2811
-
- 14 Jan, 2021 1 commit
-
-
Lyza Danger Gardner authored
Redesign the results count for the notebook to better handle its behavior when loading (sometimes large) sets of annotations. Previously, the annotation count used the `visibleCount` store method, which returns a count of annotations that are visible on-screen. While loading, use the service's reported `total` value to show the number of annotations that will be visible once loading is complete. Do not show thread count while loading, as this cannot be determined until all annotations are present and threaded. Fixes #2864
-
- 13 Jan, 2021 3 commits
-
-
Lyza Danger Gardner authored
In both the share-this-annotation and share-this-page's-annotations components, provide a UI path for when the annotation's URI or the page's URI (respectively) are not web-available. That is, to be "shareable in context", the URI in question needs to have an `http:` or `https:` protocol. For "share this page's annotations" when the page is not web-accessible, explain why the page's annotations can't be shared in context and don't provide a sharing link, as it won't work. For "share this annotation," when the annotation's URI (document) isn't web-accessible, provide some explanatory text about how it can't be shared in-context, but provide a link to the single-annotation view, when available. These changes are intended to avoid confusion when users try to share an annotation or a set of annotations that have been made on a local (e.g. PDF) document. Some SCSS patterns have also been adjusted to accommodate more flexible styling of the share-single-annotation panel. The width of the compact panel pattern has been increased slightly so that the new wording variant for not-shareable-in-context single annotations doesn't risk taking up too much vertical space. Fixes #2786
-
Lyza Danger Gardner authored
-
Lyza Danger Gardner authored
Add a typing utility module with a helper function to "cast" the input so that TS understands that it is not null/undefined. This is useful in situations where logic would prevent a reference from being empty but TS can't follow that logic.
-
- 11 Jan, 2021 6 commits
-
-
dependabot-preview[bot] authored
Bumps [enzyme-adapter-preact-pure](https://github.com/preactjs/enzyme-adapter-preact-pure) from 2.2.3 to 3.0.0. - [Release notes](https://github.com/preactjs/enzyme-adapter-preact-pure/releases) - [Changelog](https://github.com/preactjs/enzyme-adapter-preact-pure/blob/master/CHANGELOG.md) - [Commits](https://github.com/preactjs/enzyme-adapter-preact-pure/compare/v2.2.3...v3.0.0) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
dependabot-preview[bot] authored
Bumps [postcss](https://github.com/postcss/postcss) from 8.2.2 to 8.2.4. - [Release notes](https://github.com/postcss/postcss/releases) - [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md) - [Commits](https://github.com/postcss/postcss/compare/8.2.2...8.2.4) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
dependabot-preview[bot] authored
Bumps [sinon](https://github.com/sinonjs/sinon) from 9.2.2 to 9.2.3. - [Release notes](https://github.com/sinonjs/sinon/releases) - [Changelog](https://github.com/sinonjs/sinon/blob/master/CHANGELOG.md) - [Commits](https://github.com/sinonjs/sinon/compare/v9.2.2...v9.2.3) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
dependabot-preview[bot] authored
Bumps [autoprefixer](https://github.com/postcss/autoprefixer) from 10.1.0 to 10.2.1. - [Release notes](https://github.com/postcss/autoprefixer/releases) - [Changelog](https://github.com/postcss/autoprefixer/blob/main/CHANGELOG.md) - [Commits](https://github.com/postcss/autoprefixer/compare/10.1.0...10.2.1) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
dependabot-preview[bot] authored
Bumps [sass](https://github.com/sass/dart-sass) from 1.32.0 to 1.32.2. - [Release notes](https://github.com/sass/dart-sass/releases) - [Changelog](https://github.com/sass/dart-sass/blob/master/CHANGELOG.md) - [Commits](https://github.com/sass/dart-sass/compare/1.32.0...1.32.2) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
dependabot-preview[bot] authored
Bumps [aws-sdk](https://github.com/aws/aws-sdk-js) from 2.820.0 to 2.824.0. - [Release notes](https://github.com/aws/aws-sdk-js/releases) - [Changelog](https://github.com/aws/aws-sdk-js/blob/master/CHANGELOG.md) - [Commits](https://github.com/aws/aws-sdk-js/compare/v2.820.0...v2.824.0) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
- 08 Jan, 2021 2 commits
-
-
Kyle Keating authored
- Added override to button--icon-only mixin for min-size - Set the override value for menu__toggle to "inherit" This regression was caused when adding the buttons.button--icon-only mixin to the post annotation drop down toggle button (menu__toggle). This is indeed an icon button, but for visual fidelity, we are not going to bump the minimum size to the 44px in mobile view.
-
Lyza Danger Gardner authored
Fixes #2851
-
- 04 Jan, 2021 10 commits
-
-
Eduardo Sanz García authored
Previously, all users were able to click the flag icon. However, unauthenticated users were shown a toast message if they click the icon: `You must be logged in to report an annotation`. The behaviour has been changed so that the flag button is only shown (and clickable) for authenticated users. Resolves #1037
-
Lyza Danger Gardner authored
* It was unused * It had no tests * If there were tests, they might have kicked up that it's also buggy; it wasn't getting recalculated after threads were removed due to filtering For posterity: the equivalent is `thread.children.length`
-
dependabot-preview[bot] authored
Bumps [postcss](https://github.com/postcss/postcss) from 8.2.1 to 8.2.2. - [Release notes](https://github.com/postcss/postcss/releases) - [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md) - [Commits](https://github.com/postcss/postcss/compare/8.2.1...8.2.2) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
dependabot-preview[bot] authored
Bumps [sass](https://github.com/sass/dart-sass) from 1.30.0 to 1.32.0. - [Release notes](https://github.com/sass/dart-sass/releases) - [Changelog](https://github.com/sass/dart-sass/blob/master/CHANGELOG.md) - [Commits](https://github.com/sass/dart-sass/compare/1.30.0...1.32.0) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
dependabot-preview[bot] authored
Bumps [preact](https://github.com/preactjs/preact) from 10.5.7 to 10.5.9. - [Release notes](https://github.com/preactjs/preact/releases) - [Commits](https://github.com/preactjs/preact/compare/10.5.7...10.5.9) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
dependabot-preview[bot] authored
Bumps [core-js](https://github.com/zloirock/core-js) from 3.8.1 to 3.8.2. - [Release notes](https://github.com/zloirock/core-js/releases) - [Changelog](https://github.com/zloirock/core-js/blob/master/CHANGELOG.md) - [Commits](https://github.com/zloirock/core-js/compare/v3.8.1...v3.8.2) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
dependabot-preview[bot] authored
Bumps [eslint](https://github.com/eslint/eslint) from 7.16.0 to 7.17.0. - [Release notes](https://github.com/eslint/eslint/releases) - [Changelog](https://github.com/eslint/eslint/blob/master/CHANGELOG.md) - [Commits](https://github.com/eslint/eslint/compare/v7.16.0...v7.17.0) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
dependabot-preview[bot] authored
Bumps [aws-sdk](https://github.com/aws/aws-sdk-js) from 2.817.0 to 2.820.0. - [Release notes](https://github.com/aws/aws-sdk-js/releases) - [Changelog](https://github.com/aws/aws-sdk-js/blob/master/CHANGELOG.md) - [Commits](https://github.com/aws/aws-sdk-js/compare/v2.817.0...v2.820.0) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
dependabot-preview[bot] authored
Bumps [eslint-plugin-react](https://github.com/yannickcr/eslint-plugin-react) from 7.21.5 to 7.22.0. - [Release notes](https://github.com/yannickcr/eslint-plugin-react/releases) - [Changelog](https://github.com/yannickcr/eslint-plugin-react/blob/master/CHANGELOG.md) - [Commits](https://github.com/yannickcr/eslint-plugin-react/compare/v7.21.5...v7.22.0) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
Robert Knight authored
https://github.com/hypothesis/h/pull/5862 introduced an optimization to startup of the sidebar app by preloading the `/api/` and `/api/links` requests. Because this change was made in h it only benefitted the embedded client, but not the browser extension. This commit moves the logic to add the `<link rel="preload">` links to the client's boot script. The upside of this is that it works with all distributions of the client and can be changed in future without needing to modify h and the browser extension etc. separately. The downside is that because the `<link>` is no longer in the sidebar app's HTML but in the boot script the prefetch will start a little later. Since Cloudflare caches the `/api/` and `/api/links` responses, these requests are still very likely to have completed by the time the client code starts executing.
-
- 28 Dec, 2020 1 commit
-
-
dependabot-preview[bot] authored
Bumps [aws-sdk](https://github.com/aws/aws-sdk-js) from 2.814.0 to 2.817.0. - [Release notes](https://github.com/aws/aws-sdk-js/releases) - [Changelog](https://github.com/aws/aws-sdk-js/blob/master/CHANGELOG.md) - [Commits](https://github.com/aws/aws-sdk-js/compare/v2.814.0...v2.817.0) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
- 26 Dec, 2020 3 commits
-
-
Kyle Keating authored
-
dependabot-preview[bot] authored
Bumps [browserify](https://github.com/browserify/browserify) from 16.5.2 to 17.0.0. - [Release notes](https://github.com/browserify/browserify/releases) - [Changelog](https://github.com/browserify/browserify/blob/master/changelog.markdown) - [Commits](https://github.com/browserify/browserify/compare/v16.5.2...v17.0.0) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
dependabot-preview[bot] authored
Bumps [@sentry/browser](https://github.com/getsentry/sentry-javascript) from 5.29.0 to 5.29.2. - [Release notes](https://github.com/getsentry/sentry-javascript/releases) - [Changelog](https://github.com/getsentry/sentry-javascript/blob/master/CHANGELOG.md) - [Commits](https://github.com/getsentry/sentry-javascript/compare/5.29.0...5.29.2) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
- 21 Dec, 2020 3 commits
-
-
dependabot-preview[bot] authored
Bumps [@babel/preset-env](https://github.com/babel/babel/tree/HEAD/packages/babel-preset-env) from 7.12.10 to 7.12.11. - [Release notes](https://github.com/babel/babel/releases) - [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md) - [Commits](https://github.com/babel/babel/commits/v7.12.11/packages/babel-preset-env) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
dependabot-preview[bot] authored
Bumps [aws-sdk](https://github.com/aws/aws-sdk-js) from 2.809.0 to 2.814.0. - [Release notes](https://github.com/aws/aws-sdk-js/releases) - [Changelog](https://github.com/aws/aws-sdk-js/blob/master/CHANGELOG.md) - [Commits](https://github.com/aws/aws-sdk-js/compare/v2.809.0...v2.814.0) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
dependabot-preview[bot] authored
Bumps [dompurify](https://github.com/cure53/DOMPurify) from 2.2.2 to 2.2.6. - [Release notes](https://github.com/cure53/DOMPurify/releases) - [Commits](https://github.com/cure53/DOMPurify/compare/2.2.2...2.2.6) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-