Hi guys :)
"Creators" are OK with either approach ;)
We're glad that it's not only us who make use of the macro :)
I have put "not compatible" with 4.x, because the macro refused to work on
our 4.2 installation. After investigation it turned out that for some
reason JS extension's "Use this extension" property (of the macro page)
was
set to "On demand" instead of "Always", so it should be updated as
well in
the repository.
Indeed, but first line of macro content is:
$xwiki.jsx.use("Macro.TabsMacro")
So "on demand" should work properly, as the JSX is used whenever the macro
is used ?
-----Original Message-----
From: users-bounces(a)xwiki.org [mailto:users-bounces@xwiki.org] On Behalf
Of Vincent Massol
Sent: Tuesday, February 26, 2013 17:24 PM
To: XWiki Users
Subject: Re: [xwiki-users] Tabs Macro
Hi Jeremie,
On Feb 26, 2013, at 3:38 PM, Jeremie BOUSQUET <jeremie.bousquet(a)gmail.com>
wrote:
Hello,
I've been using the Tabs Macro [1], and have some feedback on it.
First a very positive because I really like that macro ;-)
But ... :
- in extension page [1], it is said that it's not compatible with 4.x,
though I use it without any issue on a 4.4.1 instance.
- only issue I have (already with XE 3.x) is that several empty lines
are added BEFORE the tabs.
The empty lines consist of 5 lines of :
<div class="wikimodel-emptyline"></div>
To fix the issue, I updated the Macro.TabsMacro macro code, there was
a misplaced empty line in it:
///// BEGIN //////
{{velocity}}
$xwiki.jsx.use("Macro.TabsMacro")
#set ($tabsSortedMapString = $context.macro.params.idsToLabels)
## split incoming string into 2 velocity arrays
#set ($tabMapEntriesStringArray = $tabsSortedMapString.split(','))
#set( $tabIdsArray = [])
#set( $tabNamesArray = [])
#foreach($tabEntryString in $tabMapEntriesStringArray )
#if ("$!tabEntryString" == "" )
## ignore null or empty (see
http://wiki.apache.org/velocity/CheckingForNull)
#else
#set ($tabEntryArray = $tabEntryString.trim().split('='))
#if ($tabEntryArray.size()!=2)
{{error}}Parameter $tabEntryString is not a valid
key=value pair in the provided map: $tabsSortedMapString{{/error}}
#else
#set ($addResult =
$tabIdsArray.add($tabEntryArray.get(0).trim()))
#set ($addResult =
$tabNamesArray.add($tabEntryArray.get(1).trim()))
#end
(*)
#end
#end
## generate tabs (**)
{{html}}
<div class="floatcontainer">
///// END /////
I had to remove line marked with (*), and add 2 empty lines right
after (**). This result in having only 1 empty line added before the
tabs, which is not perfect but better.
As I didn't find a JIRA project for this macro, I'm posting it here ...
Cool
What would be great Jeremie is if you could contact the creators of the
Tabs Macro extension and ask them if they're ok to publish their project on
xwiki-contrib (or on their own github which we would fork in
xwiki-contrib's github).
Alternatively what you could do, is simply set up a project in
xwiki-contrib based on the published 1.0.1 sources and just let the
original creators know about it (this is possible since the sources are
under LGPL).
WDYT?
Thanks
-Vincent
_______________________________________________
users mailing list
users(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/users
_______________________________________________
users mailing list
users(a)xwiki.org
http://lists.xwiki.org/mailman/listinfo/users