Last modified: 2013-12-03 16:13:46 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 T59024, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 57024 - Add text to Special:Search to inform people on wikis with Cirrus as primary
Add text to Special:Search to inform people on wikis with Cirrus as primary
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
CirrusSearch (Other open bugs)
unspecified
All All
: High normal (vote)
: ---
Assigned To: Nik Everett
: design
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-11-13 18:56 UTC by Dan Garry
Modified: 2013-12-03 16:13 UTC (History)
4 users (show)

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


Attachments
Picture of the text above the search box. (19.16 KB, image/png)
2013-11-14 21:38 UTC, Nik Everett
Details

Description Dan Garry 2013-11-13 18:56:05 UTC
So that people are adequately informed, any wiki that has CirrusSearch as primary should have the following wikitext added below the search box on [[Special:Search]]:

This wiki is now using CirrusSearch, MediaWiki's new search engine. ([[mw:Special:MyLanguage/Help:CirrusSearch|Learn more about CirrusSearch]])
Comment 1 Gerrit Notification Bot 2013-11-14 21:32:26 UTC
Change 95541 had a related patch set uploaded by Manybubbles:
Show notification on search page if configured

https://gerrit.wikimedia.org/r/95541
Comment 2 Nik Everett 2013-11-14 21:34:06 UTC
Woops, added it above.  Let me see if below is possible without stupid hacks.
Comment 3 Nik Everett 2013-11-14 21:38:43 UTC
Created attachment 13801 [details]
Picture of the text above the search box.

Looks like adding it below the search box would require a change to Mediawiki as well as Cirrus.  If we can get away with above I'd like to do so.  It'll keep our master branch compatible with 1.22
Comment 4 Dan Garry 2013-11-15 07:27:57 UTC
(In reply to comment #3)
> Created attachment 13801 [details]
> Picture of the text above the search box.
> 
> Looks like adding it below the search box would require a change to Mediawiki
> as well as Cirrus.  If we can get away with above I'd like to do so.  It'll
> keep our master branch compatible with 1.22

Where does this patch put the notice? I'm not too concerned about the details of the actual location, as long as it's clearly visible.
Comment 5 Nik Everett 2013-11-15 13:35:57 UTC
(In reply to comment #4)
> Where does this patch put the notice? I'm not too concerned about the details
> of the actual location, as long as it's clearly visible.

The picture is a screenshot of the patch (in Firefox).  So, right above the search term input box.
Comment 6 Dan Garry 2013-11-15 13:41:44 UTC
(In reply to comment #5)
> The picture is a screenshot of the patch (in Firefox).  So, right above the
> search term input box.

Oops, sorry, I missed the screenshot! It all looks good from my end.
Comment 7 Nik Everett 2013-11-15 13:44:34 UTC
Cool!  Are you Deskana on gerrit?  If so I'll make sure you are on appropriate patches.
Comment 8 MZMcBride 2013-11-15 13:47:01 UTC
(In reply to comment #0)
> So that people are adequately informed, any wiki that has CirrusSearch as
> primary should have the following wikitext added below the search box on
> [[Special:Search]]:
> 
> This wiki is now using CirrusSearch, MediaWiki's new search engine.
> ([[mw:Special:MyLanguage/Help:CirrusSearch|Learn more about CirrusSearch]])

I don't like this phrasing. It's not MediaWiki's new search engine as much as it's Wikimedia's new search backend. However, the language itself is overly complex for the audience, I think.

What about something like "We've recently made search improvements (learn more)." The people who click "learn more" can then have terms like "CirrusSearch" injected into their lives. Everyone else can remain happily ignorant.

I'd also recommend a page on Meta-Wiki rather than a page on mediawiki.org, again given the context and audience.
Comment 9 Dan Garry 2013-11-15 14:16:42 UTC
Good point. Including the jargon "CirrusSearch" isn't necessary. We'll use Nemo's second suggestion.
Comment 10 Gerrit Notification Bot 2013-11-15 23:54:54 UTC
Change 95541 merged by jenkins-bot:
Show notification on search page if configured

https://gerrit.wikimedia.org/r/95541
Comment 11 Nik Everett 2013-11-16 00:56:39 UTC
Switching back to assigned - we need to decide on which wikis we should show this.  All the wikis with Cirrus as primary?

We should wait to deploy the config change that turns this on until a bunch of translations come it.  Catalan and Italian are probably the most important here.
Comment 12 Gerrit Notification Bot 2013-11-18 14:06:38 UTC
Change 96022 had a related patch set uploaded by Manybubbles:
Description for new message key.

https://gerrit.wikimedia.org/r/96022
Comment 13 Nik Everett 2013-11-18 14:21:45 UTC
Assigning to Dan Garry to answer Comment 11.
Comment 14 Dan Garry 2013-11-18 14:26:29 UTC
(In reply to comment #11)
> Switching back to assigned - we need to decide on which wikis we should show
> this.  All the wikis with Cirrus as primary?

Yes, all wikis where it's primary.
 
> We should wait to deploy the config change that turns this on until a bunch
> of translations come it.  Catalan and Italian are probably the most important
> here.

Sure, we can wait for localisations. I can inform the relevant wikis that we've turned it on via messages to the Village Pump.
Comment 15 Nik Everett 2013-11-18 14:29:21 UTC
Assigning back to myself now that I have an answer.  That was quick!  I'll sit on it for a while until we get translations.
Comment 16 Gerrit Notification Bot 2013-11-18 18:08:35 UTC
Change 96022 merged by jenkins-bot:
Description for new message key.

https://gerrit.wikimedia.org/r/96022
Comment 17 Nik Everett 2013-11-27 18:56:51 UTC
Patch to turn on the message: https://gerrit.wikimedia.org/r/#/c/97939/

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


Navigation
Links