Gwayne: joined #tikiwiki
joined #tikiwiki
joined #tikiwiki chibaguy: joined #tikiwiki
Is "remember me" at dev.tiki.org, etc. working for other people? It isn't for me, it seems. Tiki|bot: Recent Bug: - http://dev.tiki.org/item5577
New Forum Posts: About the superfish menu dropdowns - http://tiki.org/tiki-view_forum_thread.php?forumId=26&comments_parentId=55114 Telesight: joined #tikiwiki Tiki|bot: Recent Bug: - http://dev.tiki.org/item5578 Tiki-KGB: 03walper r54220 10(5 files in 4 dirs) * [MRG] Automatic merge, branches/14.x 54212 to 54219
03walper r54221 10branches/14.x/_htaccess
[bp/r54147][ENH] Slightly optimized htaccess following googles cache directive advise. Disabled cache-control in favour of expires. Separate modified since definitions for js/css and images.
03walper r54222 10branches/14.x/doc/devtools/svntools.php
[bp/r54148][FIX] Fixed developer svn tools like svnbranchupdate to work on windows.
03walper r54223 10trunk * [MRG] Automatic merge, branches/14.x 54219 to 54222
03walper r54224 10branches/14.x/lib/core/Tracker/Field/DynamicList.php
[bp/r54209][FIX] Dynamic item list select gadget is too narrow. (Thanks anbumania)
03walper r54225 10branches/14.x/templates/trackerinput/url.tpl * [bp/r54210][FIX] URL text gadget is too narrow. (Thanks anbumania)
03walper r54226 10trunk * [MRG] Automatic merge, branches/14.x 54222 to 54225 wolfganga: joined #tikiwiki
Hello redflo: joined #tikiwiki wolfganga: chibaguy - are you there? chibaguy: hi wolfganga wolfganga: chibaguy - do you know where all the js and css files are included? Those that apppear later on as script tags or style tags in the html?
i found things like: TikiLib::lib('header')->add_jsfile('lib/jquery_tiki/tiki-toolbars.js'); chibaguy: hmm, maybe lib/setup/ wolfganga: ok
another one: do you know wether there is a central log function - where you can log things like errors etc.? chibaguy: I don't know really. I mainly deal with CSS and a little with the .tpl files. wolfganga: no problem -. thank you anyway chibaguy: sure wolfganga_: joined #tikiwiki Jyhem_laptop: joined #tikiwiki luciash: hi gezza: joined #tikiwiki luciash: chibaguy: it remembers me from yesterday still on doc.t.o but I did not close the browser since then wolfganga_: hi
Hi has anyone tried the prefs: tiki_minify_css, tiki_minify_css_single_file tiki_minify_javascript, tiki_minify_late_js_files ?
Looks like a really great feature bc it would reduce the 50+ requests per page.
But it seems to not 100% working, css gets screwed up, and need to manually delete the cachec files in temp after, because the ui is no longer usable Jyhem_laptop: polom wolfganga_: hi chibaguy: hi luciash. Ok. I've been closing the browser, and it doesn't remember me.
(I think all the Tiki sites)
wolfganga_, the minify prefs generally have worked but I haven't tried recently. wolfganga_: the js script part seems toi be ok, the css stuff fails
maybe something with @import vs @import url
it would be REALLY great if that would work again, it would reduce the number of requests a lot!
worse: if you enable the css the site is no longer eally usable. need to rest the prefs in the db AND need to deelete the temp folder Jyhem_laptop: Hmm, Tiki14 does not allow comments on pages which have commas in their name :-(
That already is broken in Tiki12, but in a different way. jonnyb: joined #tikiwiki gezza: wolfganga_: minifying css files separately seems ok, the issue comes only when I enable minify css into single file, is it the same to you? fabricius: wolfganga_: Hi Wolfgang wolfganga_: gezza - just testing
and already made some improvements for javascript
hello fabricius
nice - just 2 js calls - could add it to one - but not sure if it would break something in case minify fails.
but 2 are better than 20 fabricius: maybe chatting in the evening? did start to document the stuff yesterday (see email) I do not think, I need long explanations, but maybe some pointer here and there wolfganga_ wolfganga_: gezza - i will commit right away - so you already have those changes
thorsten - about what - the new DIL? jonnyb: polom
wolfganga_: i was looking at the unicode/encoding problem with tracker popups, you know that's nothing to do with ajax? wolfganga_: No i do not. I just noticed it in that area Tiki-KGB: 03walper r54227 10branches/14.x/lib/headerlib.php
[FIX] Javascript compression did not include certain js files tagged as depandancy. See prefs: tiki_minify_javascript, tiki_minify_late_js_files jonnyb: it's done with a title attribute (or something) on the a element wolfganga_: i saw it in trackerpopups and in the xhr response from the ajax service handler
with a title attribut? that sounds - strange ;-) jonnyb: also i would be very wary in changing the js minify stuff in stable
sorry, used to be a title in the bad old days, uses a data attribute now Tiki-KGB: 03walper r54228 10trunk/lib/headerlib.php 10trunk * [MRG] Automatic merge, branches/14.x 54225 to 54227 jonnyb: data-content
also i saw you backported a ENH commit to 14.x - you know these are not allowed before 14.0 is released? wolfganga_: the js patch i just commited does not break anything - i promise. jonnyb: dependancy js files were created specially for files that broke minifying wolfganga_: no i did not know jonnyb: please read https://dev.tiki.org/Where+to+commit wolfganga_: ok will read that
as for the minify - i added failsafe
so if dependancys fail , they will be added explicit on top jonnyb: personally i wouln't have touched that in the stable branch while in a freeze - it's always been a delicate mechanism and not critical for releasing 14.0 imho
so if it's broken something you'll have to fix it, ok? ;) fabricius: wolfganga_: yes, the DIL and vi/ei_tpl wolfganga_: ok i see
so really only fixex?
fixes?
gezza - are you there? jonnyb: yes, the priority is to release 14.0 as soon as possible with as few bugs as possible, there is already a large list of "blocker" bugs that need fixing, so changing stuff that works adequately and adding new enhancements should go in trunk and wait for 15.x or be backported to 14.x after the release of 14.0 wolfganga_: fabricius - yes will be ok gezza: wolfganga_: yes, i'am back now wolfganga_: just commited a small patch - maybe you want to update - so we are in sync. gezza: ok, but that is for js files, not css wolfganga_: gezza i confirm: miniy css is ok, put it in single file breaks
i would consider this a blocker as well, because if one enables this option he/she has to manually mofify the pref in the db as well as to manually clean the temp folder
yse i did the js files first. gezza: you can click back to disable it Jyhem_laptop: joined #tikiwiki gezza: there is a warning next to the pref about that it might not work wolfganga_: yes back works - just two times reload did not for ... ;-)
can you see why it breaks? gezza: it would be better of course if it worked :) wolfganga_: concatation should not break css files
if it would work - we are doen to 12 reauests from 54+
(including teh js stuff) gezza: the warning says: "This setting may not work out of the box for all styles. import needs to use @import url("...") and not @import "..." wolfganga_: i did read that and already search for @import in css files gezza: so first would need to review the less and css fikes
files wolfganga_: yes gezza: chibaguy could give a hint maybe chibaguy: Most of the imports are in .less files before compiling, to produce one .css file. wolfganga_: jonnyb - coming back to the utf8 - data attribute. Have you found seomthing? jonnyb: just looking now... trying to remember how it works ;)
think it comes from Tracker_Field_Abstract::renderPopup wolfganga_: yes there it comes from - but: chibaguy: So there is one tiki_base.css and one theme style sheet. These shouldn't contain any import statements (except for web fonts). wolfganga_: a patch there would not explain the scrambled response in the xhr call - wouldn't it? gezza: hmm, just tried Amelia theme
it seems ok jonnyb: which xhr call? i don't see one gezza: minified into single file wolfganga_: in my email to the dev list
i made a sample
that was the xhr call created by the itemlink box gezza: fivealive also ok, but there is an issue with the default theme
the default theme has only one line: @import url(../../../vendor/twitter/bootstrap/dist/css/bootstrap.min.css); jonnyb: wolfganga_: are you talking about the item titles on tiki-view_tracker.php? wolfganga_: jonnyb: sorry for being confsuing. to be more clear: the "umlaut" issue shows up on the trackerpopup. jonnyb: i see it on tiki-view_tracker.php - are you talking about somewhere else? wolfganga_: and i also noticed that the xhr call initiated by the itemlink field, also return that "umlaut" issue, at least in the response.
But, this might be ok there -
i just noticed this - and that was what the email said. jonnyb: but which page are you talking about? wolfganga_: no page . jonnyb: no url? :O
your mail refers to "the popup window in tracker" but i'm not clear what you mean by that wolfganga_: i did not talk about a page. The url in my email is the one from the xhr call initiated by an itemlink field. no page
the call that asks the ajax service handler jonnyb: i don't understand, in my experience all xhr requests get called from javascript on a certain page - you'vbe lost me i'm afraid wolfganga_: so it setup a itemlink field and looked to teh request in chrome debugger - thats all jonnyb: i'll just try and fix the issue i can see on tiki-view_tracker.php
ok, but i see the same problem on the "popup" on the items list too... wolfganga_: teh xhr call gets initiated from js - that is correct. and that js is triggered from an itemlink field in edit modus.
so if you edit a trackeritem that has an itemlink field, it will trigger that xhr call. ;-)
yes chibaguy: The "default" theme is actually bootstrap.css, so it's just imported. I can make a bootstrap clone with Less to avoid the import statement, if necessary. The imports will happen pre-compiling. wolfganga_: chibaguy - seems that will fix it chibaguy: ok. I'll do that. wolfganga_: gezza - well spotted jonnyb: wolfganga_: for me ItemLink seems ok, but my xhr request goes to tiki-search-lookup?maxRecords=250&format=&filter[type]=trackeritem... etc, not tiki-ajax_services.php
this is from the create item modal on tiki-view_tracker.php wolfganga_: let me check jonnyb: and accents etc seem to work fine
i made a linked item called "føü®" and it appears fine Tiki|bot: New Forum Posts: Star name for Tiki 14 (was: Tiki 14: branches/14.x created) - http://tiki.org/tiki-view_forum_thread.php?forumId=26&comments_parentId=55118 wolfganga_: This is teh xhr request from my itemlink field:
http://172.20.20.150/tiki/code/tiki-ajax_services.php?maxRecords=250&format=&filter[type]=trackeritem&filter[tracker_id]=6&filter[tracker_status]=o%20OR%20p%20OR%20c&controller=search&action=lookup jonnyb: curious, odd that's it's different from mine (this is on create new item on tiki-view_tracker.php yes?) wolfganga_: Its result contains: "title":"Ru00f6ver GmbH"} which displays corrcet as Röver though. Tiki-KGB: 03jonnybradley r54229 10branches/14.x/lib/jquery_tiki/tiki-jquery.js
[FIX] ajax: Fix error display for when jquery.validate feature is disabled. Not ideal but it prevents the js errors and displays an error at the top of the form (jquery.validate enabled is recommended) jonnyb: maybe it does different things depending what the linked item field type is
so if it displays correctly what is the problem? does it corrupt the data or something? wolfganga_: my link shows the xhr call to pouplate the itemlink on edit.
its a bit tricky to catch
got the link from tab console in chrome - also enebaled "preservee Logs" jonnyb: ah, i see it - tiki-search-lookup is the same as tiki-ajax_services.php?controller=search&action=lookup (i have sefurl enabled)
sorry, same same wolfganga_: Yes itemlink displays correct - although the encoding in the xhr response look wired. Because i looked wired - i _thought_ that it is somehow related to the popover windows issue.
;-) on the dev system i have sef disabled - sry for confusion. jonnyb: for me in chrome it looks the same as on the page (i.e. correct utf-8)
anyway, i think it's totally different code so not related (i think...) wolfganga_: strange - i copy pasted the output ....
i guess you are right
if we face search issues with utf8 then i will rembeber this - maybe
jonnyb can you fix the popoupwindow then? jonnyb: back on it, i think i'm near to a fix wolfganga_: thank you Tiki-KGB: 03chibaguy r54230 10branches/ 0314.x/themes/default/less/default.less 1014.x/themes/default/css/default.css
[FIX] Import Bootstrap component .less files in default.less, to avoid import statement in default.css. jonnyb: seems to be the json_encode at lib/smarty_tiki/function.popup.php:106 that's doing it
cool, just adding JSON_UNESCAPED_UNICODE seems to do it :) Tiki-KGB: 03jonnybradley r54231 10branches/14.x/lib/smarty_tiki/function.popup.php
[FIX] popups: Don't escape unicode characters in popups (as seen in tracker popups for instance, thanks Wolfgang) jonnyb: chibaguy: (hi!) are you aware of the @import (reference) thing for less? i was thinking it might be useful in my trouble with those gradients (which i still haven't solved) but might be handy generally
http://lesscss.org/features/#import-options chibaguy: Hi jonnyb. jonnyb: hi :) wolfganga_: thanks jonnyn will try it right away jonnyb: i tihnk it means you can sort of include a file so phpstorm (or whatever) can interpret the syntax properly but when it compiles it ignores the import
thought it might be handy?
thanks wolfganga_ chibaguy: yes, looks interesting. I haven't tried it yet. jonnyb: still stuck not being able to compile the tiki themes from less :(
anyway, have to pop out now, bbl wolfganga_: jonnyb - can i takeover the change to trunk?
takeover = merge jonnyb: i'll do it, 1 mo... wolfganga_: ok jonnyb: on it's way :) wolfganga_: ^^ Tiki-KGB: 03jonnybradley r54232 10(5 files in 5 dirs) * [MRG] Automatic merge, branches/14.x 54227 to 54231 wolfganga_: gezza, chibguy - it still does not work with "minify css in one file"
either there is something missing or it is screwed up during minification gezza: i tried an it worked for me
clear cache, log out and log in again wolfganga_: i use the default theme (wich is default bottstrap)
ok will do (had cleard cache but not looged out) jonnyb: ok, back later... Tiki-KGB: 03anbumania r54233 10branches/14.x/lib/jquery_tiki/tiki-jquery.js * [FIX]Add Field and Edit Advanced Option not working Help Jyhem_laptop: repolom
how can I include a Tiki plugin inside a smarty template ? chibaguy_: joined #tikiwiki Jyhem_laptop: like {bloglist Id=1} jonnyb: joined #tikiwiki wolfganga_: gezza it stil not works for me, do i need to rebuiold css after svn update? gezza: try switching to another theme than switch back to default chibaguy_: wolfganga_, the css files are intact already, as files in subversion. gezza: it works for me fine
chibaguy, is it wokring for you too? chibaguy_: minify css? I'll try. wolfganga_: chibaguy yes i think so too, because without css_single_file, it works chibaguy_: yes, it works as separate files for me - 8 files. wolfganga_: and does it work in a single file? chibaguy_: Yes, I just tested with FiveAlive-lite.css and a theme option. wolfganga_: i changed to tthem fivealite - now nothing is working
blank chibaguy_: hmm
the minified file is 254KB ;-) but anyway is just one file. :-) wolfganga_: the true enhancement is the numer of request.
my tiki is dead - after i changed to fivealite Jyhem_laptop: found it: {wikiplugin _name=bloglist Id="1" Items="5" …}{/wikiplugin} wolfganga_: chibaguy - does it work you with one css file? chibaguy_: Yes.
Oh do you mean to minify as one file?
That works. wolfganga_: yes that is what i menat arildb: joined #tikiwiki wolfganga_: remoevd cache (temp) tiki is back there
but single file css still does not work... chibaguy_: Do you have to change the preference in the database, to get the site back?
Ah, it's back. wolfganga_: yes changed theme back to defgault and removed files from temp chibaguy_: hmm, I wonder what the difference is in the sites, that one-file minify is successful or not wolfganga_: me to - i use trunk here - but the dev stuff is currently windows arildb_: joined #tikiwiki wolfganga_: Hey i found this one@import url(/tiki/code/themes/default/css/../../../vendor/twitter/bootstrap/dist/css/bootstrap.min.css);
that should not be there after the svn update ? Tiki-KGB: 03jonnybradley r54234 10branches/14.x/lib/jquery_tiki/tiki-jquery.js
[FIX] modals: Param options still needs initialising if not supplied, the problem here was a missing | char making it a bitwise or, not a comparison (my fault, sorry, reverts and fixes r54233) wolfganga_: chibaguy - should there be a bootstrap.min.css after your patch? chibaguy_: No. wolfganga_: Can pls check that when you disable single file, that there is no bootstrap.min.css loaded? chibaguy_: ok arildb: joined #tikiwiki chibaguy_: There's no bootstrap.min.css. Just "default.css" (137KB, so the full bootstrap.css file) wolfganga_: ok chibaguy_: that import was replaced by my commit r54230, or should have been. wolfganga_: Seesm that i have a differnt problem, just getting a blankpage again - this time using theme default -. something strange happens here gezza: chibaguy: we chatted before about that margin issue in the login box for "I forgot my password" (the text gets cut off)
can I just remove this rule from tiki_base: .tiki .dropdown-menu li:last-of-type, .menuSection ul li:last-of-type {margin-bottom: -5px;}
or is it needed elsewhere?
maybe you see consequences chibaguy_: gezza, yes, the rule is needed for menus. I'll make a special case for the login box, if you like. gezza: that would be great, thanks Tiki-KGB: 03anbumania r54235 10branches/14.x/lib/smarty_tiki/function.bootstrap_modal.php * [FIX] Edit popups should be modal (specifically Edit Tracker Item) Tiki|bot: Recent Bug: - http://dev.tiki.org/item5579 wolfganga_: Found my whihte screen issue
teh minifyer for css has an issue when runniung against default.css on windows
sepcifically the preg_replace callback
as a workaorund i comennted that part out.
what to do? i guess that is a third party thing
/tiki/code/vendor_extra/pear/Minify/CSS/Compressor.php [around line: 115] - $css = preg_replace_callback('/ jonnyb: wolfganga_: maybe check if there's an update to that PEAR stuff? wolfganga_: besides that issue - loadtime now quite good - down to 11 requests from 50+
should put this also on live - huge difference Tiki-KGB: 03jonnybradley r54236 10branches/ 0312.x/templates/activity/tiki.wiki.update.tpl 1012.x 1012.x/lib/core/Search/Query/WikiBuilder.php 0312.x/templates/activity/tiki.wiki.create.tpl * [bp/r49455][MOD] Add support for notification streams in LIST-like plugins wolfganga_: jonnyb the latest version has no change in the area of the issue jonnyb: shame, i tihnk those pear libs are pretty much abandoned these days :(
maybe that css file casues an out of memory preg error - these can be checked for wolfganga_: yes i think so, too
anyway plan-b starts
i can comment the stuff in the pearlib - works. problem: it the lib gets updated that change is lost
or - that is what i try now: wrap the call in a try catch handler jonnyb: yes, it's hard to upstream stuff to pear too
if it's a http://php.net/manual/en/function.preg-last-error.php a try/catch won't get it i think
probably easier to reformat the offending css file probably wolfganga_: ist the bootstrap stuff....
the new default.css jonnyb: it has a 387 character line, that's not nice wolfganga_: i guess bc it is already minified jonnyb: it's huge as well, surely that's a repeat of all the bootstrap and icons css? wolfganga_: yes - was needed to get the single css thing workin jonnyb: ok, fair enough dhz: joined #tikiwiki wolfganga_: i cannot catch the error in the preg_replace callback of minyfy css - tried exception and set_error_handler.
both do not catch them - sees a really fatal php error
seems jonnyb: ew, not nice
does it get into the callback fn at all? wolfganga_: set_error_handler('myHandler'); $minified = @Minify_CSS::minify($content, array('prependRelativePath' => $currentdir.'/', 'bubbleCssImports' => true)); restore_error_handler();
php simly dies
handler is defined upfront: if (!function_exists('myHandler')) { function myHandler($ec, $et, $ef, $el) {
jonnyb - as i said - i can patch the vendor lib - and it works. Problem will be next update.... jonnyb: looking at vendor_extra/pear/Minify/CSS/Compressor.php now wolfganga_: its failing thewre: line 114 // remove ws in selectors $css = preg_replace_callback('/
aeemsa to be an issue in unsing php in windows - since chiba and gezzo have no issues jonnyb: testing here (on Mac os) wolfganga_: will be away for the next 50min.... jonnyb: me too, see you later
hmm, seems fine on mac os php 5.5.22
bbl gilles: joined #tikiwiki GillesM: joined #tikiwiki Telesight: joined #tikiwiki panamaus: joined #tikiwiki fabricius: joined #tikiwiki Jyhem: joined #tikiwiki
joined #tikiwiki
joined #tikiwiki fabricius: joined #tikiwiki redflo: joined #tikiwiki Jyhem_laptop: joined #tikiwiki Tiki|bot: New Forum Posts: FYI - PHP 5.5.1 fails on minifiyng default.css - PHP 5.5.19 is ok - http://tiki.org/tiki-view_forum_thread.php?forumId=26&comments_parentId=55126