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

[Follow up to #39552] Title for Edit button needs to be updated to Done when editing custom SE #41210

Closed
2 of 5 tasks
hffvld opened this issue Sep 23, 2024 · 4 comments · Fixed by brave/brave-core#25819
Closed
2 of 5 tasks
Assignees
Labels

Comments

@hffvld
Copy link
Contributor

hffvld commented Sep 23, 2024

Description

Follow-up to #39552.

Steps to reproduce

  1. Launch Brave
  2. Hamburger menu > Settings
  3. Search Engines > Add Custom Search Engine
  4. In URL type https://youtube.com/search?q=%s and in Title type YT > Done
  5. Slide the newly created SE to the left > Tap Edit
  6. Update the Title > Observe

Actual result

If editing custom SE (any of URL or Title), the button title (top right corner) says Edit instead of Done.


1 2
1 2

Expected result

The title for Edit button needs to be updated to Done when editing custom SE.

Reproduces how often

Easily reproduced

Brave version

Brave build: 1.70 (117)
BraveCore: 1.70.117 (129.0.6668.59)

Device/iOS version

iPad Mni (6th gen) / iPadOS 18

Affected browser versions

  • latest AppStore
  • latest TestFlight
  • previous TestFlight

Reproducibility

  • with Brave Shields disabled
  • in the latest version of mobile Safari

Miscellaneous information

No response

@hffvld hffvld added bug QA/Yes OS/iOS Fixes related to iOS browser functionality labels Sep 23, 2024
@kylehickinson
Copy link
Collaborator

It seems like it was explicitly set to "Edit" instead of "Done" when editing a custom search engine. Do we want to go back to "Done" here? Not sure on reasoning cc @jamesmudgett @timchilds

@timchilds
Copy link

@kylehickinson Done makes sense to me 👍

The user cannot save the custom engine edits until they press the button right?

@kylehickinson
Copy link
Collaborator

@timchilds Correct, they have to tap the button to save whatever changes they've made in the form

@hffvld
Copy link
Contributor Author

hffvld commented Nov 4, 2024

Verified on iPhone 14 using version(s):

Device/OS: iPhone 14 / iOS 17.7.1
Brave build: 1.72 (93)
BraveCore: 1.72.93 (131.0.6778.14)

STEPS:

  1. Follow the STR/TP from [Follow up to #39552] Title for Edit button needs to be updated to Done when editing custom SE #41210 (comment)
  2. Verify

ACTUAL RESULTS:

  • Verified that the title for the Edit button is updated to Done when editing custom SE.

1 2 3
1 2 3

@hffvld hffvld added QA Pass - iPhone and removed QA/In-Progress Indicates that QA is currently in progress for that particular issue labels Nov 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants