Ok, perfect.

But why not include [% themelang %]/css/datatables.css into datatables.inc ?

Le 20/09/2013 11:41, Jonathan Druart a écrit :
Fridolyn,

Take a look at bugs 10651 and bug 10868, I QAed 1 hour ago :)

Jonathan

2013/9/20 Fridolyn SOMERS <[email protected]>:
Hie,

Looking at datatables in templates, I notices there are two ways it is
managed.

In some pages of acquisition and in OPAC, the include datatables.inc
(containing JS links and translatable strings) is used + the css :

   <link rel="stylesheet" type="text/css" href="[% themelang
%]/css/datatables.css" />
   [% INCLUDE 'datatables.inc' %]

In more cases, this include is not used and datatables-strings.inc is used
for translatable strings :
   <link rel="stylesheet" type="text/css" href="[% themelang
%]/css/datatables.css" />
   <script type="text/javascript" src="[% themelang
%]/lib/jquery/plugins/jquery.dataTables.min.js"></script>
   [% INCLUDE 'datatables-strings.inc' %]
   <script type="text/javascript" src="[% themelang %]/js/datatables.js">

Wouldn't it be better to always use one include containing everything ?
I think adding the CSS to existing datatables.inc can make the point.

--
Fridolyn SOMERS
Biblibre - Pôle support
[email protected]
_______________________________________________
Koha-devel mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

--
Fridolyn SOMERS
Biblibre - Pôle support
[email protected]
_______________________________________________
Koha-devel mailing list
[email protected]
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to