Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

Remove Shortcuts Button #5567

Closed
colintheshots opened this issue Sep 25, 2019 · 19 comments
Closed

Remove Shortcuts Button #5567

colintheshots opened this issue Sep 25, 2019 · 19 comments
Assignees
Labels
E2 Estimation Point: easy, half a day to 2 days eng:qa:verified QA Verified Feature:Search P3 Some future sprint
Milestone

Comments

@colintheshots
Copy link
Contributor

colintheshots commented Sep 25, 2019

Why/User Benefit/User Problem

The Shortcuts button now does nothing, since it's the default UX behavior for the search screen. It's always activated.

Some users find this annoying: https://www.reddit.com/r/firefox/comments/d7ak8s/does_anyone_know_how_can_i_disable_the_shortcuts/

What/Requirements

Remove the button.

Acceptance Criteria (how do I know when I’m done?)

The button will no longer appear on the search screen.

┆Issue is synchronized with this Jira Task

@colintheshots colintheshots added feature request 🌟 New functionality and improvements E1 Estimation Point: trivial, less than half day P3 Some future sprint labels Sep 25, 2019
@colintheshots colintheshots added this to To be Triaged in Fenix Sprint Kanban via automation Sep 25, 2019
@NotWoods NotWoods added Hacktoberfest Highlights issues for Hacktoberfest contributors to help us! help wanted Help wanted from a contributor. More complex than good first issue. labels Sep 25, 2019
@nisrulz
Copy link

nisrulz commented Sep 25, 2019

Hi,

I'd like to take a stab at this.

@ekager
Copy link
Contributor

ekager commented Sep 27, 2019

This still needs UX feedback

@ekager ekager removed Hacktoberfest Highlights issues for Hacktoberfest contributors to help us! help wanted Help wanted from a contributor. More complex than good first issue. labels Sep 27, 2019
@sblatz sblatz removed this from To be Triaged in Fenix Sprint Kanban Sep 27, 2019
@sblatz
Copy link
Contributor

sblatz commented Sep 27, 2019

@colintheshots Please do not add feature requests to the sprint board until they have been prioritized by @vesta0.

@sblatz
Copy link
Contributor

sblatz commented Sep 27, 2019

It's not true that this button "does nothing." I agree we should consider changing the default to not show search suggestions, but pressing it does hide them.

We could also allow users to disable "search shortcuts" in settings in this page (which would be very easy engineering effort)

image

@vesta0 vesta0 added the needs:UX-feedback Needs UX Feedback label Sep 28, 2019
@brampitoyo
Copy link

@colintheshots Thanks for filing this issue!

  • Let’s remove the Shortcuts button
  • Let’s add a new item under Settings → Search called “Show search shortcuts”, that allows users to turn off search shortcuts. It defaults to ON, and works exactly as @sblatz has described

Thanks heaps for the idea, @sblatz.

@brampitoyo brampitoyo added eng:ready Ready for engineering and removed needs:UX-feedback Needs UX Feedback labels Oct 2, 2019
@sblatz sblatz self-assigned this Oct 2, 2019
@sblatz sblatz added E2 Estimation Point: easy, half a day to 2 days and removed E1 Estimation Point: trivial, less than half day labels Oct 2, 2019
sblatz added a commit to sblatz/fenix that referenced this issue Oct 2, 2019
@Cheap-Skate
Copy link

Now the Shortcuts button is removed could the Scan button be removed or placed elsewhere?
That way we'd all win back a bit of vertical space to fill with search results. The small number of search results is IMHO a big step back from Fennec, see eg #1161, #479.

eg

  • put it where it is in Fennec, on the right hand side of the URL bar?
    or
  • leave it where it is, but make it disappear as soon as I start typing in the URL bar and give the vertical space back to search results (I'm unlikely to want to Scan after I have started typing???)

@brampitoyo
Copy link

@Cheap-Skate File an issue, and I’ll help triage it! When our team has some time available, we’ll consider it :-)

sblatz added a commit that referenced this issue Oct 3, 2019
* For #5567: Removes search shortcut button

* No issue: Simplifies logic for displaying shortcuts
@sblatz sblatz added the eng:qa:needed QA Needed label Oct 3, 2019
@sblatz sblatz added this to To be Triaged in Fenix Sprint Kanban via automation Oct 3, 2019
@sblatz sblatz removed the eng:qa:needed QA Needed label Oct 3, 2019
@project-bot project-bot bot moved this from To be Triaged to In Progress in Fenix Sprint Kanban Oct 3, 2019
@sblatz sblatz added the eng:qa:needed QA Needed label Oct 3, 2019
@sblatz sblatz added this to the v2.2 milestone Oct 3, 2019
@sblatz sblatz moved this from In Progress to Ready for QA in Fenix Sprint Kanban Oct 3, 2019
@sblatz sblatz added Feature:Search and removed eng:ready Ready for engineering feature request 🌟 New functionality and improvements labels Oct 3, 2019
@sv-ohorvath
Copy link
Contributor

Verified the shortcuts button is hidden now, the new preference in Search settings and no new issues found.
build: Nightly 10/4
device: Pixel 3 (Android 9), Huawei P9 lite (Android 6).

