Last modified: 2014-02-03 16:24:36 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T26248, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 24248 - Vector skin search "Go" button uses fulltext search button tooltip
Vector skin search "Go" button uses fulltext search button tooltip
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Interface (Other open bugs)
1.20.x
All All
: Normal normal (vote)
: 1.23.0 release
Assigned To: Bartosz Dziewoński
: easy
Depends on:
Blocks: 44881
  Show dependency treegraph
 
Reported: 2010-07-03 13:24 UTC by Lejonel
Modified: 2014-02-03 16:24 UTC (History)
6 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Lejonel 2010-07-03 13:24:20 UTC
The magnifying glass button in the Vector skin search box uses the fulltext search button text ([[:Mediawiki:Tooltip-search-fulltext]]) but it is a go search button. This can be solved in one of the following ways

# Change button function to fulltext search to match the tooltip
# Change the tooltip to [[Mediawiki:Tooltip-search-go]] to match the search button function
# Create a new Vector skin search tooltip, since the searching with a single button works different from searching with two search buttons in Monobook skin
# Add a fulltext search button and use the correct tooltips like in Monobook skin
Comment 1 Lejonel 2010-09-08 19:13:31 UTC
Changing from enhancement to normal bug. Since the tooltip says it searches for page text, even though the button only search for text if there is no page title match.
Comment 2 Thehelpfulone 2012-06-22 19:45:10 UTC
Reassign to default per bug 37789
Comment 3 Lejonel 2012-12-29 18:19:33 UTC
Now there is a fulltext option in the vector search box (I don't remember if this existed when the bug was reported, but I think not since that was part of my fourth suggested solution). This fulltext option uses its own new message [[MediaWiki:Searchsuggest-containing]]. But the vector magnifying glass still uses the incorrect fulltext search tooltip  from monobook skin.

Now the problem seems to be that vector skin uses 'image' as $mode for makeSearchButton(), and in includes/SkinTemplate.php that function assumes wrongly that 'image' mode is always fulltext search. 

I think the easiest way to fix this is to create a new message (maybe [[Mediawiki:Tooltip-search-image]]) specifically for this 'image search mode', with the text "Search for this text". Without mentioning if it is searching page titles or page content, since using an image as search button does not really say anything about if the search is "go", "fulltext", or something else completely.

(marking as "easy" since this should not be hard to change.)
Comment 4 Daniel Friesen 2012-12-29 20:09:17 UTC
Monobook and other skins can have a single 'Go' button with the same behaviour as the search image.
Comment 5 Lejonel 2012-12-30 18:23:17 UTC
> Monobook and other skins can have a single 'Go' button with the same
> behaviour as the search image.

Yes. I think this is because the newer image search button is behaving as the older 'Go' button. That is why the image search button should not use the tooltip of the fulltext search.

If image search button is always a 'Go' search, it could use the old 'Go' tooltip ([[Mediawiki:Tooltip-search-go]]: "Go to a page with this exact name if exists"). I am not sure that this is always the case, that is why I suggest creating a new tooltip message when an image is used as button.

Another solution is to make a difference between search modes ('go' or 'fulltext') and button modes (button with text or button with image). And letting the tooltip depend only on the search mode, not what type of button is used.
Comment 6 Gerrit Notification Bot 2013-08-31 20:24:36 UTC
Change 82100 had a related patch set uploaded by Matmarex:
Rewrite rendering of Vector simple search

https://gerrit.wikimedia.org/r/82100
Comment 7 Andre Klapper 2013-09-26 14:45:16 UTC
https://gerrit.wikimedia.org/r/#/c/82100/ has received one +1.
Comment 8 Gerrit Notification Bot 2014-02-03 02:30:26 UTC
Change 82100 merged by jenkins-bot:
Rewrite rendering of Vector simple search

https://gerrit.wikimedia.org/r/82100

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links