From 70160fe01c28b835d8c0a8e4484127e08468e93e Mon Sep 17 00:00:00 2001 From: Dmitry S Date: Thu, 9 Sep 2021 23:26:43 -0400 Subject: [PATCH] (core) Rename page option 'Duplicate' to 'Duplicate Page', and shorten warning to be more noticeable Test Plan: Changes too minor to affect tests Reviewers: paulfitz Reviewed By: paulfitz Differential Revision: https://phab.getgrist.com/D3016 --- app/client/components/duplicatePage.ts | 7 +++---- app/client/ui2018/pages.ts | 2 +- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/app/client/components/duplicatePage.ts b/app/client/components/duplicatePage.ts index feffaaec..64db9561 100644 --- a/app/client/components/duplicatePage.ts +++ b/app/client/components/duplicatePage.ts @@ -22,13 +22,12 @@ export async function duplicatePage(gristDoc: GristDoc, pageId: number) { confirmModal('Duplicate page', 'Save', () => makeDuplicate(gristDoc, pageId, inputEl.value), ( dom('div', [ - "Enter name for the new page. ", - "Note that this does not copy data, ", - "but creates another view of the same data. ", cssField( cssLabel("Name"), inputEl = cssInput({value: pageName + ' (copy)'}), - ) + ), + "Note that this does not copy data, ", + "but creates another view of the same data.", ]) )); } diff --git a/app/client/ui2018/pages.ts b/app/client/ui2018/pages.ts index d43e7766..27f59b05 100644 --- a/app/client/ui2018/pages.ts +++ b/app/client/ui2018/pages.ts @@ -30,7 +30,7 @@ export function buildPageDom(name: Observable, actions: PageActions, ... dom.cls('disabled', actions.isReadonly)), menuItem(actions.onRemove, 'Remove', testId('remove'), dom.cls('disabled', (use) => use(actions.isReadonly) || actions.isRemoveDisabled())), - menuItem(actions.onDuplicate, 'Duplicate', testId('duplicate'), + menuItem(actions.onDuplicate, 'Duplicate Page', testId('duplicate'), dom.cls('disabled', actions.isReadonly)), dom.maybe(actions.isReadonly, () => menuText('You do not have edit access to this document')), ];