bloodhound-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Apache Bloodhound" <>
Subject Re: [Apache Bloodhound] #87: Generic conversion of menu for preference pages for unknown plugins to use bootstrap
Date Wed, 23 May 2012 08:09:50 GMT
#87: Generic conversion of menu for preference pages for unknown plugins to use
  Reporter:  gjm          |      Owner:  olemis
      Type:  enhancement  |     Status:  accepted
  Priority:  major        |  Milestone:  RC1 for initial release
 Component:  dashboard    |    Version:
Resolution:               |   Keywords:  theme preferences plugin

Comment (by olemis):

 Two patches attached to this ticket .

 The [attachment:t87_r1341394_prefs_legacy.diff first one] implements
 generic conversion of preferences pages by following the guidelines
 explained in [comment:ticket:79:8 my previous comment] posted to #79 so as
 to prevent scattering . In order to illustrate the result , this is what
 [ Growl preferences
 panel] looks like before applying patch ...

 [[Image(bh_theme_x_31_growl_prefs_before.png, width=600)]]

 ... and next image illustrates the result after applying it .

 [[Image(bh_theme_x_32_growl_prefs_after.png, width=600)]]

 [attachment:t87_r1341394_theme_extensions.diff Second patch] suggests
 including an extension point so as to inject custom templates dynamically
 while rendering `bloodhound_theme.html`. This might be useful later in
 order to handle similar scenarios without modifying template code
 directly. Besides there's another open subject related to performance .
 Extension point makes possible to reduce the number of xtension templates
 injected in theme template according to the context as explained in the
 following snippet :


     def get_theme_templates(self, req, template, data, content_type):
         """List templates that will be included dynamically in theme
         So far they consist of :

         - Generic admin template for unsupported admin panels
         - Generic prefs template for unsupported preferences panels
         # TODO: Reduce the number of templates applied by matching
         # e.g. if this is an admin page do not include prefs template ,
         # OTOH if neither an admin nor prefs page why bother
         # including any of them. This might improve performance.
         yield 'bh_admin.html'
         yield 'bh_prefs.html'


 Both patches are built on top of those attached to #79 , and should be
 applied against r1341394 in the following order.


 $ hg qapplied


Ticket URL: <>
Apache Bloodhound <>
The Apache Bloodhound (incubating) issue tracker

View raw message