Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[HOLD for new search] Shortcuts - Subtitle update is required for CTRL+K shortcut because old "Search" is renamed #43566

Closed
6 tasks done
lanitochka17 opened this issue Jun 12, 2024 · 13 comments
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Needs Reproduction Reproducible steps needed Not a priority Planning Changes still in the thought process Weekly KSv2

Comments

@lanitochka17
Copy link

lanitochka17 commented Jun 12, 2024

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!


Version Number: 1.4.82-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team

Action Performed:

  1. Go to staging.new.expensify.com
  2. Open Keyboard shortcuts menu via CTRL+J

Expected Result:

The subtitle for CTRL+K should be updated to "Open find" or "Open chat finder", because the old Search has been renamed and the new Search is another thing

Actual Result:

The subtitle for CTRL+K is still "Open search dialog"

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

  • Android: Native
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

Bug6510241_1718167137783.bandicam_2024-06-12_12-28-46-015.mp4

377

View all open jobs on GitHub

@lanitochka17 lanitochka17 added Daily KSv2 Bug Something is broken. Auto assigns a BugZero manager. labels Jun 12, 2024
Copy link

melvin-bot bot commented Jun 12, 2024

Triggered auto assignment to @mallenexpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

@lanitochka17
Copy link
Author

@mallenexpensify FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

@lanitochka17
Copy link
Author

We think that this bug might be related to #vip-vsp

@etCoderDysto
Copy link
Contributor

etCoderDysto commented Jun 12, 2024

Proposal

Please re-state the problem that we are trying to solve in this issue.

The subtitle for CTRL+K is still "Open search dialog".

What is the root cause of that problem?

The root cause is that we didn't update the copy associated with keyboardShortcutsPage.shortcuts.search key in en.ts and es.ts file.

App/src/languages/en.ts

Lines 2781 to 2787 in 8d11d0b

keyboardShortcutsPage: {
title: 'Keyboard shortcuts',
subtitle: 'Save time with these handy keyboard shortcuts:',
shortcuts: {
openShortcutDialog: 'Opens the keyboard shortcuts dialog',
escape: 'Escape dialogs',
search: 'Open search dialog',

What changes do you think we should make in order to solve the problem?

we can update the copeis on both files to be search: 'Open chat finder' and change the Spanish copy to search: 'Abrir buscador de chat'

What alternative solutions did you explore? (Optional)

We can rename the SEARCH object and its descriptionKey property to 'FIND' and 'find' here, and add 'find' key to keyboardShortcutsPage.shortcuts in en.ts and es.ts file with their corresponding updated copy.

Result:

Screen.Recording.2024-06-12.at.4.25.30.in.the.afternoon.mp4

@melvin-bot melvin-bot bot added the Overdue label Jun 14, 2024
@mallenexpensify mallenexpensify added Weekly KSv2 Needs Reproduction Reproducible steps needed and removed Daily KSv2 labels Jun 14, 2024
@melvin-bot melvin-bot bot removed the Overdue label Jun 14, 2024
@mallenexpensify mallenexpensify changed the title Shortcuts - Subtitle update is required for CTRL+K shortcut because old "Search" is renamed [HOLD for new search] Shortcuts - Subtitle update is required for CTRL+K shortcut because old "Search" is renamed Jun 14, 2024
@MelvinBot
Copy link

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

@mallenexpensify mallenexpensify added the Planning Changes still in the thought process label Jun 14, 2024
@mallenexpensify
Copy link
Contributor

We have 'new search' coming out soon. I want to hold on that project to ensure we get the copy correct.

@mallenexpensify
Copy link
Contributor

Holding on new search

@mallenexpensify mallenexpensify removed the Weekly KSv2 label Jun 26, 2024
@melvin-bot melvin-bot bot removed the Overdue label Jun 26, 2024
@melvin-bot melvin-bot bot added the Overdue label Jul 29, 2024
@mallenexpensify
Copy link
Contributor

Holding on new search

@melvin-bot melvin-bot bot removed the Overdue label Jul 31, 2024
@melvin-bot melvin-bot bot added the Overdue label Sep 2, 2024
@mallenexpensify
Copy link
Contributor

Still holding

@melvin-bot melvin-bot bot removed the Overdue label Sep 4, 2024
@melvin-bot melvin-bot bot added the Overdue label Oct 7, 2024
@mallenexpensify
Copy link
Contributor

Hold

@melvin-bot melvin-bot bot removed the Overdue label Oct 9, 2024
@mallenexpensify
Copy link
Contributor

ooooh, some version of new search is out, bumping to weekly

@mallenexpensify mallenexpensify added Weekly KSv2 and removed Monthly KSv2 labels Nov 6, 2024
@melvin-bot melvin-bot bot added the Overdue label Nov 15, 2024
@mallenexpensify
Copy link
Contributor

mallenexpensify commented Nov 20, 2024

Checking on in #expensify-open-source

Coming from this GH, we have Search in the bottom, middle of LHN and it leads to expense search. We also have 🔍 in upper right corner that shows Search as the tooltip. Are we planning to keep these the same or update the text for one?

@mallenexpensify
Copy link
Contributor

No action on the post, marking 'not a priority' and closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Needs Reproduction Reproducible steps needed Not a priority Planning Changes still in the thought process Weekly KSv2
Projects
None yet
Development

No branches or pull requests

4 participants