Paul Rouget [:paul] Reporter
	Description • 12 years ago

      No description provided.

	Girish Sharma [:Optimizer]
	Comment 1 • 12 years ago

If we want "Search CSS Selectors" then we need to increase the width of the search box to around 200px.
If we want "Search Selectors", then the current width is alright.

In both the options, I think it becomes less clearer to the user that searching here will return and select an HTML element.

Even though the search box takes CSS selectors as input, its final search result is an HTML element only.

What do you think Paul ?

	Paul Rouget [:paul] Reporter
	Comment 2 • 12 years ago

The icon obviously suggest a "search" functionality. No need to add "search" in the placeholder. What about just "CSS Selector"?

	Girish Sharma [:Optimizer]
	Comment 3 • 12 years ago

(In reply to Paul Rouget [:paul] from comment #2)
> The icon obviously suggest a "search" functionality. No need to add "search"
> in the placeholder. What about just "CSS Selector"?

No, if we want to be consistent with every other placeholder for a search box in the whole Firefox UI. Like the debugger's say "Filter Scripts", Web Console's says "Filter", Inspector's Computed View's say "Search". So basically every placeholder says either to search or to filter.

We can go with "Search Nodes" or "Search Elements" and then add a tooltip to tell that it takes CSS selectors as input ?

	Paul Rouget [:paul] Reporter
	Comment 4 • 12 years ago

(In reply to Girish Sharma [:Optimizer] from comment #3)
> We can go with "Search Nodes" or "Search Elements" and then add a tooltip to
> tell that it takes CSS selectors as input ?

Ok.

	Girish Sharma [:Optimizer]
	Comment 5 • 12 years ago

I suggest "Search Elements" as placeholder and "Search for a CSS selector" as the tooltip. This will also cover the use case that inspector can be used on xul pages and thus "Search HTML" makes no sense there.

@Paul, what do you think ?

	J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)
	Comment 6 • 12 years ago

To toss in another perspective, I did not realize until I saw this bug that the search box expects a CSS selector, and leaving that hidden in a tooltip well probably leave others confused as well.

I would prefer to see "selector" somewhere in the placeholder if possible, though I understand you don't want the box to be too big. Perhaps "Search by Selector" (or "on" / "via" in place of "by")?

Just a suggestion from a outsider's perspective. :)

	Girish Sharma [:Optimizer]
	Comment 7 • 12 years ago

(In reply to J. Ryan Stinnett [:jryans] from comment #6)
> To toss in another perspective, I did not realize until I saw this bug that
> the search box expects a CSS selector, and leaving that hidden in a tooltip
> well probably leave others confused as well.
> 
> I would prefer to see "selector" somewhere in the placeholder if possible,
> though I understand you don't want the box to be too big. Perhaps "Search by
> Selector" (or "on" / "via" in place of "by")?
> 
> Just a suggestion from a outsider's perspective. :)

I like "Search by Selector", if that does not overflow on Linux (as fonts are bigger there) then we can go with it :)

	Girish Sharma [:Optimizer]
	Comment 8 • 12 years ago

(In reply to J. Ryan Stinnett [:jryans] from comment #6)
> To toss in another perspective, I did not realize until I saw this bug that
> the search box expects a CSS selector, and leaving that hidden in a tooltip
> well probably leave others confused as well.

The suggestion popup that will soon land make things much more clearer and visible. As soon as the user types any alphabet, some suggestion will show up and he will know that what is the searchbox doing. So I think that this feature should not be hidden anymore then.

	Paul Rouget [:paul] Reporter
	Comment 9 • 11 years ago

Do we still want to fix this?

	Girish Sharma [:Optimizer]
	Comment 10 • 11 years ago

I am impartial to "Search by Selector" string vs the current. But we might want bug 835896 fixed, if yes, then after that the opinion might change.

	Sami Jaktholm
	Comment 12 • 9 years ago

Bug 991745 changed the placeholder to "Search with CSS Selectors".