incubator-bloodhound-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Apache Bloodhound" <bloodhound-...@incubator.apache.org>
Subject Re: [Apache Bloodhound] #45: Refine XML markup to embed dashboard layouts and widgets
Date Mon, 18 Jun 2012 22:25:24 GMT
#45: Refine XML markup to embed dashboard layouts and widgets
-------------------------+-------------------------------------------------
  Reporter:  olemis      |      Owner:  olemis
      Type:              |     Status:  accepted
  enhancement            |  Milestone:  RC1 for initial release
  Priority:  minor       |    Version:
 Component:  dashboard   |   Keywords:  markup widget layout genshi
Resolution:              |  template
-------------------------+-------------------------------------------------

Old description:

> Right now a combination of XML + JSON has to be inserted in Genshi
> templates so as to embed widgets and layouts implemented by dashboard
> plugin .
>
> For some reasons it's better to implement this feature using ''XML''
> instead. Nonetheless my previous attempt trying to get this done was not
> as successful as I expected .
>
> Further details may be found in comments in Genshi templates included in
> dashboard plugin

New description:

 Right now a combination of XML + JSON has to be inserted in Genshi
 templates so as to embed widgets and layouts implemented by dashboard
 plugin .

 For some reasons it's better to implement this feature using ''XML''
 instead. For instance , widget markup looked like shown below in the early
 days (prior to r1351312 ) looked like the snippet shown below

 {{{
 #!xml

   <bh:widget id="simple_widget" urn="WidgetName" altlinks="false">
     <bh:args>
       {"args" : {"arg1" : "value1",
                  "arg2" : "value2"
                 }
       }
     </bh:args>
   </bh:widget>

 }}}

 ... whereas in current implementation ''only XML'' is needed (which is
 more suitable to use together with ''Genshi'' statements e.g. `py:for` ,
 `py:if` ... )

 {{{
 #!xml

   <bh:widget id="simple_widget" urn="WidgetName" altlinks="false">
     <bh:args>
       <bh:arg name="arg1">value1</bh:arg>
       <bh:arg name="arg2">value2</bh:arg>
     </bh:args>
   </bh:widget>

 }}}

 Nonetheless my initial attempt trying to get this done was not as
 successful as I expected .

 Further details may be found in comments in Genshi templates included in
 dashboard plugin

--

Comment (by olemis):

 Replying to [comment:4 gjm]:
 > It might be an idea to update this ticket with the full rationale for
 why it is better to be only working with XML for this.
 >

 Done with the explanation for widgets . Please review ''';)''' . The
 explanation for layouts deserves a whole article ... I'll write a summary
 as soon as possible .

 > Also, is this ticket otherwise complete?

 No . It's missing a patch to upgrade similar markup for for layouts . I
 started working on it but ... I could not make much progress as there's
 part of the code not working as I expected . ventually this will be ready
 .

-- 
Ticket URL: <https://issues.apache.org/bloodhound/ticket/45#comment:5>
Apache Bloodhound <https://issues.apache.org/bloodhound/>
The Apache Bloodhound (incubating) issue tracker

Mime
View raw message