- 24 Jun, 2019 1 commit
-
-
dependabot-preview[bot] authored
Bumps [focus-visible](https://github.com/WICG/focus-visible) from 5.0.0 to 5.0.1. - [Release notes](https://github.com/WICG/focus-visible/releases) - [Commits](https://github.com/WICG/focus-visible/compare/v5.0.0...v5.0.1) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
- 20 Jun, 2019 4 commits
-
-
Robert Knight authored
Add "Copy link" action to groups menu
-
Robert Knight authored
-
Robert Knight authored
-
Robert Knight authored
Feedback on the first iteration was that "Copy link" was ambiguous. Add the term "invite" or "activity" to the label to indicate what the link is for, depending on whether it is a group that the user can join or not.
-
- 19 Jun, 2019 5 commits
-
-
Robert Knight authored
Replace references to old groups menu styles in new component
-
Robert Knight authored
Update icon for "New private group" menu item
-
Robert Knight authored
Update to latest icon from https://hypothicons.netlify.com.
-
Robert Knight authored
Add an action to provide a convenient way to copy the sharing link directly from the groups menu. Depending on whether the group is private or open and whether the receiving user is a member or not, the link may present the user with a Join page or the activity page for the group. To simplify the client logic, the text just says "Copy link".
-
Robert Knight authored
I evaluated popular npm packages for this purpose (eg. [1]) but encountered issues where they were not transpiled for use in older browsers or included workarounds for browsers we no longer support (eg. IE 9). If we do decide to use an existing package in future, we can keep this utility module as a wrapper around it. [1] https://github.com/sindresorhus/copy-text-to-clipboard
-
- 18 Jun, 2019 21 commits
-
-
Robert Knight authored
This replaces some references to the styles for the old groups menu in the new implementation. The menu label is now slightly darker as it uses the same grey as the menu items. Otherwise there should be no noticeable visual changes. The CSS itself has been simplified compared to the previous implementation.
-
Robert Knight authored
-
Robert Knight authored
Update group menu styles
-
Robert Knight authored
-
Robert Knight authored
This enables consumers to use `&&` or `||` conditionals to compute the `icon` prop.
-
Robert Knight authored
Convert `<search-input>` component to Preact
-
Robert Knight authored
-
Robert Knight authored
- {show, hide} => {renders, doesn't render} to clarify that we're checking whether an element is rendered at all, as opposed to merely visually hidden - Replace ambiguous GitHub issue reference with URL
-
Robert Knight authored
This makes it easier to see when a submenu item is hovered.
-
Robert Knight authored
- Use the string "blank" instead of `null` as a more explicit indicator that a menu item should show a blank space for an icon - Rename two variables to indicate their boolean-ness
-
Robert Knight authored
The icon is from the Hypothicons set at https://hypothicons.netlify.com
-
Robert Knight authored
-
Robert Knight authored
Following a design review, adjust the styling of menus and in particular the groups menu to better match the mocks [1] and create better separation between different elements. - Add a 10px margin below section headers - Add a subtle border at bottom of expanded submenus - Remove red hover color for menu items - Make submenu items slightly lighter and darken them on hover - Reduce font size to match top bar and reduce font weight for submenu items - Increase submenu toggle caret size slightly (10 => 12px) and darken the caret on hover, consistent with the effect when submenu items are hovered. - Avoid hyphenating messages in the footer of group list submenus [1] https://sketch.cloud/s/D0OMk/a/EbnxQA/play
-
Robert Knight authored
Bump focus-visible from 4.1.5 to 5.0.0
-
Robert Knight authored
Bump @octokit/rest from 16.28.0 to 16.28.2
-
Robert Knight authored
- Remove `@import` commands which duplicated SASS imports elsewhere in the bundle - Remove `:focus` selector which had no effect - Remove an unnecessary `@at-root` block which was used to contain a variable that was only used in one place and replace that usage with a constant instead
-
Robert Knight authored
- Rename `simple-search.scss` and the CSS classes it defines to match the name of the module which defines the component structure and logic - Remove reference to `simple-search-inactive` CSS class which did not exist
-
Robert Knight authored
-
Robert Knight authored
Merge pull request #1176 from hypothesis/dependabot/npm_and_yarn/babel-plugin-transform-async-to-promises-0.8.12 Bump babel-plugin-transform-async-to-promises from 0.8.11 to 0.8.12
-
Robert Knight authored
Bump enzyme-adapter-preact-pure from 1.13.4 to 2.0.0
-
Robert Knight authored
This test happened to pass before, but failed after an adapter upgrade. All external state changes should be wrapped in `act` to flush state updates and effects and followed by `wrapper.update` to update the Enzyme adapter.
-
- 17 Jun, 2019 4 commits
-
-
dependabot-preview[bot] authored
Bumps [@octokit/rest](https://github.com/octokit/rest.js) from 16.28.0 to 16.28.2. - [Release notes](https://github.com/octokit/rest.js/releases) - [Commits](https://github.com/octokit/rest.js/compare/v16.28.0...v16.28.2) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
dependabot-preview[bot] authored
Bumps [enzyme-adapter-preact-pure](https://github.com/preactjs/enzyme-adapter-preact-pure) from 1.13.4 to 2.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/v1.13.4...v2.0.0) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
dependabot-preview[bot] authored
Bumps [babel-plugin-transform-async-to-promises](https://github.com/rpetrich/babel-plugin-transform-async-to-promises) from 0.8.11 to 0.8.12. - [Release notes](https://github.com/rpetrich/babel-plugin-transform-async-to-promises/releases) - [Commits](https://github.com/rpetrich/babel-plugin-transform-async-to-promises/compare/v0.8.11...v0.8.12) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
dependabot-preview[bot] authored
Bumps [focus-visible](https://github.com/WICG/focus-visible) from 4.1.5 to 5.0.0. - [Release notes](https://github.com/WICG/focus-visible/releases) - [Commits](https://github.com/WICG/focus-visible/compare/v4.1.5...v5.0.0) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
-
- 14 Jun, 2019 2 commits
-
-
Robert Knight authored
Convert loading spinner to Preact
-
Robert Knight authored
-
- 13 Jun, 2019 3 commits
-
-
Hannah Stepanek authored
Remove passing of search
-
Robert Knight authored
-
Hannah Stepanek authored
Previously a search object was built inside the hypothesis-app component and passed down to its child components. Now, just use the store.getState().filterQuery directly. This avoids unnecessarily passing things between components and a layer of indirection between the search object and the value in the store. Rename searchQuery to filterQuery.
-