1 2012-03-07T01:37:46 *** brunomartin
2 2012-03-07T01:43:34 *** raignarok
3 2012-03-07T02:08:28 *** MattMaker
4 2012-03-07T02:28:05 *** MattMaker
5 2012-03-07T03:52:13 *** sudo_dirk
6 2012-03-07T04:20:55 *** MattMaker
7 2012-03-07T05:15:17 *** MattMaker
8 2012-03-07T05:58:32 *** sudo_dirk
9 2012-03-07T07:44:30 *** kursor
10 2012-03-07T08:08:10 <dreimark> moin
11 2012-03-07T08:14:06 <sudo_dirk> moin dreimark
12 2012-03-07T08:14:53 <sudo_dirk> dreimark: http://codereview.appspot.com/5744043 ... wikiconfig parameter support for arnica
13 2012-03-07T10:21:50 *** greg_f
14 2012-03-07T11:11:36 <dreimark> sudo_dirk: very interesting, which py version do you have < 2.6? because of Changed in version 2.6: Returns a named tuple ArgSpec(args, varargs, keywords, defaults).
15 2012-03-07T11:12:17 <dreimark> do we have to raise the requirements
16 2012-03-07T11:13:23 <dreimark> no big deal with 1.9.4
17 2012-03-07T11:14:02 <dreimark> can you paste a wikiconfig example too
18 2012-03-07T11:57:20 *** kanha
19 2012-03-07T12:08:22 <kanha> moin
20 2012-03-07T12:10:02 *** kanha
21 2012-03-07T12:11:25 *** kanha
22 2012-03-07T12:15:33 <kanha> ThomasWaldmann:I fix that openid saving problem about which i was telling you yesterday.
23 2012-03-07T12:19:43 <kanha> should i report that issue?
24 2012-03-07T12:47:22 <dreimark> kanha: yes, and send a cr
25 2012-03-07T13:57:18 *** dave_largo
26 2012-03-07T14:22:52 <kanha> moin
27 2012-03-07T14:24:47 <ThomasWaldmann> kanha: you still did not merge
28 2012-03-07T14:25:22 <kanha> that i did
29 2012-03-07T14:25:23 <ThomasWaldmann> you can avoid work if you merge frequently, avoiding doing work on some outdated code
30 2012-03-07T14:25:46 * ThomasWaldmann is looking at your public repo
31 2012-03-07T14:26:00 <kanha> I am talking about recent code
32 2012-03-07T14:26:24 <ThomasWaldmann> https://bitbucket.org/abhisheksingh/moin-2.0/changesets < you see those coloured lines at the left?
33 2012-03-07T14:27:08 <kanha> yes I can see
34 2012-03-07T14:27:47 <ThomasWaldmann> you see that your last public changeset is based on a changeset from me that is not the newest (see dark blue line)
35 2012-03-07T14:29:10 <kanha> yes it is there
36 2012-03-07T14:30:08 <ThomasWaldmann> "it is there"?
37 2012-03-07T14:30:47 <kanha> those dark blue lines.so I need to merge
38 2012-03-07T14:32:54 <kanha> Is that problem is fixed about whatt am talking?
39 2012-03-07T14:36:57 <ThomasWaldmann> i don't know
40 2012-03-07T14:37:22 <ThomasWaldmann> i just wanted to point out that you might make yourself more work if you don't merge often
41 2012-03-07T14:37:37 *** sravi
42 2012-03-07T14:37:42 <ThomasWaldmann> because it will get harder to merge the longer you wait
43 2012-03-07T14:38:28 <kanha> I am having some confusion with merging can u explain a little to me please?
44 2012-03-07T14:38:58 <ThomasWaldmann> http://moinmo.in/MoinDev/MercurialGuide see below "Merging"
45 2012-03-07T14:39:40 *** sravi
46 2012-03-07T14:40:19 <ThomasWaldmann> there are also other tutorials and even books about mercurial, see top of that page
47 2012-03-07T14:40:28 <ThomasWaldmann> brb
48 2012-03-07T14:44:46 *** xjjk
49 2012-03-07T14:44:52 *** xjjk
50 2012-03-07T14:50:54 <dreimark> TheSheep: do you know a workaround for onclick on chromium?
51 2012-03-07T14:56:49 *** kursor
52 2012-03-07T14:58:12 <TheSheep> dreimark: I don't understand
53 2012-03-07T14:59:06 <dreimark> TheSheep: http://johannes.sipsolutions.net/Photos/Canada
54 2012-03-07T14:59:23 <dreimark> all actions based on onclick for arnica slides don't work
55 2012-03-07T15:00:38 <dreimark> if you look with firefox browser and epiphany work but not chromium-browser
56 2012-03-07T15:03:23 <kanha> I fix bug #160 here is patch for review http://codereview.appspot.com/5756077
57 2012-03-07T15:03:51 <TheSheep> dreimark: http://www.sheep.art.pl/misc/test.html
58 2012-03-07T15:03:54 <TheSheep> dreimark: works for me
59 2012-03-07T15:04:16 <TheSheep> dreimark: you must be doing something wrong
60 2012-03-07T15:08:35 <TheSheep> dreimark: onclick attribute is a bad idea nayways
61 2012-03-07T15:09:08 <TheSheep> dreimark: you should instead have a function that runs on document ready and connects the handlers to elements based on their classes
62 2012-03-07T15:13:22 <ThomasWaldmann> kanha: yeah, that's correct :)
63 2012-03-07T15:13:44 <ThomasWaldmann> i guess there was some problem at some time, so it got commented. but of course we need it in the end.
64 2012-03-07T15:14:18 <ThomasWaldmann> so commit it, merge main repo into yours, make a pull request
65 2012-03-07T15:14:20 <ThomasWaldmann> bbl
66 2012-03-07T15:34:42 <dreimark> TheSheep: ok, i'll look into it
67 2012-03-07T15:34:45 <dreimark> thx
68 2012-03-07T15:36:59 <dreimark> <span id="arnica_first_slide"><a class="first_slide" onclick="arnica_first_slide();" name="fs" title="first slide" ></a></span>
69 2012-03-07T15:37:27 <dreimark> that ^^ did not work
70 2012-03-07T15:37:35 <dreimark> its a bit different to your example
71 2012-03-07T15:37:42 <dreimark> it is
72 2012-03-07T15:37:49 <sudo_dirk> dreimark: i use python 2.6 and 2.7 depends on developing and production system. what do you mean with do we have to raise the requirements. config example see http://paste.pocoo.org/show/562400/
73 2012-03-07T15:38:16 <dreimark> sudo_dirk: because the result was changed in 2.6
74 2012-03-07T15:38:29 <sudo_dirk> i read this
75 2012-03-07T15:38:44 <dreimark> and moin runs with 2.5 too
76 2012-03-07T15:39:05 <dreimark> since moin-1.9.4 we require 2.6 see CHANGES
77 2012-03-07T15:39:19 <sudo_dirk> ah right. but shouldnt it work on 2.5. its just a named list now
78 2012-03-07T15:39:20 <dreimark> but a small change fixes it for 2.5
79 2012-03-07T15:39:44 <dreimark> it is to do to verify that
80 2012-03-07T15:40:08 <TheSheep> dreimark: perhaps it doesn't work because it is empty?
81 2012-03-07T15:41:39 * TheSheep checks
82 2012-03-07T15:41:40 <TheSheep> nope
83 2012-03-07T15:42:44 <TheSheep> dreimark: aah, I see the problem
84 2012-03-07T15:42:51 <dreimark> tell
85 2012-03-07T15:42:58 <TheSheep> dreimark: the think you are clicking on is not that element
86 2012-03-07T15:43:08 <TheSheep> dreimark: it's an element added with :before
87 2012-03-07T15:43:29 <TheSheep> the image, I mean
88 2012-03-07T15:43:43 <dreimark> i see
89 2012-03-07T15:44:43 <dreimark> that is a different issue then
90 2012-03-07T15:45:06 <sudo_dirk> dreimark: bbl.
91 2012-03-07T15:45:09 <TheSheep> yup, changing it to a background image and display-block makes it work
92 2012-03-07T15:45:39 <TheSheep> dreimark: http://paste.pocoo.org/show/562404/
93 2012-03-07T15:46:21 <TheSheep> dreimark: I guess you could report that inconsistency as a bug in chrome
94 2012-03-07T15:46:46 <dreimark> yes, doing
95 2012-03-07T15:47:18 <sudo_dirk> dreimark: i little thing before i go. getargspec is also used that way in wikiutil.
96 2012-03-07T15:47:41 <dreimark> sudo_dirk: :) I know johill has introduced that
97 2012-03-07T15:47:49 <dreimark> i just read the docs again
98 2012-03-07T15:48:18 <dreimark> i look later in the evening in cr and try it
99 2012-03-07T15:48:25 <dreimark> it looks so far good
100 2012-03-07T15:48:32 <sudo_dirk> dreimark: ok. thanks.
101 2012-03-07T15:49:37 <sudo_dirk> possibly it could get a general functionality in wikiutil if it is used that way in moin2.
102 2012-03-07T15:50:39 <sudo_dirk> arnica_settings().keys() could also be used
103 2012-03-07T15:51:08 <sudo_dirk> but now... bbl
104 2012-03-07T16:28:05 *** kanha
105 2012-03-07T17:10:55 <CIA-41> Reimar Bauer <rb.proj AT googlemail DOT com> default * 576:a4d66592dea7 extensions/data/plugin/parser/text_x_arnica.py: text_x_arnica.get_image_alias: simplified stripping
106 2012-03-07T17:10:57 <CIA-41> Reimar Bauer <rb.proj AT googlemail DOT com> default * 577:79d3e999dc16 extensions/htdocs/arnica/img/ (3 files): arnica: improved icons for slideshow action (thanks to Johannes Berg)
107 2012-03-07T17:10:58 <CIA-41> Reimar Bauer <rb.proj AT googlemail DOT com> default * 575:0db77eb786fb extensions/data/plugin/parser/text_x_arnica.py: text_x_arnica: Bug Fix: ignore not existing images in get_image_alias
108 2012-03-07T17:13:50 <dreimark> the chromium fix is not that easy
109 2012-03-07T17:13:55 <dreimark> bbl
110 2012-03-07T18:12:20 *** kanha
111 2012-03-07T18:16:50 *** kanha
112 2012-03-07T18:19:21 *** kanha
113 2012-03-07T18:23:30 *** kanha
114 2012-03-07T18:43:48 <ThomasWaldmann> https://pagekite.net/support/free-for-foss/ < maybe SOC students can show stuff right from their dev machine using that?
115 2012-03-07T18:58:36 *** greg_f
116 2012-03-07T19:05:30 <dreimark> ThomasWaldmann: sounds interesting, if student agrees
117 2012-03-07T19:28:25 <dreimark> sudo_dirk: review done
118 2012-03-07T19:36:04 <dreimark> sudo_dirk: please merge
119 2012-03-07T19:55:00 <ThomasWaldmann> they use python :) (with slightly special coding style)
120 2012-03-07T20:59:25 <CIA-41> Reimar Bauer <rb.proj AT googlemail DOT com> default * 578:14ed62b2daab extensions/htdocs/arnica/css/ (arnica_slides.css msie_arnica_slides.css): arnica: beautified css
121 2012-03-07T21:07:21 <sudo_dirk> dreimark: have a look on my comments (cr), please. what do would you like me to merge...
122 2012-03-07T21:08:02 <sudo_dirk> -do :-)
123 2012-03-07T21:13:40 <dreimark> sudo_dirk: the repo
124 2012-03-07T21:14:18 <dreimark> that saves the merged main, if you pull before you commit later
125 2012-03-07T21:15:12 <dreimark> sudo_dirk: make a identifier global var if you want to have it arnica
126 2012-03-07T21:17:38 <dreimark> i want the function in wikiutil, thomas is not available currently
127 2012-03-07T21:17:47 <dreimark> so we can discuss this maybe tom.
128 2012-03-07T21:18:28 <dreimark> and we have to add an easy applicable example what we expect from extension developers
129 2012-03-07T21:18:42 <dreimark> to the commit message and docu
130 2012-03-07T21:18:49 <TheSheep> ThomasWaldmann: alternative to images as icons? http://fortawesome.github.com/Font-Awesome/
131 2012-03-07T21:18:57 *** sudo_dirk2
132 2012-03-07T21:19:19 <sudo_dirk2> dreimark: did you read my suggestion for wikiutil in cr?
133 2012-03-07T21:19:20 <dreimark> TheSheep: he is currently at a lug, not sure if he has internet
134 2012-03-07T21:19:27 <dreimark> sudo_dirk2: yes
135 2012-03-07T21:19:47 <sudo_dirk2> is that ok?
136 2012-03-07T21:23:33 <dreimark> sudo_dirk2: answered. We should avoid API changes in 1.9
137 2012-03-07T21:23:56 <dreimark> or there must be a very good reason.
138 2012-03-07T21:24:36 <sudo_dirk2> is it an API change, if you add a argument like ..., config_prefix=None)
139 2012-03-07T21:24:57 <sudo_dirk2> an None means functionality like now
140 2012-03-07T21:25:07 <dreimark> it is, it may be is not a dangerous one.
141 2012-03-07T21:25:27 <sudo_dirk2> right it is a compatible extension
142 2012-03-07T21:25:46 <dreimark> lets wait what ThomasWaldmann says
143 2012-03-07T21:26:04 <sudo_dirk2> so if you add this by default you change the functionality. with a parameter as descibed not.
144 2012-03-07T21:26:53 <dreimark> if the function is separatly added it is an extension of the module which makes no conflicts
145 2012-03-07T21:27:03 <dreimark> because of the namespace of the module itself
146 2012-03-07T21:27:48 <sudo_dirk2> ok, let's discuss this tom.
147 2012-03-07T21:31:44 *** aa_
148 2012-03-07T21:31:58 *** aa_
149 2012-03-07T21:48:25 <CIA-41> Reimar Bauer <rb.proj AT googlemail DOT com> default * 579:e3bd724d9fb1 extensions/htdocs/arnica/css/arnica_slides.css: arnica: typo in image name for first slide icon fixed
150 2012-03-07T22:08:08 *** dave_largo
151 2012-03-07T22:16:23 <dreimark> chrome is strange. if i call the msie workaround it behaves the same as without it. if i add the default css and the msie one i have all icons twice and they can be clicked
152 2012-03-07T22:27:27 <sudo_dirk2> dreimark: done. look at http://codereview.appspot.com/5744043.
153 2012-03-07T22:31:08 <sudo_dirk2> gn8.
154 2012-03-07T22:32:04 <dreimark> one comment more tom.
155 2012-03-07T22:32:13 <dreimark> good night sudo_dirk2
156 2012-03-07T22:32:41 <sudo_dirk2> no problem. gn8.
157 2012-03-07T22:35:57 *** sudo_dirk2
158 2012-03-07T23:09:15 <ThomasWaldmann> re
159 2012-03-07T23:10:49 <ThomasWaldmann> TheSheep: "even works with IE4 (no kidding)" haha
160 2012-03-07T23:17:38 <ThomasWaldmann> hmm, <i>-tag? why not span?
