LordSputnik: I hope you haven't read my recent code. I am making some serious changes including a rebase of bb-134. I will tell you when I am done with that.
LordSputnik: I will be adding some more docstrings in a while.
LordSputnik: I want to reach 4000 lines added :)
LordSputnik: But that won't be some critical changes like those I've just done.
gcilou
Freso, reosarevok needs extra class that's why ;D
Freso
I thought it might be. :)
gcilou
Nah, I gave a better desc. as a line note :)
Bookzombie has quit
Bookzombie joined the channel
Freso
Yeah, so I mostly have whitespace comments.
The Python looks fine AFAICT. I don't know how OCD Gentlecat is about the HTML stuff. If he's as OCD as me, there's a bunch of small things that need fixing. If he's not, probably a bunch less. ;)
Freso guesses opatel99 is not around right now
gcilou
Freso, I've been editing the whitespace :) I'll push it soon
Freso
gcilou: 👍
gcilou
Freso, what's up with this "If you do <br/><br/>, you probably actually want </p><p>"?
Freso
gcilou: Two line breaks usually denote a new paragraph.
In a HTML a new paragraph is made by ending one (</p>) and creating a new one (<p>).
*In HTML
gcilou
Oh so instead of just breaking up the paragraph, just do several <p>s?
Freso
(If we're on HTML 5, the </p> isn't needed, I think.)
Yeah.
gcilou
Ok I gotcha
Freso is old-fashioned and likes the XHTML of closing elements and believes in semantic markup
kepstin
Freso: as far as html5 is concerned, the parser is required to deal with <p> with missing end, but it's still preferred to close them.
Freso
kepstin: Cheers :)
kepstin
(the <p> with no end actually dates back to pre-standardization html, the fact that it's still in the parser is for legacy compatibility)
Freso
Yeah.
gcilou listens to her cat growling in her dream :)
kepstin
the parser also has to deal with things like people using xml-style self-closing elements <foo/> inside an html document; iirc it explicitly ignores any / before a closing bracket.
flamingspinach has quit
gcilou
Freso, better?
flamingspinach joined the channel
Freso
God I really don't like how Facebook slows everything down.
gcilou: I'll check it out in a sec.
gcilou
ok
stanislas
LordSputnik: I've just added you as a collaborator in my CaliBBre repo.
LordSputnik: You've said something like that you want to move it under bookbrainz-ws, so I just thought that this might be a start.
* under bookbrainz
opatel99
Sorry for delay
JesseW joined the channel
JesseW: !
JesseW has quit
Freso
opatel99: I'll live. :)
gcilou: Sending an e-mail, then I'll look.
gcilou
okey!
stanislas
LordSputnik: I've just seen my CaliBBre plugin from 2 months ago, it's so shit I can't believe It has ever been accepted. I've had like 35 commits with no valid commit messages and the code is so ugly. I am now sorting out that mess.
Freso
gcilou: At a glance, the HTML looks much nicer with all that stray whitespace removed. ;)
gcilou
Yup! I think so too :)
opatel99
stanislas: I had no idea what PEP8 was until zas destroyed by pull request with it. If I didn't learn anything else from GCi, I at least learned PEP8.
Like 2nd day of GCi I think ^
Freso
PEP8 is a good thing to have learned if you're ever going to collaborate on anything using Python. :)
gcilou: No new comments, but I have two unaddressed comments. :) (One of them I think I wrote in the last batch, but maybe I didn't?)
gcilou
About the style?
oh I see now
Freso
opatel99: Not very. :)
I frequently re-realise how "chirlu" works.
opatel99
I got that one pretty early. Reversed.
stanislas
Freso: 'Freso' stands for something other than your name ?
opatel99
LordSputnik probably turns into "I am Lord Voldemort" after some manipulation.
Freso
No.
gcilou
I got how Freso worked early but just figured "chirlu" out like yesterday
Freso
:)
opatel99
gcilou: I have no idea what gcilou is. GC makes sense or it could be GCi but what is ilou/lou?
gcilou
lou is a nickname of mine. Caroline->line->lina->linalou->lou/louie lol
bitmap
gcilou: sorry, not sure I'll have a proper bio til next week. what you have works for now :)