- 01 Mar, 2021 12 commits
-
-
dependabot[bot] authored
Bumps [typescript](https://github.com/Microsoft/TypeScript) from 4.1.5 to 4.2.2. - [Release notes](https://github.com/Microsoft/TypeScript/releases) - [Commits](https://github.com/Microsoft/TypeScript/compare/v4.1.5...v4.2.2) Signed-off-by: dependabot[bot] <support@github.com>
-
Robert Knight authored
The `guest` argument is required but the `config` argument is now optional. Swap them so that required arguments come before optional ones and set a default value for `config`.
-
Robert Knight authored
Change the relationship between the `Guest` and `Sidebar` classes from one of inheritance to one where `Sidebar` receives a reference to the `Guest` instance and calls methods on it or subscribes to events from it. The `Guest` also no longer "owns" references to the bucket bar / toolbar objects that logically belong to the sidebar. Instead it emits events which the sidebar responds to. This change makes the interface of the `Guest` used by the `Sidebar` more explicit and ensures a better boundary between the two. This also makes it easier for the `Sidebar` tests to be concerned only with the interface of the `Guest` and not its implementation details. In future this change will also make it possible to have a frame which does not contain a sidebar but is not annotateable. We had a need for this historically when integrating with epub viewers, although it was never implemented. Updating the tests for `PdfSidebar` was complicated by the fact that the `PdfSidebar` tests are not pure unit tests. They instantiate the `Sidebar` base class and so depend on many implementation details of it. To make this change and others easier, the `PdfSidebar` tests have been changed to mock the `Sidebar` base class. The steps involved in this are non-obvious so I extracted the logic into a utility function. - Change `Sidebar` to no longer inherit `Guest` but accept it as a constructor argument - Remove direct references to the `BucketBar` and `ToolbarController` instances from the `Guest` class and instead emit events from the `Guest` which the `Sidebar` responds to. - Add a `mockBaseClass` testing helper in `src/test-util/mock-base.js` and change the `PdfSidebar` tests to use it, so that they are less coupled to `Sidebar` implementation details.
-
dependabot[bot] authored
Bumps [@babel/core](https://github.com/babel/babel/tree/HEAD/packages/babel-core) from 7.12.17 to 7.13.8. - [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.13.8/packages/babel-core) Signed-off-by: dependabot[bot] <support@github.com>
-
dependabot[bot] authored
Bumps [aws-sdk](https://github.com/aws/aws-sdk-js) from 2.848.0 to 2.853.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.848.0...v2.853.0) Signed-off-by: dependabot[bot] <support@github.com>
-
dependabot[bot] authored
Bumps [@babel/preset-env](https://github.com/babel/babel/tree/HEAD/packages/babel-preset-env) from 7.12.17 to 7.13.8. - [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.13.8/packages/babel-preset-env) Signed-off-by: dependabot[bot] <support@github.com>
-
dependabot[bot] authored
Bumps [eslint](https://github.com/eslint/eslint) from 7.20.0 to 7.21.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.20.0...v7.21.0) Signed-off-by: dependabot[bot] <support@github.com>
-
dependabot[bot] authored
Bumps [core-js](https://github.com/zloirock/core-js/tree/HEAD/packages/core-js) from 3.9.0 to 3.9.1. - [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/commits/v3.9.1/packages/core-js) Signed-off-by: dependabot[bot] <support@github.com>
-
dependabot[bot] authored
Bumps [puppeteer](https://github.com/puppeteer/puppeteer) from 7.1.0 to 8.0.0. - [Release notes](https://github.com/puppeteer/puppeteer/releases) - [Changelog](https://github.com/puppeteer/puppeteer/blob/main/CHANGELOG.md) - [Commits](https://github.com/puppeteer/puppeteer/compare/v7.1.0...v8.0.0) Signed-off-by: dependabot[bot] <support@github.com>
-
dependabot[bot] authored
Bumps [@octokit/rest](https://github.com/octokit/rest.js) from 18.2.0 to 18.3.0. - [Release notes](https://github.com/octokit/rest.js/releases) - [Commits](https://github.com/octokit/rest.js/compare/v18.2.0...v18.3.0) Signed-off-by: dependabot[bot] <support@github.com>
-
Eduardo Sanz García authored
The purpose of this PR is to be able to easily run karma tests in other browsers without manually editing the karma configuration or making local copies of the karma configuration. Additions: - `yarn test --no-browser` to avoid launching the default browser. It is up to the developer to run the tests in whatever browser she/he chooses by navigating to http://localhost:9876/. - `yarn test --browser <browser>` run the tests in a different browser/s instead of the default browser. Chrome launcher comes by default with karma. Firefox, Safari and others can be installed independently. Modifications: - `yarn gulp test-watch` has been substituted by `yarn test --watch` for consistency.
-
Robert Knight authored
The `disableShadowSidebar` flag and the corresponding behavior no longer exists.
-
- 26 Feb, 2021 4 commits
-
-
Robert Knight authored
The `ToolbarController#getWidth` method started incorrectly returning 0 after 243c06d3. This commit moved the content of the toolbar to a child of `ToolbarController._container` and the `getWidth` method was not updated accordingly. The incorrect result of `this.toolbar.getWidth()` in `Sidebar` later resulted in an incorrect calculation of the sidebar's width and caused `PdfSidebar` to think that the window was too narrow to support side-by-side mode. Fix the problem by updating `ToolbarController#getWidth` so that it returns the correct value again.
-
Lyza Danger Gardner authored
-
Eduardo Sanz García authored
Convert a statement into an if/else for readability.
-
Eduardo Sanz García authored
- Remove `disableShadowSidebar` flag - Simplify code structure - Remove iframe from externalContainer on `destroy`. Closes #2908
-
- 25 Feb, 2021 10 commits
-
-
Kyle Keating authored
This prevents long lists from going off-screen and becoming unwieldy.
-
Kyle Keating authored
If the current user shows up in the filter list. (e.g they have annotations), then sort that user to the front of the list and add " (Me)" suffix to the end of the displayed name to make it easier for a user to find and filter their own annotations.
-
Eduardo Sanz García authored
This small trick borrowed from https://css-tricks.com/preventing-a-grid-blowout/ keeps the grid content within the grid boundaries, thus improving the responsive design.
-
Eduardo Sanz García authored
In a PR #3055, we remove reference to the plugin system for the BucketBar and we create a property `bucketBar` in `Guest` class. I overlook one of this reference in the `sidebar`.
-
Eduardo Sanz García authored
BucketBar is not used anymore as a plugin.
-
Eduardo Sanz García authored
- Added types - Moved out of the plugin directory - Avoid extending from Delegator (unnecessary)
-
Eduardo Sanz García authored
The sidebar component directs the opening of the notebook. This is for two reasons: * it's a little bit easier to save the UI state of the current sidebar (immediately before the notebook is opened), in case in the future we want to restored it. * to avoid adding additional events, like `show/hideSidebarControls`, that probably would be needed for granular control of the sidebar, in case the notebook would be directing the process of closing/re-opening the sidebar.
-
Eduardo Sanz García authored
Because I am planning to introduce another method to hide the controls, I renamed the hide/show methods in the sidebar and notebook components to open/close. I also feel it describes better the visual outcome. For consistency, I renamed the show/hideNotebook to open/closeNotebook.
-
- 24 Feb, 2021 1 commit
-
-
Eduardo Sanz García authored
Moved the user-filter below the heading and added a little bit of top and bottom space.
-
- 23 Feb, 2021 4 commits
-
-
Robert Knight authored
Rename CSS class names for `Toolbar` component to match the component itself. This component name is really overly generic but at least this brings the CSS and Preact components into line. One of the toolbar's classes was applied to a container element created by `ToolbarController`. This class has been moved to an element created by the `Toolbar` component for consistency. This means there is an apparently redundant (no styling, one child) container div when looking at the DOM structure for the toolbar + bucket bar. This will go away if the whole sidebar container is made into a single Preact component in future.
-
Robert Knight authored
Rename all CSS classes used by the `AdderToolbar` component to follow the `Component__element--modifier` syntax that we use for other components. Also rename the `Adder.scss` file to match the Preact component name.
-
Robert Knight authored
This is not the correct solution, but as only one import is affected, this is a simple workaround with no effect on other code.
-
dependabot[bot] authored
Bumps [escape-string-regexp](https://github.com/sindresorhus/escape-string-regexp) from 1.0.5 to 4.0.0. - [Release notes](https://github.com/sindresorhus/escape-string-regexp/releases) - [Commits](https://github.com/sindresorhus/escape-string-regexp/compare/v1.0.5...v4.0.0) Signed-off-by: dependabot[bot] <support@github.com>
-
- 22 Feb, 2021 9 commits
-
-
dependabot[bot] authored
Bumps [terser](https://github.com/terser/terser) from 5.5.1 to 5.6.0. - [Release notes](https://github.com/terser/terser/releases) - [Changelog](https://github.com/terser/terser/blob/master/CHANGELOG.md) - [Commits](https://github.com/terser/terser/commits) Signed-off-by: dependabot[bot] <support@github.com>
-
dependabot[bot] authored
Bumps [core-js](https://github.com/zloirock/core-js) from 3.8.3 to 3.9.0. - [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.3...v3.9.0) Signed-off-by: dependabot[bot] <support@github.com>
-
Eduardo Sanz García authored
Because of recent changes in the sidebar (#2904), the BucketBar component is no longer used as a plugin. @robertknight suggested to remove all unneeded functionality. Closes #2907
-
Eduardo Sanz García authored
On PR #2904, it was forgotten to pass the bucket bar options to the constructor. Closes #3047
-
Robert Knight authored
Remove entries that refer to dependencies that are no longer used by the client or update restrictions that no longer apply. These were automatically imported from Dependabot's config and never got pruned when they became obsolete.
-
dependabot-preview[bot] authored
-
Lyza Danger Gardner authored
-
dependabot-preview[bot] authored
Bumps [typescript](https://github.com/Microsoft/TypeScript) from 4.1.3 to 4.1.5. - [Release notes](https://github.com/Microsoft/TypeScript/releases) - [Commits](https://github.com/Microsoft/TypeScript/compare/v4.1.3...v4.1.5) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
dependabot-preview[bot] authored
Bumps [mocha](https://github.com/mochajs/mocha) from 8.2.1 to 8.3.0. - [Release notes](https://github.com/mochajs/mocha/releases) - [Changelog](https://github.com/mochajs/mocha/blob/master/CHANGELOG.md) - [Commits](https://github.com/mochajs/mocha/compare/v8.2.1...v8.3.0) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-