Fix Check for updates menu item title capitalisation (#7396)

* Check for updates -> Check for Updates

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

* Revert debugging changes

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

* Adding ellipsis to every Check for Updates items

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

---------

Signed-off-by: Alex Andreev <alex.andreev.email@gmail.com>
This commit is contained in:
Alex Andreev authored and GitHub committed 2023-03-23 11:37:10 +03:00
1 parent 4b1d740d61
commit 4a3f08d798
3 files changed
+7 -7

No files matched your search

@@ -28,7 +28,7 @@ const checkForUpdatesMenuItemInjectable = getInjectable({
id: "check-for-updates", id: "check-for-updates",
parentId: isMac ? "mac" : "help", parentId: isMac ? "mac" : "help",
orderNumber: isMac ? 20 : 50, orderNumber: isMac ? 20 : 50,
label: "Check for updates", label: "Check for Updates...",
isShown: updatingIsEnabled, isShown: updatingIsEnabled,
onClick: () => { onClick: () => {
@@ -146,7 +146,7 @@ describe("installing update using tray", () => {
it("name of tray item for checking updates indicates that checking is happening", () => { it("name of tray item for checking updates indicates that checking is happening", () => {
expect( expect(
builder.tray.get("check-for-updates")?.label, builder.tray.get("check-for-updates")?.label,
).toBe("Checking for updates..."); ).toBe("Checking for Updates...");
}); });
it("user cannot install update yet", () => { it("user cannot install update yet", () => {
@@ -177,7 +177,7 @@ describe("installing update using tray", () => {
it("name of tray item for checking updates no longer indicates that checking is happening", () => { it("name of tray item for checking updates no longer indicates that checking is happening", () => {
expect( expect(
builder.tray.get("check-for-updates")?.label, builder.tray.get("check-for-updates")?.label,
).toBe("Check for updates"); ).toBe("Check for Updates...");
}); });
it("renders", () => { it("renders", () => {
@@ -241,7 +241,7 @@ describe("installing update using tray", () => {
it("name of tray item for checking updates no longer indicates that downloading is happening", () => { it("name of tray item for checking updates no longer indicates that downloading is happening", () => {
expect( expect(
builder.tray.get("check-for-updates")?.label, builder.tray.get("check-for-updates")?.label,
).toBe("Check for updates"); ).toBe("Check for Updates...");
}); });
it("renders", () => { it("renders", () => {
@@ -269,7 +269,7 @@ describe("installing update using tray", () => {
it("name of tray item for checking updates no longer indicates that downloading is happening", () => { it("name of tray item for checking updates no longer indicates that downloading is happening", () => {
expect( expect(
builder.tray.get("check-for-updates")?.label, builder.tray.get("check-for-updates")?.label,
).toBe("Check for updates"); ).toBe("Check for Updates...");
}); });
it("renders", () => { it("renders", () => {
@@ -47,10 +47,10 @@ const checkForUpdatesTrayItemInjectable = getInjectable({
} }
if (checkingForUpdatesState.value.get()) { if (checkingForUpdatesState.value.get()) {
return "Checking for updates..."; return "Checking for Updates...";
} }
return "Check for updates"; return "Check for Updates...";
}), }),
enabled: computed(() => !checkingForUpdatesState.value.get() && !downloadingUpdateState.value.get()), enabled: computed(() => !checkingForUpdatesState.value.get() && !downloadingUpdateState.value.get()),