Camelia, the Perl 6 bug

IRC log for #mojo, 2013-09-28

| Channels | #mojo index | Today | | Search | Google Search | Plain-Text | summary

All times shown according to UTC.

Time Nick Message
01:04 hrupp joined #mojo
01:18 good_news_everyone joined #mojo
01:18 good_news_everyone [mojo] kraih pushed 1 new commit to validation: http://git.io/DXwk8A
01:18 good_news_everyone mojo/validation dfad27f Sebastian Riedel: more documentation for validator
01:18 good_news_everyone left #mojo
01:18 sri ok, thinks it's about merge ready :)
01:20 good_news_everyone joined #mojo
01:20 good_news_everyone [mojo] kraih pushed 1 new commit to validation: http://git.io/qvCbLA
01:20 good_news_everyone mojo/validation c980c66 Sebastian Riedel: fixed typo in example
01:20 good_news_everyone left #mojo
01:24 tianon fwiw, it looks really sweet to me, and I'm looking forward to using it on several projects :)
01:24 tianon will simplify a lot of existing code
01:24 tianon sri++ :)
01:25 good_news_everyone joined #mojo
01:25 good_news_everyone [mojo] kraih pushed 1 new commit to master: http://git.io/IZ9YwA
01:25 good_news_everyone mojo/master 35215b9 Sebastian Riedel: updated Changes
01:25 good_news_everyone left #mojo
01:26 sri and merged \o/
01:27 sri it will stay experimental for a bit, so we can still remove it again if it turns out to be a huge mistake ;p
01:33 marty sri++  \o/
01:35 good_news_everyone joined #mojo
01:35 good_news_everyone [mojo] kraih pushed 1 new commit to master: http://git.io/uiu0nQ
01:35 good_news_everyone mojo/master f4fee84 Sebastian Riedel: form validation is a feature
01:35 good_news_everyone left #mojo
01:38 btyler joined #mojo
02:25 hpekdemir joined #mojo
02:26 hpekdemir do I need a nested route if I have my routes in "package Foo" and I want /foo/bar to controller Foo::Login::Bar action login?
02:27 hpekdemir $r->get('/foo/bar')->to(controller => 'bar', action => 'login'); in routes. and package Foo::Login::Bar has sub login. but somehow it doesn't work.
02:28 hpekdemir so I thought maybe I need to load the namespace Foo::Login::Bar into my routes. but that didn't work neither
02:35 crab /last -msg 5
02:35 crab oops
02:37 akiym joined #mojo
02:58 hpekdemir ok solved.
03:08 preflex_ joined #mojo
04:25 priodev joined #mojo
04:25 jnbek joined #mojo
04:33 sivoais joined #mojo
04:51 yakudza joined #mojo
05:13 basic6_ joined #mojo
05:40 dotandimet joined #mojo
05:58 dotandimet1 joined #mojo
06:03 avkhozov joined #mojo
06:53 basiliscos joined #mojo
07:03 Vandal joined #mojo
07:04 dod joined #mojo
07:25 dod joined #mojo
07:27 mire__ joined #mojo
07:31 dod1 joined #mojo
07:59 d4rkie joined #mojo
08:47 denisboyun joined #mojo
08:52 dqw6 joined #mojo
08:53 denisboyun joined #mojo
08:55 sh4 joined #mojo
09:01 avkhozov joined #mojo
09:41 avkhozov joined #mojo
09:57 mire_ joined #mojo
10:05 fhelmber_ joined #mojo
10:27 punter joined #mojo
10:47 sh4 joined #mojo
10:47 dod joined #mojo
11:02 KindTwo joined #mojo
11:07 KindOne joined #mojo
11:22 KindOne joined #mojo
11:27 KindTwo joined #mojo
11:29 D4RK-PH0ENiX joined #mojo
11:39 KindTwo joined #mojo
11:49 KindTwo joined #mojo
11:51 KindOne joined #mojo
11:53 dotandimet joined #mojo
12:14 asarch joined #mojo
12:29 avkhozov joined #mojo
12:34 batman wonder if this is a good idea: https://github.com/jhthorsen/mandel/commit​/c98ebc0adde5a7cef35709b37d60b4d63ac1b816
12:34 batman $collection->patch(\%doc, sub { ... });
12:47 sh4 joined #mojo
12:50 punter joined #mojo
12:52 yakudza joined #mojo
12:56 batman sri: i'm really not sure if <div class="fields_with_errors"> is the way to go
12:56 batman i would rather have an extra helper which i can have or not
12:58 batman http://p.thorsen.pm/8b62c51c3e10 <-- like this
12:58 batman i think it's a lot more flexible
12:58 batman aslo, suddenly adding <div> block around <input> will break a lot of layouts
12:59 batman and i don't see what you gain from it
13:01 batman if you don't want what i'm suggesting, i would suggest just auto-adding a classname and not a whole block around the field
13:08 sri batman: you've not thought that through
13:09 batman tell me more
13:11 tianon isn't there already a callback to customize that behavior?
13:12 sri a) no layouts will be broken, validation is opt-in, so you know upfront what will happen if you use it b) i can't think of a reason why i wouldn't want to specifically style fields with errors c) explicit blocks are ugly as hell and ruin the whole thing
13:12 batman sri: having markup that sometimes contain a block element, dependent on validation... *shrug*
13:13 batman sri: i don't see why i would want to style -just- my input field, and not my label.
13:13 batman i didn't read _wrap() until now, so i see a) now
13:14 batman i don't see how you can't have b) with just a class
13:14 sri ok, please don't make recommendations before you understand the feature
13:14 batman sri: i still disagree. i understood what happens, i just didn't understand when (until now)
13:15 sri remember what i said two days ago? the code is out there, you're welcome to sen dpatches
13:15 batman so why can't you just have class => 'got-errors'
13:16 batman why would i send a patch if you bluntly going to refuse it? i need to discuss it first.
13:17 sri why would i discuss with you if you're never going to send a patch anyway?
13:17 * batman sends a patch
13:27 sri d) i bet you've not read the examples showing how to not break designs with the divs
13:35 batman are you referring to the css?
13:39 batman https://github.com/kraih/mojo/pull/539
13:54 sri batman: did you just break all sticky values?
13:55 sri the input_tag, select_field and text_area helpers are not just there for fun
13:55 batman prove -l t/mojolicious/*.t # all good
13:56 batman where do you test sticky values? and what is it?
13:56 batman is it the same as default values?
13:56 sri wait, my bad
13:57 batman ok
13:58 batman i wouldn't dare pushing before running prove -l t/mojolicious/*.t ;)
13:58 dqw6 joined #mojo
14:07 sh3 joined #mojo
14:10 Zx3 joined #mojo
14:16 sri it doesn't have to be more code than before, most of the attr handling is unnecessary, documentation needs to be changed too
14:16 d4rkie joined #mojo
14:18 batman sri: i'm not going to do that if you still think it's a bad idea
14:20 batman i'm pretty sure i won't make that before tomorrow afternoon. going to a birthday party soon...
14:20 batman but i will, if you like what i've done so far
14:26 basiliscos joined #mojo
14:33 denisboyun joined #mojo
14:38 bjoernfan joined #mojo
14:41 sri batman: i have to think about this for a bit
14:41 batman ok. then i'll hold
14:42 sri i believe the code should be shorter than before though
14:42 batman uhm.. i'll hold anyway until tomorrow afternoon, since i'll be partying :)
14:42 sri why field-with-error instead of fields_with_errors?
14:42 batman i have less function calls if that matters :)
14:43 batman because i like to have my classes with dash and my ids with underscore. not-a-big-deal
14:43 batman i read about that way of naming stuff many years ago, so it's a bit difficult to let go
14:43 batman stuff = id and class
14:44 sri i'm not arguing against it, just curious
14:44 batman ok
14:45 batman my parents just came home. have to go...
14:45 batman see you tomorrow :)
14:46 sri odds are i will fix it up if i decide to make the change ;p
14:48 batman hehe
14:48 batman even better :)
14:52 batman want me to fix the crappy commit message?
14:52 sri no need, can't merge that commit anyway
14:53 batman excellent!
14:55 tianon lol
14:57 denis_boyun_ joined #mojo
15:02 d4rkie joined #mojo
15:03 good_news_everyone joined #mojo
15:03 good_news_everyone [mojo] kraih pushed 1 new commit to master: http://git.io/C9Plug
15:03 good_news_everyone mojo/master a253f44 Sebastian Riedel: changed how fields that failed validation are highlighted
15:03 good_news_everyone left #mojo
15:03 sri batman++ # lets try it
15:03 sri golfed it down to 2 lines less code than the old code
15:04 tianon heh, didn't even notice that regex typo before :)
15:05 tianon "usernames are limited to a single letter between e and t - sorry, we don't have more than 28kb for our whole database and want to keep signups low"
15:05 sri me neither before i tried the examples again ;p
15:06 sri i wonder how much of the taghelpers code can still be golfed down
15:06 tianon fwiw, I really like the change to class
15:06 sri lots of ref 'CODE' and @_ % 2 checks
15:09 sri i suppose a label_for helper with field-with-error would make sense too
15:32 dqw6 joined #mojo
15:35 tempire sri: Do you recall what this font was - http://mojocasts.com/e2#Generic%20Placeholders
15:37 sri tempire: looks like helvetica
15:37 tempire I'm pretty sure it was something special. It was your favorite for a while.
15:38 tempire from the initial vim config you created
15:38 sri oh, you mean the terminal font :)
15:38 sri Consolas then
15:38 sri still is my favorite
15:43 dod joined #mojo
15:48 dod1 joined #mojo
15:52 good_news_everyone joined #mojo
15:52 good_news_everyone [mojo] kraih pushed 1 new commit to master: http://git.io/siHr1A
15:52 good_news_everyone mojo/master b54b82c Sebastian Riedel: added label_for helper
15:52 good_news_everyone left #mojo
15:53 sri now you can style the label too
15:58 sri validation is about 120 lines of new code, wonder if it's worth it
16:00 tianon love label_for - that's one we've looked for quite a few times and ended up with using the tag helper (which isn't really much more or less typing, but it feels right)
16:00 tianon and having it plug automatically into validation is super cool
16:06 bjoernfan joined #mojo
16:08 sri \o/
16:09 cfedde \./
16:09 tempire dernit
16:09 tempire having trouble getting an exact match for m2
16:10 tempire this is why I've procrastinated so long in updating them.
16:21 moritz 66
16:21 moritz sorry, lag
16:21 tempire got it!
16:24 tempire hmm.
16:24 tempire now how did force osx screen resolution to 640x480?
16:26 tempire or
16:26 tempire who knows a screen recording app that limits itself to one window?
16:41 batman sri: that is awesome!!!!
16:41 batman thanks a lot
16:41 batman owe you beer :)
16:41 * batman goes partying
16:41 * moritz does a 'cpanm Galileo' and sees it install half of CPAN
16:41 batman label_for is also awesome
16:43 moritz Galileo uses at least 4 different object systems (Mojo::Base, plain perl, Moo, Moose)
16:46 tianon it just needs to install three different frameworks and we'll be golden - Dancer and Catalyst come to mind :)
16:47 cfedde this is one of the down sides to Tim Towtdi
18:10 gryphon joined #mojo
18:29 marcus Cfedde: no, this is Sparta!
18:29 * marcus kicks cfedde into the well.
18:41 good_news_everyone joined #mojo
18:41 good_news_everyone [mojo] kraih pushed 1 new commit to master: http://git.io/biaViA
18:41 good_news_everyone mojo/master 36a51a4 Sebastian Riedel: small optimizations
18:41 good_news_everyone left #mojo
18:51 bowtie_ joined #mojo
18:55 good_news_everyone joined #mojo
18:55 good_news_everyone [mojo] kraih pushed 1 new commit to master: http://git.io/1e7_cA
18:55 good_news_everyone mojo/master b87365a Sebastian Riedel: documentation tweaks
18:55 good_news_everyone left #mojo
18:59 Daga1 joined #mojo
19:11 * sri still wonders which checks to include in core
20:00 SmokeMachine joined #mojo
20:09 dotandimet joined #mojo
20:21 dotandimet joined #mojo
20:24 perlite_ joined #mojo
20:27 good_news_everyone joined #mojo
20:27 good_news_everyone [mojo] kraih pushed 1 new commit to master: http://git.io/ZoqLmg
20:27 good_news_everyone mojo/master d49c3da Sebastian Riedel: simplify optgroup generation
20:27 good_news_everyone left #mojo
20:34 rem_lex|pivo joined #mojo
20:44 good_news_everyone joined #mojo
20:44 good_news_everyone [mojo] kraih pushed 1 new commit to master: http://git.io/fNByMA
20:44 good_news_everyone mojo/master 9a6e554 Sebastian Riedel: fixed typo in documentation
20:44 good_news_everyone left #mojo
20:47 good_news_everyone joined #mojo
20:47 good_news_everyone [mojo] kraih pushed 1 new commit to master: http://git.io/tHZW7A
20:47 good_news_everyone mojo/master 001c893 Sebastian Riedel: documentation tweak
20:47 good_news_everyone left #mojo
21:24 btyler joined #mojo
21:26 d4rkie joined #mojo
21:59 dvinciguerra joined #mojo
22:00 sri oh, i'm a moron
22:00 sri and everybody who reviewed the validation recipe too ;p
22:01 tianon what other obviousness did we miss? D:
22:02 sri the password confirmation doesn't actually work
22:03 sri it needs to be required if the password is present!
22:03 sri making the whole second example redundant :S
22:04 tianon heh
22:04 tianon so optional, but if the other field is present, required?
22:09 good_news_everyone joined #mojo
22:09 good_news_everyone [mojo] kraih pushed 1 new commit to master: http://git.io/El7oPw
22:09 good_news_everyone mojo/master 338d852 Sebastian Riedel: fixed password confirmation in validation recipe
22:09 good_news_everyone left #mojo
22:09 sri it's kinda nice that one small recipe covers so much ground now though
22:10 tianon that's really awesome - that's a deliciously complex example as far as standard validation goes :)  really loving this feature, thanks for implementing it :)
22:11 sri :)
22:18 marcus batman++
22:23 good_news_everyone joined #mojo
22:23 good_news_everyone [mojo] kraih pushed 1 new commit to master: http://git.io/HQq9yg
22:23 good_news_everyone mojo/master 5a1b821 Sebastian Riedel: fixed links in custom validation check recipe
22:23 good_news_everyone left #mojo
22:29 sri stuff like date and email won't make it into core, but i'm considering numberic, different and not_in
22:29 sri s/b//
22:29 marcus sri: sounds sensible.
22:30 marcus numeric :)
22:33 marcus wow. I'm so happy.
22:33 marcus Think I finally tracked down the idle connection bug in wirc.
22:33 marcus Seems we're not resending messages on error :-/
22:42 tianon date and email are trivial to add in terms of the existing core validators, and there's too many opinions involved in those two, so that sounds perfectly reasonable to me, too :)  sri++
23:34 Daga1 joined #mojo

| Channels | #mojo index | Today | | Search | Google Search | Plain-Text | summary