MoinMoin: MoinMoinChat/Logs/moin-dev/2012-11-18

   1 2012-11-18T04:11:53  *** MattMaker has quit IRC
   2 2012-11-18T04:54:06  <sunu> dreimark: drawing save doesn't work for me on non-gae deployment either.
   3 2012-11-18T06:00:40  <sunu> dreimark: The way we save in ext-server_moinsave.js is somehow creating this problem.
   4 2012-11-18T06:04:12  *** MattMaker has joined #moin-dev
   5 2012-11-18T06:24:50  *** dwcramer has quit IRC
   6 2012-11-18T07:51:28  <sunu> dreimark:  In ext-server_moinsave.js name is undefined. We should split the string by '/%2Bget/' not '/+get/'
   7 2012-11-18T07:52:53  <sunu> Otherthan that the weird thing is the svg somehow saves in firefox but not in chrome.
   8 2012-11-18T09:25:21  *** spy has joined #moin-dev
   9 2012-11-18T10:05:33  <sunu> dreimark: And also we are trying to write to temporary file while creating the time which apparently is not allowed in GAE's restricted python environment.
  10 2012-11-18T10:05:58  <sunu> i meant creating the item*
  11 2012-11-18T10:29:24  *** Suprano has joined #moin-dev
  12 2012-11-18T10:31:19  *** greg_f has joined #moin-dev
  13 2012-11-18T10:33:40  *** marenz has quit IRC
  14 2012-11-18T11:30:08  <dreimark> sunu: i update svg-edit on the default branch
  15 2012-11-18T11:30:57  <dreimark> the tempfile is the issue on gae which breaks it
  16 2012-11-18T11:37:00  <sunu> dreimark: is test.moinmo.in updated ?
  17 2012-11-18T11:39:01  <dreimark> sunu: no
  18 2012-11-18T11:39:41  <dreimark> sunu: but you can create a twdraw or anywikidraw item there (if you have java)
  19 2012-11-18T11:41:07  <sunu> No, I just wanted to test in chrome. Because I the saving failed even after doing the %2B spliting with chrome on my local moin.
  20 2012-11-18T11:42:14  <sunu> I had to move the redirection code into the callback to fix it.
  21 2012-11-18T11:42:56  <sunu> dreimark: ^
  22 2012-11-18T11:51:41  <dreimark> sunu: you have installed the new xstatic package?
  23 2012-11-18T11:54:00  <dreimark> please add an issue on  https://bitbucket.org/ReimarBauer/xstatic-svg-edit-moin
  24 2012-11-18T11:54:05  <dreimark> and the patch
  25 2012-11-18T11:54:42  <dreimark> I am currently also not happy that I need an alert for saving in ff
  26 2012-11-18T11:58:12  <sunu> dreimark: No, I did the splitting locally and tried before. I'll try with the new package and let you know.
  27 2012-11-18T11:59:52  <dreimark> at least in firefox it works that way
  28 2012-11-18T12:01:35  <dreimark> may be moving the redirection to call back is a good solution for both
  29 2012-11-18T12:02:49  <sunu> yeah that way it works in both :)
  30 2012-11-18T12:08:05  <dreimark> have you a patch for me?
  31 2012-11-18T12:09:11  <sunu> yeah I do :)
  32 2012-11-18T12:11:02  <sunu> dreimark: will upload soon. btw do you have any idea about the tempfile issuse . I am trying with StringIO and tempfile.TemporaryFile(). But it's not really working .
  33 2012-11-18T12:11:10  <sunu> dreimark: ^
  34 2012-11-18T12:16:20  <dreimark> thinking also on that, and did a similar attempt which also don't work
  35 2012-11-18T12:17:19  <dreimark> currently i am thinking on using something else than tar, e.g. some xml file should also work
  36 2012-11-18T12:17:43  <dreimark> "file" data
  37 2012-11-18T12:21:28  <dreimark> we only need something which can use append for an amount of requests
  38 2012-11-18T12:41:36  <sunu> dreimark: Weirdly moving redirection into callback doesn't work with the new xstatic package.
  39 2012-11-18T12:42:08  <sunu> so my patch is waste.
  40 2012-11-18T13:12:06  <dreimark> arg, but i now can reproduce the issue
  41 2012-11-18T13:25:16  <dreimark> sunu: https://bitbucket.org/ReimarBauer/xstatic-svg-edit-moin/changeset/b3d6245b1e01ee55cee8ec02ad4736de7b9ca838
  42 2012-11-18T13:25:32  <dreimark> check if that change is makeing the differnce with your patch
  43 2012-11-18T13:40:09  <sunu> dreimark: yes. That makes the difference. Weird.
  44 2012-11-18T13:40:49  <sunu> if I set it true saving works in chrome.
  45 2012-11-18T13:49:26  <sunu> and also the alert is needed for ff only chrome saves without that.
  46 2012-11-18T14:12:06  * dreimark hates such differences in browsers
  47 2012-11-18T14:34:52  <dreimark> sunu: http://stackoverflow.com/questions/583791/is-it-possible-to-generate-and-return-a-zip-file-with-app-engine an idea for zipfile
  48 2012-11-18T14:52:38  *** RogerHaase has joined #moin-dev
  49 2012-11-18T16:17:24  * sunu reads the stackoverflow discussion.
  50 2012-11-18T16:21:58  *** dwcramer has joined #moin-dev
  51 2012-11-18T16:30:41  *** dwcramer has quit IRC
  52 2012-11-18T16:35:53  *** bilal has joined #moin-dev
  53 2012-11-18T16:44:22  *** dwcramer has joined #moin-dev
  54 2012-11-18T16:47:00  *** greg_f has quit IRC
  55 2012-11-18T16:54:35  <RogerHaase> dreimark:  dab0253193eb changes modernized common.css but not main.styl
  56 2012-11-18T16:56:41  <dreimark> sunu: i have a solution, likly have to test it on gae first
  57 2012-11-18T16:57:39  <dreimark> RogerHaase: this common.css was a mistake, i revert it
  58 2012-11-18T16:58:26  <RogerHaase> ok
  59 2012-11-18T17:00:13  <moinBot> http://hg.moinmo.in/moin/2.0/rev/afa63a0cc1e1 2012-11-18 Reimar Bauer <rb.proj AT googlemail DOT com>  partial revert of dab0253193eb
  60 2012-11-18T17:11:05  <sunu> dreimark: ok :)
  61 2012-11-18T17:39:41  *** bilal has quit IRC
  62 2012-11-18T17:44:32  <dreimark> sunu: http://codereview.appspot.com/6853065
  63 2012-11-18T17:44:52  <dreimark> i try to simplify awd and twd currently
  64 2012-11-18T18:21:54  <dreimark> sunu: http://rb-moin2-svg-edit-test.appspot.com/Home
  65 2012-11-18T18:22:26  <dreimark> it saves but it needs an additional reload - it look like it did not know that the cache must be renewed
  66 2012-11-18T18:29:58  *** dwcramer has quit IRC
  67 2012-11-18T19:01:40  * ThomasWaldmann stops working on pytest 2.3 for now
  68 2012-11-18T19:01:53  <ThomasWaldmann> https://bitbucket.org/thomaswaldmann/moin-2.0/issue/278/pytest-23-moin-test-issues#comment-2417933 see there
  69 2012-11-18T19:01:58  <sunu> dreimark: ok. I see. I was playing around with the zipfile idea, but this is much better. But we have to adjust the other 2 drawing classes too.
  70 2012-11-18T19:03:09  <sunu> hi ThomasWaldmann :)
  71 2012-11-18T19:04:04  *** dwcramer has joined #moin-dev
  72 2012-11-18T19:04:26  <sunu> dreimark: I'll have a good look tomorrow . Feeling too sleepy now . gn.
  73 2012-11-18T19:05:49  * sunu sleeps
  74 2012-11-18T19:28:08  *** sudo_dirk has joined #moin-dev
  75 2012-11-18T19:28:43  <sudo_dirk> dreimark: thanks for updating the setup routine :-)
  76 2012-11-18T19:32:00  <sudo_dirk> i did remember that the html-editor is far on the right side of the page. is that a browser specific problem? i am using firefox 16.0.2...
  77 2012-11-18T19:48:44  <ThomasWaldmann> sudo_dirk: seems to be a css issue, but doesn't happen on chromium. can you check the issue tracker?
  78 2012-11-18T20:31:31  *** dwcramer has quit IRC
  79 2012-11-18T20:50:00  *** dwcramer has joined #moin-dev
  80 2012-11-18T20:50:45  <sudo_dirk> ThomasWaldmann: yes, i will have a look. If I can't find that issue, you will hear again.
  81 2012-11-18T20:54:04  <dreimark> sudo_dirk: you have to change it in the stylus file
  82 2012-11-18T21:08:23  *** MattMaker has quit IRC
  83 2012-11-18T21:36:04  *** MattMaker has joined #moin-dev
  84 2012-11-18T23:24:06  *** RogerHaase has left #moin-dev
  85 2012-11-18T23:24:23  <sudo_dirk> dreimark: no, i used exactly the moin2 from repo (no changes)
  86 2012-11-18T23:24:29  *** spy has quit IRC
  87 2012-11-18T23:31:57  <sudo_dirk> dreimark: sry, did not read correct ;-)
  88 2012-11-18T23:32:35  <sudo_dirk> where do i find that file?
  89 2012-11-18T23:38:42  <sudo_dirk> is it themes/modernized/static/css/stylus/common.css?
  90 2012-11-18T23:50:10  <ThomasWaldmann> main.styl