[00:33] sol__ joined #tikiwiki [01:18] Tiki-KGB 03lindonb r54502 10(24 files in 15 dirs) * [MRG] Automatic merge, branches/14.x 54490 to 54500 [01:32] Tiki-KGB 03lindonb r54503 10branches/ 10(7 files in 3 dirs) [01:32] Tiki-KGB [FIX] articles: for tiki-read_article page - iconset conversion and add icons to main top buttons; move actions to ellipsis; remove media class and add spacing to article class so that ellipsis dropdown is not cut off. [01:53] Jyhem_laptop joined #tikiwiki [01:58] Tiki-KGB 03chibaguy r54504 10branches/ 10(43 files in 39 dirs) [01:58] Tiki-KGB [FIX] Renamed layout template to reflect its function better. (Less compiling strangely failed for some theme stylesheets so CSS isn't completely updated yet.) [02:13] Gwayne joined #tikiwiki [02:20] Tiki-KGB 03chibaguy r54505 10branches/ 10(5 files in 5 dirs) [02:20] Tiki-KGB [FIX] Theme stylesheets updated manually after less compiling failed for them (layout template renamed to reflect its function better). [03:40] Tiki-KGB 03lindonb r54506 10(57 files in 48 dirs) * [MRG] Automatic merge, branches/14.x 54500 to 54505 [03:40] Tiki-KGB 03lindonb r54507 10branches/14.x/templates/translated-lang.tpl * [FIX] translation menu: add ability for this to be a submenu [03:42] Tiki-KGB 03lindonb r54508 10branches/14.x/templates/tiki-view_articles.tpl [03:42] Tiki-KGB [FIX] view articles: iconset conversion and add icons to main top buttons; move actions to ellipsis, including translation menu as a submenu; remove media class and add spacing to article class so that ellipsis dropdown is not cut off. [03:42] Tiki-KGB 03lindonb r54509 10branches/14.x/templates/tiki-read_article.tpl * make not published notice a warning and not an error [03:44] Tiki-KGB 03lindonb r54510 10branches/14.x/lib/smarty_tiki/block.self_link.php * documentation [03:44] Tiki-KGB 03lindonb r54511 10(6 files in 4 dirs) * [MRG] Automatic merge, branches/14.x 54505 to 54510 [03:44] Tiki|bot joined #tikiwiki [03:54] Tiki-KGB 03lindonb r54512 10branches/ 10(9 files in 3 dirs) [03:54] Tiki-KGB use "more" instead of "ellipsis" for dropdown icon name in case we want to change it later [03:56] Tiki-KGB 03lindonb r54513 10(11 files in 5 dirs) * [MRG] Automatic merge, branches/14.x 54510 to 54512 [04:44] Tiki-KGB 03lindonb r54514 10branches/ 1014.x/templates/modules/mod-quickadmin.tpl 1014.x/templates/tiki-wiki_topline.tpl [04:44] Tiki-KGB [FIX] bootstrapification: Use submenus for actions in quick admin and wiki topline instead of multiple menus. Can't get submenus to fly out to the side like http://getbootstrap.com/2.3.2/components.html#dropdowns, but still works and better than multiple menus. [04:46] Tiki-KGB 03lindonb r54515 10trunk/templates 10trunk/templates/modules/mod-quickadmin.tpl 10trunk 10trunk/templates/tiki-wiki_topline.tpl * [MRG] Automatic merge, branches/14.x 54512 to 54514 [05:48] Tiki-KGB 03chibaguy r54516 10branches/ 1014.x/themes/base_files/css/tiki_base.css 1014.x/themes/base_files/less/tiki-bootstrap_overrides.less 1014.x/templates/tiki-view_articles.tpl [05:48] Tiki-KGB [REF] For semantics, news articles should be class=media (they contain media-left and media-body elements), but to enable the actions dropdown visibility, a media-overflow-visible class is added. [07:17] Tiki|bot New Forum Posts: Composer / could not be resolved to an installable set of packages / jquery - http://suite.tiki.org/tiki-view_forum_thread.php?forumId=6&comments_parentId=55292 [07:17] Tiki|bot New Forum Posts: Fatal error - http://suite.tiki.org/tiki-view_forum_thread.php?forumId=6&comments_parentId=55284 [07:17] Tiki|bot New Forum Posts: i18n question - http://suite.tiki.org/tiki-view_forum_thread.php?forumId=26&comments_parentId=55282 [07:27] Tiki|bot New Forum Posts: Composer / could not be resolved to an installable set of packages / jquery - http://tiki.org/tiki-view_forum_thread.php?forumId=6&comments_parentId=55292 [07:27] Tiki|bot New Forum Posts: Fatal error - http://tiki.org/tiki-view_forum_thread.php?forumId=6&comments_parentId=55284 [07:27] Tiki|bot New Forum Posts: i18n question - http://tiki.org/tiki-view_forum_thread.php?forumId=26&comments_parentId=55282 [08:20] Tiki-KGB 03lindonb r54517 10(5 files in 5 dirs) * [MRG] Automatic merge, branches/14.x 54514 to 54516 [08:24] Tiki-KGB 03lindonb r54518 10branches/ 1014.x/templates/translated-lang.tpl 1014.x/templates/article_trailer.tpl * [FIX] html errors and warnings [08:38] Tiki|bot New Forum Posts: Composer / could not be resolved to an installable set of packages / jquery - http://suite.tiki.org/tiki-view_forum_thread.php?forumId=6&comments_parentId=55292 [08:38] Tiki|bot New Forum Posts: Fatal error - http://suite.tiki.org/tiki-view_forum_thread.php?forumId=6&comments_parentId=55284 [08:38] Tiki|bot New Forum Posts: i18n question - http://suite.tiki.org/tiki-view_forum_thread.php?forumId=26&comments_parentId=55282 [08:48] Tiki|bot New Forum Posts: Composer / could not be resolved to an installable set of packages / jquery - http://tiki.org/tiki-view_forum_thread.php?forumId=6&comments_parentId=55292 [08:48] Tiki|bot New Forum Posts: Fatal error - http://tiki.org/tiki-view_forum_thread.php?forumId=6&comments_parentId=55284 [08:48] Tiki|bot New Forum Posts: i18n question - http://tiki.org/tiki-view_forum_thread.php?forumId=26&comments_parentId=55282 [08:57] chibaguy joined #tikiwiki [08:57] chibaguy polom [09:03] chibaguy I changed the name of a layout template directory and now have been getting a smarty error (missing file) even though I've cleared templates_c and temp/cache, and made sure in the db the layout preference is for an existing tempplate. What else can be done to recover the site? [09:04] redflo joined #tikiwiki [09:04] chibaguy ...and cleared browser cache. [09:09] chibaguy files in templates_c show the old name of the layout template directory. [09:55] Loll joined #tikiwiki [09:56] chibaguy Hmm, does anyone else see the switch-theme module with three selectors (when the theme has an option)? -- Theme / Option / "None" [09:57] chibaguy Or two selectors - Theme and "None" - when the theme doesn't have an option. [10:02] Tiki-KGB 03lindonb r54519 10branches/ 1014.x/templates/tiki-upload_file_progress.tpl 1014.x/lib/smarty_tiki/modifier.iconify.php 1014.x/templates/browse_file_gallery.tpl [10:02] Tiki-KGB [FIX] file galleries: was showing icon all the time instead of image in the upload window. [10:02] Tiki-KGB 03lindonb r54520 10branches/14.x/templates/fgal_context_menu.tpl * [FIX] file galleries: fix context menu formatting for certain items [10:04] Tiki-KGB 03lindonb r54521 10(8 files in 4 dirs) * [MRG] Automatic merge, branches/14.x 54516 to 54520 [10:07] Loll Hi. I'm using version 12.3. My uploaded images are not shown on a wikipage using {img fileId="13" thumb="y" rel="box[g]"} (only a broken image icon is displayed and by by clicking on it the message "This image failed to load" appears. When I use /tiki-download_file.php?fileId=13 the image is correctly downloaded. Any suggestions? [10:34] Tiki-KGB 03chibaguy r54522 10branches/ 10(43 files in 41 dirs) * [REF] Expose dropdown title to theme variables for good color contrast, etc. [10:46] chibaguy Hi Loll, sorry, I don't have any ideas off-hand. That image syntax works ok in my branch 14 test site (just did a quick check). [10:51] Telesight joined #tikiwiki [10:52] chibaguy luciash, if you're around, there's a problem with the top and topbar module zones background colors - the situation varies according to the theme. Some have good alignment but with others there are 15px gaps. So sometimes the -15px default for class=row is good and sometimes not. I'm trying to find a global solution; hope we don't need to deal with it theme by theme. [10:52] chibaguy So if you have any ideas ... ;-) [11:09] Tiki|bot New Forum Posts: Images failed to load - http://tiki.org/tiki-view_forum_thread.php?forumId=6&comments_parentId=55299 [11:18] chibaguy If anyone has any ideas on how to fix a site that's broken due to a file name change, I'd like to hear. I changed a template layout directory and now I have a smarty error as apparently the old location is requested. [11:19] chibaguy Cleared site and browser caches, no help. [11:20] chibaguy Checked the database to see that an existing layout template is set as the preference. [11:20] chibaguy Why doesn't "smarty" understand the situation? ;-) [11:23] chibaguy I could add the old directory name again so smarty can find the file, but does that just delay solving the problem? [11:34] chibaguy Ok, I gave up and made that unsolution - copied and renamed a directory to make smarty happy. [11:38] chibaguy Just for closure, I found that in L&F admin, the old layout directory was still set as default, so I could change it and then delete the old directory, which I had reinstated just to get access to L&F admin. I don't know if I was checking the pref in a wrong db. [11:48] Tiki-KGB 03chibaguy r54523 10branches/ 1014.x/themes/base_files/css/tiki_base.css 1014.x/themes/base_files/less/tiki-admin.less * [FIX] Admin box legacy icons (images) were positioned too high. [12:26] Tiki-KGB 03jonnybradley r54524 10branches/14.x/lib/wiki-plugins/wikiplugin_div.php * [FIX] notices [12:28] fabricius joined #tikiwiki [12:31] luciash polom [12:32] luciash chibaguy: hi, reading [12:32] chibaguy hi luciash [12:33] chibaguy I'm checking the different themes to see if there's a pattern to the problem; a little complicated because of different themes and layouts. [12:33] chibaguy The topbar background for example could be adjusted in each theme stylesheet, but it'd be better if there was a global fix somehow. [12:34] luciash Loll: make sure your server has gd lib installed which handles thumbnail images generation properly [12:35] luciash chibaguy: any examples to see? [12:35] chibaguy yes, just a minute. [12:36] Tiki|bot Recent Bug: - http://dev.tiki.org/item5596 [12:37] luciash chibaguy: about the smarty issue i think it must be defined somewhere to allow smarty (security) to read templates from a newly introduced dir [12:38] chibaguy ah yeah. Apparently the problem was that I had a layout template set as default and then removed its files. So user error. [12:38] luciash ah i see [12:38] chibaguy (I checked the db to change the pref but it didn't change in L&F. I don't know if I was looking at the wrong db. [12:40] chibaguy Ok. http://zukakakina.com . You can switch the themes there. [12:41] chibaguy So you can see things line up nicely for the Bootswatch themes, I suppose because there's no border or background around the middle columns, etc. [12:42] chibaguy But in themes with backgrounds like Strasa or others, there's a gap at the sides of the topbar. [12:43] chibaguy (Also notice the extra selector in the themes-switch module - I'll ask gezza about that.) [12:46] chibaguy It seems like maybe there's no universal setting - maybe the topbar side margins just have to be set 0px or -15px depending on the backgrounds and borders of the theme. [12:46] chibaguy (Plus I need to check with other layout templates.) [13:01] luciash checking [13:11] luciash chibaguy: I would really set the negative margins (-15px on class .tiki .topbar) only on the themes which actually need it; so far I can see it only on Strasa and TheNews... not that bad, what do you think? [13:12] chibaguy I guess it has to be like that - set the margins for themes that need it. [13:13] luciash +1 [13:13] luciash btw, I did not notice the extra selector [13:13] luciash seems ok to me [13:14] chibaguy Probably I should make it a variable to put in the themes' variable file. [13:14] Loll luciash: Thanks for your suggestion. I will check this . However during tiki installation it was shown that GD is installed. [13:14] Loll I had to find out how to check if this works properly [13:15] luciash Loll: it might be also mime type handling problem but I do not know on non-apache server how to check [13:15] chibaguy Loll, i don't think GD is involved in this problem. [13:15] luciash chibaguy: good idea [13:15] chibaguy luciash, Loll has a problem with images not displaying in wiki pages - tiki 12. [13:16] chibaguy Although they can be downloaded ok. [13:16] luciash chibaguy: i think i got it [13:16] luciash chibaguy: not sure he said thumb=y images only or full size ones too [13:17] luciash Loll: is your server publicly accessible? [13:17] Loll In the filegallery the thumnails are well displayed. [13:17] chibaguy he posted here: {img fileId="13" thumb="y" rel="box[g]"} [13:17] luciash yes, that is thumb generated image [13:17] Loll No, unfortunately not accessible through internet [13:17] luciash ok, check in your firebug console what error it reports, it might help [13:19] Loll OK, I will check this as soon as my php update to 5.5 is finished [13:32] lindonb joined #tikiwiki [13:37] jonnyb joined #tikiwiki [13:37] jonnyb pompom [13:39] Loll Firebug console does not show any errors [13:46] fabricius joined #tikiwiki [13:53] chibaguy hi jonnyb [13:54] luciash jonnyb: polom [13:54] luciash Loll: but it should, when the images are broken... weird [13:56] jonnyb hi chibaguy, luciash, Loll and all [13:56] Loll I'm not sure that the images are broken. They are perhaps not loaded. [13:56] jonnyb fabricius: too! :) [13:57] luciash Loll: there should be reason why the are not loaded (404, 403, etc. error); do you see this logged in as an admin? [13:58] luciash Loll: or logged out visitor? [13:58] ricks99 joined #tikiwiki [14:00] ricks99 polom y'all [14:00] jonnyb hi ricks99 [14:00] chibaguy hi ricks99 [14:02] luciash hi rick [14:02] fabricius Hi jonnyb, ricks99, chibaguy, luciash, refizul, Loll ... just popping by ... coming back later in the evening [14:03] luciash hi fabricius [14:03] ricks99 hi all :) [14:03] chibaguy hi fabricius. could you bring beers when you return, please? [14:04] fabricius chibaguy: will try, but I fear i will be only able to transfer at one of the next TikiFests [14:04] fabricius luciash, chibaguy: what is the status of the design for the Tiki community websites for the after Tiki14 release period? [14:05] chibaguy :-) yeah, I wish I could make it to the central europe tikifest, but probably there's a funds problem. [14:05] chibaguy Oh, the design is getting pretty close, I think. [14:06] Tiki-KGB 03jonnybradley r54525 10(47 files in 44 dirs) * [MRG] Automatic merge, branches/14.x 54520 to 54524 [14:07] fabricius luciash, chibaguy: do you consider the plan of one-column and two-column landing pages and shouldn't we make a webinar together with ricks99 +X to consider the website layout and structure, which ricks99 and me have been discussiong in recent weeks? I think we should get ideas together. [14:08] Loll luciash: I'm log in as admin [14:08] chibaguy Yes, sounds like a good idea. Probably post on the dev and or artwork list and see when people can attend. [14:09] Loll luciash: i'm trying to install Tiki13 in paralel [14:09] ricks99 fabricius & chibaguy +1 [14:12] fabricius chibaguy, luciash: I am pretty sure, I cannot make it to CEST this year, cause we have a conference at the same time. Chances are that we do a small one end of August in Bonn. I would be interested to coordinate a bigger TikiFest in Bonn 2016 end of August around FROSCON together with a TikiFest CEST a week before or after, for that some can go for two weeks or so from one to the other FEST, and get some time [14:12] fabricius to see Europe - maybe a short brake weekend in Berlin and a day in Prague - especially tinking at people from far away to optionally take the partner and see a bit more from Europe that a HackSpace [14:12] fabricius than a HackSpace [14:17] nelsonko joined #tikiwiki [14:22] ricks99 left #tikiwiki [14:38] lindonb joined #tikiwiki [14:48] Tiki-KGB 03chibaguy r54526 10branches/ 10(51 files in 47 dirs) [14:48] Tiki-KGB [FIX] New variables to enable setting top module and topbar module side margins and bottom margin per theme. Thanks to luci for input. [14:55] chibaguy_ joined #tikiwiki [14:56] Loll luciash: It seems to work in version 13 [15:24] Tiki-KGB 03chibaguy r54527 10branches/ 1014.x/themes/darkroom/less/tiki-selectors.less 1014.x/templates/layouts/fixed_top_modules/layout_view.tpl 1014.x/themes/darkroom/css/darkroom.css [15:24] Tiki-KGB [FIX] Unneeded container div in layout_view.tpl caused horizontal offset. Appearance of admin navbar improved in Darkroom theme. [15:25] luciash re [15:27] luciash fabricius: a pity you cannot come, maybe we could change dates still but depends on other people and stuff... maybe Bonn then is a good idea [15:28] luciash Loll: interesting (I wonder why it wouldn't work in 12) [15:30] luciash fabricius: Nelson is already interested to take partner to TikiFest CEST ;) [15:30] Loll luciash: Is there a way to export the settings of a tiki installation? in this way I could compare settings of both installations [15:30] luciash Loll: yes, on Profiles Admin panel the Advanced tab iirc [15:31] luciash Loll: or export 12.x db, then use it for the 13.x as upgrade [15:37] fabricius joined #tikiwiki [15:38] arildb joined #tikiwiki [15:47] Tiki|bot Recent Bug: - http://dev.tiki.org/item5597 [16:04] Tiki-KGB 03chibaguy r54528 10branches/ 1014.x/themes/business/less/tiki-selectors.less 1014.x/themes/business/css/business.css * [FIX] Missing borders on horizontal menu links re-added (Business theme). [16:22] jonnyb joined #tikiwiki [16:42] Tiki|bot New Forum Posts: Images failed to load - http://suite.tiki.org/tiki-view_forum_thread.php?forumId=6&comments_parentId=55299 [16:42] Tiki|bot New Forum Posts: Composer / could not be resolved to an installable set of packages / jquery - http://suite.tiki.org/tiki-view_forum_thread.php?forumId=6&comments_parentId=55292 [16:42] Tiki|bot New Forum Posts: Fatal error - http://suite.tiki.org/tiki-view_forum_thread.php?forumId=6&comments_parentId=55284 [16:50] jonnyb hi nelsonko - are you there (and have a moment?) [16:51] nelsonko hi jonnyb [16:51] jonnyb i've managed to get the sorting and pagination working with the list plugin (most of it was already in there of course), so that's good... [16:52] Tiki|bot New Forum Posts: Images failed to load - http://tiki.org/tiki-view_forum_thread.php?forumId=6&comments_parentId=55299 [16:52] Tiki|bot New Forum Posts: Composer / could not be resolved to an installable set of packages / jquery - http://tiki.org/tiki-view_forum_thread.php?forumId=6&comments_parentId=55292 [16:52] Tiki|bot New Forum Posts: Fatal error - http://tiki.org/tiki-view_forum_thread.php?forumId=6&comments_parentId=55284 [16:53] nelsonko jonnyb: sounds like good news. any specific commits I should look at? [16:53] jonnyb sry, phone.. [16:55] jonnyb but i'm trying to test with two custom_search plugins on a page and there seem to be other issues - just checking that this was the use case you added the forcesortmode param for? [17:00] jonnyb and the table.tpl template doesn't really work for custom_search - doesn't know about ajax at all... [17:04] jonnyb ew, looks like full screen edit is quite broken, anotehr thing for the list :( [17:17] nelsonko jonnyb: I think the use case is one custom search + 2 list plugins on a page, but I think the problem will occur if you have one custom search + 1 list as well [17:18] nelsonko (I've not used 2 custom search on the same page) [17:18] jonnyb aha, thanks nelsonko - and is that with both using the standard table.tpl? or custom ones? [17:18] nelsonko custom tpls [17:18] nelsonko both using custom tpls [17:18] jonnyb i'm finding it tricky getting the standard one to work with ajax [17:19] jonnyb i see some code that looks like it's trying to set something up at lib/core/Services/Search/CustomSearchController.php:54 but it doesn't seem to be used anywhere... [17:19] jonnyb don't worry, it's all very deep this, i'll keep nagging away at it :) [17:27] Loll Are the template in 13.1 different than in 12.3 ?? [17:31] jonnyb hi Loll - yes, big changes between 12 and 13 and again in 14 (bootstrap process) [18:12] Tiki-KGB 03jonnybradley r54529 10branches/ 1014.x/templates/search/list/table.tpl 1014.x/lib/core/Search/Query/WikiBuilder.php 1014.x/lib/core/Search/Formatter/Builder.php 1014.x/lib/wiki-plugins/wikiplugin_list.php [18:12] Tiki-KGB [FIX] list: Make the existing pagination argument options (offset_arg and max) work with the standard table.tpl template. [18:12] Tiki-KGB Add a new sort_arg option to pagination (seemed like the best place for it, even though it's not really about pagination) so you can define a separate arg for each list plugin instance, allowing multiple sortable and "pageable" lists per page. [18:12] Tiki-KGB E.g. use {pagination offset_arg="offset42"} and {pagination sort_arg="sort_mode42"} to the list plugin to enable. [18:12] Tiki-KGB Custom templates may need some updating to use this (thanks nelsonko) [18:12] Loll I must say that the Fivealine theme was better in version 12 than 13.... Also the modules aligned better [18:14] jonnyb Loll: yes, we know, should be better in 14.x - in the mean time i think fivealive-lite got more attention than the other one in 13.x [18:18] Tiki-KGB 03jonnybradley r54530 10(57 files in 53 dirs) * [MRG] Automatic merge, branches/14.x 54524 to 54529 [18:23] nelsonko jonnyb: so should allow_sort_replace param be removed from list plugin? [18:24] jonnyb i think so, will do it once this has been tested a bit [18:24] jonnyb also think i'm on the way to having multiple custom searches on a page, also not conflicting [18:25] nelsonko jonnyb: ok, thanks [18:25] jonnyb and now you can set the sort_arg for the lists you don't need that forcesortmode param on custom_search (i think & hope) [18:25] jonnyb have merged it back to trunk in case you can test it there more easily [18:29] nelsonko jonnyb: as for the forcesortmode in custom search, I added that in r51341 just in case people wanted the old behaviour back. [18:30] jonnyb oh - for me it broke the old behaviour [18:30] jonnyb without that param set my javascript is prevented from changing the sort order [18:31] nelsonko Basically, the issue I was facing was that if you set a sort mode within the custom search, and then enter some text search, you get results that are not sorted by relevance, but instead by whatever the sort mode was - so it looks to the user that the searching wasn't working [18:32] nelsonko it seemed unnatural and incorrect. So I changed the behaviour and added the "forcesortmode" parameter such that if you set that on, you will get back the old behavior [18:32] jonnyb but now doing customsearch_0.sort_mode has no effect any more, it used to let you change the sort [18:32] jonnyb oh right, so the default is to change the current behaviour? [18:33] jonnyb i wasn't expecting that... [18:33] nelsonko I had not realized it would prevent changing the sort [18:33] jonnyb i'm not sure what you mean by "force" in this context [18:34] jonnyb lib/wiki-plugins/wikiplugin_customsearch.php:397 [18:35] jonnyb customsearch.options.forcesortmode defaults to false [18:36] gezza joined #tikiwiki [18:36] nelsonko as I mentioned, I had not realized it prevents changing the sort [18:38] jonnyb when you say "set a sort mode within the custom search" you mean by adding a {sort mode="title_asc"} in the cs plugin definition? why would you want that not to work if you've added it - i don't understand, sorry [18:38] nelsonko jonnyb: so adding {sort mode="title_asc"} gives you the default sort, but once people search for something the sort mode should IMO switch to relevance [18:39] jonnyb wow, seems very counter intuitive to me i'm afraid [18:39] jonnyb i would consider that a bug [18:39] nelsonko jonnyb: I think both use cases are needed [18:39] nelsonko I agree that {sort mode="title_asc"} would want to be preserved [18:40] nelsonko but {sort mode="modification_date_desc"} was what I had [18:40] nelsonko It depends on whether you are using it as a "filter" or as a "search" proper [18:41] jonnyb how do you mean? what's the difference? [18:41] nelsonko If you are searching for things, the resulting sort mode should be by relevance, otherwise the user will say hey the search isn't working, [18:41] jonnyb for me if i purpoefully set a sort mode it means i want the results to be sorted in that order, i just don't get how you would not want to do that [18:42] jonnyb not necessarily, my client specifically wants users to be able to sort by price, title, date etc [18:42] nelsonko What I want (in this use case) is for the sort mode to be a certain way before the user enter a text search, but once the user enters the text it should show by relevance [18:43] jonnyb now that's completely ignored and it always sorts by score_desc [18:43] nelsonko yes I agree the commit was a bad one [18:43] jonnyb wouldn't it be better to add a new "autoload_sort_mode" param or something then? [18:44] nelsonko I was worried about it which is why I added the parameter, but the default should have been the old behavior [18:44] nelsonko the forcesortmode parameter i.e. [18:45] nelsonko well, something like autoload_sort_mode, but in this use case, there are other filters as well, but not the text filter [18:45] jonnyb so maybe it should be called ignoresortparamor something? [18:45] nelsonko e.g. checkboxes, categories, etc. [18:45] nelsonko those shouldn't trigger sort by relevance, - only when the user actually search for something , should the sort mode be set to relevance [18:46] jonnyb i have to say that sounds like a very obscure use case, don't think i've ever seen a site that works like that [18:46] nelsonko in your site, the user can choose his sort mode? [18:47] jonnyb maybe more suitable for localised customisation? [18:47] nelsonko is "relevance" one of the sort modes the user can choose from? [18:47] jonnyb yes [18:47] jonnyb the default [18:47] nelsonko when the user goes to the page at first [18:47] nelsonko and there is no search query, what does it sort by? [18:48] jonnyb what the sort mode is set to in the plugin [18:48] nelsonko relevance? [18:49] nelsonko or title? [18:49] jonnyb actually there's not sort set (just checked) [18:49] leagris joined #tikiwiki [18:50] nelsonko ok, so once you set a sort, the "problem" starts to happen. That's how it happened for me [18:50] nelsonko we set a sort, say modification_date_desc [18:51] nelsonko then we received complaints that when they search for things, they don't see relevant results - because it seemed to be all sorted according to date [18:51] nelsonko basically this is the problem that I was trying to fix (pretty clumsily I admit) [18:52] jonnyb Ok, so why not have bit of custom js on the text search change to set the sort to what that site wants? i don't see it needs to be a part of tiki [18:53] nelsonko yeah, now thinking about it that could work. I was trying to fix it for Tiki because it seem like a possible recurring problem for others [18:54] jonnyb i guess i get the idea of having one sort for the autoloaded list and another for users searches, but i think having a "sort by" select or something visible on the form would do it too [18:56] nelsonko I am thinking perhaps making the search UI parser parse an additional param that says, if this field is updated to have something, or something like that, to set a certain sort mode [18:58] nelsonko something like {input type=text _filter="content" setsort="score_desc" [18:58] nelsonko or {input type=text _filter="content" sort="score_desc"} [18:59] nelsonko I mean {input type=text _filter="content" _sort="score_desc"} [18:59] jonnyb maybe, more understandable than the force thing, imho :) [19:00] nelsonko yes, sorry. [19:00] jonnyb no, sorry to have given you such a hard time about it ;) [19:01] jonnyb at least i know i can keep my client happy by setting that param in the mean time [19:04] Tiki-KGB 03gezzzan r54531 10branches/14.x/templates/admin/include_articles.tpl [19:04] Tiki-KGB [FIX] add button for submissions, cleanup of legends, align and remove some more noise from the screen [19:08] gezza hi jonnyb [19:08] jonnyb hi gezza [19:08] jonnyb looks like you're busy with the tpls, thanks! :) [19:08] jonnyb brb [19:09] gezza you were improving structures before, right? i did some improvements and noticed that if a structure is in a menu, and I add a new page to the structure, it wont appear until I clear cache [19:09] gezza is it a known thing? [19:09] gezza maybe it is intentianl [19:09] gezza intentional [19:10] gezza ok, I'll be around for a while [19:10] jonnyb b [19:11] jonnyb not really intentional, but the structure doesn't necessarily "know" it's being used in a menu, so doesn't know to clear that cache [19:11] nelsonko the caching of the menus/modules are intentional for sure, but because ... (was typing then jonny already said it really well - the structure knows not it is in a menu) [19:12] jonnyb and it would be not great to clear all the caches every thime a structure gets modified i think [19:12] gezza hmmm [19:13] gezza i get your point, but still it is confusing [19:13] jonnyb i guess it would be a relatively trivial db query to find any menu modules using it, and if found clear the module cache... but still would take time [19:13] jonnyb maybe adding a note to the structure admin page to say that - after the structure's been saved if poss [19:14] gezza maybe adda remarksbox about it..but well, I am tring to kill tips and other remarksboxes that should be in documentation [19:14] jonnyb i know, i tihnk tiki's geting overloaded with them [19:14] jonnyb the search admin page is terrible now [19:14] jonnyb imho [19:14] gezza yes, documentation should not be in the source [19:15] gezza translation overhead is huuuuuge [19:15] gezza ok, I try to find out something else [19:17] gezza also structure alias is weird at tiki-edit_structure.php, I have to reload the page twice to get the renaming shown..I guess that is not intentional [19:17] gezza :) [19:18] jonnyb yeah, i never really understood the alias thing, it was there when i arrived (think it's just confusing) [19:20] gezza it makes some sense when you reuse pages in different structures, but well, would be interested to see real use cases for it [19:20] gezza i will not touch these, I stick to visual changes :) [19:21] gezza i mean fixes :P [19:24] jonnyb :) [19:28] Tiki-KGB 03gezzzan r54532 10branches/14.x/templates/tiki-edit_structure.tpl * [FIX] use remarksboxes and arrangement of adding and categorizing pages [19:38] Tiki-KGB 03jonnybradley r54533 10branches/14.x/lib/wiki-plugins/wikiplugin_customsearch.php [19:38] Tiki-KGB [FIX] search: Expose pagination args to the search (so now pagination when using table.tpl works for custom search, sorting to come...) [19:38] nelsonko jonnyb: should I change the default for the "forcesortmode" parameter to on for now to preserve environment (to the Tiki 13 behaviour?), you think? [19:39] jonnyb i think so yes, but i'd rather see it replaced by something nicer :) [19:39] jonnyb i set it to on on my dev site and that's now working fine again, so default to 1 should do the trick for now [19:39] nelsonko yes I will try and get to it tomorrow and Friday... but for now [19:41] jonnyb thanks :) [19:42] nelsonko jonnyb: setting the default parameter in the plugin definition does not actually do anything and I actually have to add code to manually defaul t it right? [19:43] jonnyb yes sadly, plugins need to sort out their own defaults etc still [19:48] Tiki-KGB 03nkoth r54534 10branches/14.x/lib/wiki-plugins/wikiplugin_customsearch.php [19:48] Tiki-KGB [FIX] In r51341, the default behavior was changed when trying to fix another issue, breaking the ability to set the sort_mode via jq before new searches. This sets the default back to the old behavior to avoid regressions. A better fix for the other issue is being worked on, and then this forcesortmode parameter would no longer be needed. [19:48] jonnyb trying to work out how to get two custom searches on a page (might give in actually, why would you want to) but i'm looking at the js it produces, nelsonko any idea why clicking on the form sets the offset to 0? lib/wiki-plugins/wikiplugin_customsearch.php:305 [19:48] jonnyb thanks for r54534 [19:50] nelsonko jonnyb: could it be because when you change the search parameters and re-search, you'd want to start from page 1? [19:50] Tiki-KGB 03gezzzan r54535 10branches/14.x/templates/duplicate_file_gallery.tpl * [FIX] bootstrappification [19:50] jonnyb yes, but it's bound to clicking anywhere on the form (including whitespace etc)? [19:51] nelsonko jonnyb: probably an over aggressive binding [19:51] nelsonko I can't see any reason why clicking on white space etc should trigger that [19:51] jonnyb can ignore it, don't think i can get this untangled without significant refactoring [19:52] jonnyb yes, binding to input.change would seem better - anyway, moving on :) [19:52] nelsonko are select items "input"? [19:53] jonnyb no, same for textareas [19:53] nelsonko grrr :) [19:54] jonnyb so something like $("input, select, textarea", "#customsearch_0").change(... [19:54] jonnyb but never mind, food and going out time here (soon) [19:54] nelsonko :) [19:56] Tiki-KGB 03gezzzan r54536 10branches/14.x/templates/tiki-list_file_gallery.tpl * [FIX] remove unnecessary span from comment button [20:04] Tiki-KGB 03gezzzan r54537 10branches/14.x/templates/categorize.tpl * [FIX] remove abbreviation and use tips [20:20] Tiki-KGB 03gezzzan r54538 10branches/14.x/templates/tiki-list_file_gallery.tpl * [FIX] find and search in content form placement [20:21] sol__ joined #tikiwiki [20:32] Tiki-KGB 03gezzzan r54539 10branches/14.x/templates/tiki-list_file_gallery.tpl * [FIX] move the less frequently used Edit action to ellipsis too [20:42] Tiki-KGB 03gezzzan r54540 10branches/14.x/templates/edit_file_gallery.tpl * [FIX] layout for quota [21:01] Bsfez joined #tikiwiki [21:17] jonnyb hi Bsfez - and bye, i'm just off to the pub :) [21:18] jonnyb was just going to thanks gezza (again) for all these lovely bootrappy inconifying commits, looking forward to checking it all out! [21:19] gezza have a good pint or two jonnyb! [21:19] jonnyb thanks, will do - see you tomorrow [21:20] jonnyb cheers [21:22] Tiki-KGB 03jonnybradley r54541 10branches/ 1014.x/templates/search/list/table.tpl 1014.x/lib/core/Search/Query/WikiBuilder.php [21:22] Tiki-KGB [FIX] search: Fix sort in custom_search ajax results when using table.tpl (too much login in the tpl sorry, TODO refactor after 14.0) [21:22] Tiki-KGB 03jonnybradley r54542 10(11 files in 7 dirs) * [MRG] Automatic merge, branches/14.x 54529 to 54541 [22:04] Telesight joined #tikiwiki [22:06] fabricius joined #tikiwiki [22:35] fabricius joined #tikiwiki [22:44] nelsonko left #tikiwiki [22:53] luciash joined #tikiwiki [23:57] nelsonko_ joined #tikiwiki