Fenix Sprint Kanban automation moved this from Ready for QA to Sprint 10.1 Done Oct 4, 2019
@sv-ohorvath sv-ohorvath added eng:qa:verified QA Verified and removed eng:qa:needed QA Needed labels Oct 4, 2019
@project-bot project-bot bot moved this from Sprint 10.1 Done to In Progress in Fenix Sprint Kanban Oct 4, 2019
@sblatz sblatz moved this from In Progress to Sprint 10.1 Done in Fenix Sprint Kanban Oct 4, 2019
@cadeyrn
Copy link
Contributor

cadeyrn commented Oct 6, 2019

Now it's no longer possible to change the search engine after you entered something into the address bar. This can be very annoying if you notice that you selected the wrong search engine after entering the search term because you have to remove the whole input, change the search engine and type it again… It's also a functional regression for the Fennec -> Fenix migration where you can change the search engine after entering the search term.

If the only reason to remove the button was that some people find it annoying, is the new preference in the search engines to disable the shortcuts not sufficient?

/cc @colintheshots @brampitoyo

@klint
Copy link

klint commented Oct 6, 2019

Maybe the button could be back when the preference is set to Off?

@Cheap-Skate
Copy link

Question:- How will the newly adopted search behavior (show search engines before typing, search engines disappear when typing starts) work with Top Sites which IIUC is coming soon? Presumably Top Sites will appear before I type & use a lot of the screen real estate which shows search engines. ie won't Top Sites and search engine list conflict over the same screen real estate?

Suggestion:- How about select a search engine at any time by tapping the search favicon at the left end of the URL bar (Google for me).

Agree current behavior is not as good as Fennec - again IMHO showing how excellent the Fennec Awesomescreen is/was. I shall really miss it.

@Cheap-Skate
Copy link

Re tapping the search favicon, Samsung Internet does this, I like it. Down pointing caret next to the search favicon in URL bar, tapping the search favicon drops down a list of search engines.
Screenshot_20191006-175224_Samsung Internet Beta
Screenshot_20191006-175248_Samsung Internet Beta

@yoasif
Copy link
Contributor

yoasif commented Oct 6, 2019

@cadeyrn Can you open a new issue for this regression? There is also a post on reddit: https://www.reddit.com/r/firefox/comments/de03eu/why_search_with_was_removed_now_u_can_change/

@brampitoyo
Copy link

@cadeyrn wrote:

If the only reason to remove the button was that some people find it annoying, is the new preference in the search engines to disable the shortcuts not sufficient?

I agree with you that having this preference is totally sufficient.

I wrote in #5567 (comment):

Let’s add a new item under Settings → Search called “Show search shortcuts”, that allows users to turn off search shortcuts. It defaults to ON […]

The default value should be ON. Meaning: the “Shortcuts” button will be available to all users unless you manually turn it off.

Would this fix the issue?

@klint
Copy link

klint commented Oct 7, 2019

The default value should be ON. Meaning: the “Shortcuts” button will be available to all users unless you manually turn it off.

Sorry, I'm lost: are you talking about the "Shortcuts" button or the shortcut entries above with the name of each search engine???
In the latest nightly, the setting deals with the shortcut entries, and the "Shortcuts" button is always hidden. What is the new fix you are suggesting then? Is it to display the button when the entries are displayed, and hide it when the entries are hidden?
Thanks :)

@Cheap-Skate
Copy link

Sorry for confusion, let me try again. My suggestion is to bring back the "Shortcuts" button which used to show/hide the list of search shortcuts, but instead of having it as a distinct button at the bottom of the page, put its behavior into the "currently selected" search favicon at the left hand side of the text entry bar. i.e. tap the "currently selected" search favicon to bring up the list of search shortcuts. This is how Samsung Internet does it.

This fixes cadeyrn's issue:-

Now it's no longer possible to change the search engine after you entered something into the address bar. ... It's also a functional regression for the Fennec -> Fenix migration where you can change the search engine after entering the search term.

It might also fix a future problem where Top Sites will conflict for screen real estate with current default behavior of showing the list of search shortcuts on opening a new tab:-

Top Sites are coming. Presumably the default behavior of Top Sites will be to appear as soon as I open a new tab with no text in the search bar. But Fenix's current default behavior is to show a list of search shortcuts when I open a new tab. What should Fenix show on opening a new tab:- Top Sites or list of search shortcuts? My proposed solution fixes this potential issue as well.

@klint
Copy link

klint commented Oct 7, 2019

Thanks @Cheap-Skate for that clarification.
@brampitoyo, is it the same fix you are referring to? :)

@sblatz
Copy link
Contributor

sblatz commented Oct 7, 2019

Now it's no longer possible to change the search engine after you entered something into the address bar. ...

If you being a search and then backspace until the text is empty, you will be presented with the search engine switcher again, so you can change it there. That being said, that's not necessarily super discoverable or intuitive.

@brampitoyo
Copy link

@Cheap-Skate Thanks for clarifying your suggestions on #5567 (comment).

[…] instead of having [search shortcuts] as a distinct button at the bottom of the page […] tap the "currently selected" search favicon to bring up the list of search shortcuts.

This fixes @cadeyrn’s issue: […] no longer possible to change the search engine after you entered something into the address bar.

It might also fix a future problem where Top Sites will conflict for screen real estate with […] the list of search shortcuts.

This is a totally separate issue from removing the Shortcuts button. So, will you file a new issue for us? We will definitely consider this as part of the larger search experience design (#5457).

Thanks heaps!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
E2 Estimation Point: easy, half a day to 2 days eng:qa:verified QA Verified Feature:Search P3 Some future sprint
Projects
None yet
Development

No branches or pull requests