TopBar Update button (#5376)

* UpdateButton skeleton

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>

* Show UpdateButton in topbar

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>

* Default button styles

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>

* Add update icon to menu item

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>

* Replace colors for medium & high levels

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>

* Fine-tune colors for light theme

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>

* Fix keyboard events bug

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>

* Hide <UpdateButton> from topbar

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>

* Clean up

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>

* Using dropdown icon instead of sandwich

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>

* Used determenistic id for the button

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>

* Respect id prop

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>

* Add small space

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>

* Using accent color directly from active theme

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>

* Update snapshots

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>

* Fix open/close Menu errors

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>

* Linter fix

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>

* Fix linter harder

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>

* Clean up tests

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>

* Fix act() wrapper console.error

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>

* Remove aria-expanded attribute

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>
This commit is contained in:
Alex Andreev authored and GitHub committed 2022-05-18 19:36:40 +03:00
1 parent 54b92aa9b6
commit c2b4fed5c3
6 files changed
+224 -1

No files matched your search

@@ -11,7 +11,7 @@ import { Icon } from "../../icon";
import { observable } from "mobx";
import { ipcRendererOn } from "../../../../common/ipc";
import { watchHistoryState } from "../../../remote-helpers/history-updater";
import { cssNames } from "../../../utils";
import { cssNames, noop } from "../../../utils";
import topBarItemsInjectable from "./top-bar-items/top-bar-items.injectable";
import { withInjectables } from "@ogre-tools/injectable-react";
import type { TopBarRegistration } from "./top-bar-registration";
@@ -23,6 +23,7 @@ import type { NavigateToCatalog } from "../../../../common/front-end-routing/rou
import navigateToCatalogInjectable from "../../../../common/front-end-routing/routes/catalog/navigate-to-catalog.injectable";
import catalogRouteInjectable from "../../../../common/front-end-routing/routes/catalog/catalog-route.injectable";
import routeIsActiveInjectable from "../../../routes/route-is-active.injectable";
import { UpdateButton } from "../../update-button";
interface Dependencies {
navigateToCatalog: NavigateToCatalog;
@@ -129,6 +130,7 @@ const NonInjectedTopBar = observer(({
onClick={goForward}
disabled={!nextEnabled.get()}
/>
<UpdateButton update={noop} />
</div>
<div className={styles.items}>
{renderRegisteredItems(items.get())}