Page MenuHomePhabricator

SemanticMaps: Title.
Closed, ResolvedPublic

Description

Author: van.de.bugger

Description:
I use

{{ #ask: ...

format = map

}}

A point on the map has title. Title is shown as a tooltip when mouse cursor hovers the point icon, and also used as title in point's info bubble. There also #ask' parameter title' which

Allows setting text that will be displayed in the pop-ups of all markers that
do not have a specific title.

(taken from http://semantic-mediawiki.org/wiki/Help:Map_format)

However, it is not clear how to set title for a specific point. I did some experiments, then took a look into source code and found that the the first printout statement is taken as point's title, but only it has type `Page'.

In my application title is not of type Page' but of type String'.

Please consider the patch to allow the first printout of `String' type be a title.


Version: unspecified
Severity: enhancement

Details

Reference
bz32632
TitleReferenceAuthorSource BranchDest Branch
ingress-nginx: Use fourohfour for tools with active ingressesrepos/cloud/toolforge/toolforge-deploy!93taavitaavi/fourohfourmain
Improve message when ingress existstoolforge-repos/fourohfour!6taavitaavi/ingress-errorsmaster
Customize query in GitLab

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 21 2014, 11:59 PM
bzimport set Reference to bz32632.

van.de.bugger wrote:

Let the first data of type `String' be a title.

attachment SM_QueryHandler.php.patch1 ignored as obsolete

van.de.bugger wrote:

Atteempt #2.

Previous patch causes error in case of KML export.

Attached:

van.de.bugger wrote:

BTW, did yo notice a typo in `SM_QueryHandler.php' file:

/**

  • Make a separate link to the title or not? *
  • @since 0.7.3 *
  • @var boolean

*/
public $titleLinkSeperate;

Look at name of a variable. Proper spelling is `$titleLinkSeparate'.

Cool. I modified it a bit, handleResultSubject is only meant to handle actual wiki page results.

r104271

Oops, just saw I forgot to mention you in the commit - sorry for that.

van.de.bugger wrote:

Thanks for quick reaction. Verified on r104292.