XSF Editor Team - 2020-01-22

stpeter has left 00:53:51
debacle has left 01:15:30
stpeter has joined 01:28:50
stpeter has left 03:29:16
Tobi has joined 04:44:02
stpeter has joined 05:13:12
stpeter has left 05:29:46
lnj has joined 05:50:55
winfried has left 10:15:50
winfried has joined 10:15:50
winfried has left 10:21:06
winfried has joined 10:21:14
winfried has left 10:23:25
winfried has joined 10:23:26
winfried has left 10:25:27
winfried has joined 10:25:28
winfried has left 10:34:19
winfried has joined 10:34:20
winfried has left 10:35:56
winfried has joined 10:35:57
debacle has joined 10:36:24
winfried has left 10:37:19
debacle has left 10:37:19
winfried has joined 10:37:20
debacle has joined 10:37:26
winfried has left 11:02:08
winfried has joined 11:02:09
winfried has left 11:02:50
winfried has joined 11:02:51
winfried has left 12:55:15
winfried has joined 12:55:16
winfried has left 13:07:57
winfried has joined 13:07:57
stpeter has joined 13:47:15
Guus has left 13:50:19
Guus has joined 13:50:33
winfried has left 14:52:32
winfried has joined 14:52:38
winfried has left 14:53:25
winfried has joined 14:53:26
stpeter has left 14:58:49
winfried has left 14:59:22
winfried has joined 14:59:23
winfried has left 15:00:02
winfried has joined 15:00:03
flow 15:00:09
"Authors: Lance Stout , JC Brand" why is there an extra space before the comma?
winfried has left 15:02:53
winfried has joined 15:02:53
jonas’ 15:03:36
URL?
jonas’ 15:03:43
probably XSL bug
flow 15:05:57
jonas’, i think it's a css bug
jonas’ 15:05:59
maybe
jonas’ 15:06:01
URL?
flow 15:06:02
affecting all the xeps
jonas’ 15:06:14
I don’t want to go through all the XEPs to find one with >1 author, please
jonas’ 15:06:20
you seem to know at least one affected
flow 15:06:24
jonas’, I think you can select a random xep with multiple authors
jonas’ 15:06:28
aaaaaaaaaaaaaaaaa
flow 15:06:38
waaaaaaaaaa
jonas’ 15:06:40
I don’t know a single XEP off the top of my head with >1 author
jonas’ 15:06:46
and it’s not visbile in the extension list
flow 15:06:53
424
jonas’ 15:06:56
thanks
jonas’ 15:07:09
XSL bug
jonas’ 15:07:18
thanks, looking into it
flow 15:07:29
doesn't the xsl just build an html list?
jonas’ 15:07:32
yes
jonas’ 15:07:35
it adds spaces between the <li/> tho
jonas’ 15:07:38
cause bug
flow 15:11:09
curious to see the fix, as I only see the space introduced between firstname and surname in the xsl
jonas’ 15:15:44
yeah
jonas’ 15:15:48
not quite sure yet where it comes from
jonas’ 15:16:01
~~(to be clear: the "space" is the newline between the <li/> element?)~~ ✎
jonas’ 15:16:03
(to be clear: the "space" is the newline between the <li/> elements) ✏
jonas’ 15:16:31
eh, also, we’re claiming XHTML, but writing HTML :/
pep. 15:16:54
:(((
jonas’ 15:17:05
but that’s a different hill to work on
pep. 15:17:09
sure
jonas’ 15:17:34
I blame the xsltproc html writer
pep. 15:17:42
We're probably not serving the correct mime type though right? Otherwise we'd be seeing xml errors?
jonas’ 15:18:18
we’re not serving any Content-Type header
jonas’ 15:18:32
ah, we are
jonas’ slaps firefox 15:18:34
jonas’ 15:18:37
it’s of course text/html
pep. 15:18:42
:x
pep. 15:18:51
Ok so we're "not" doing xhtml :)
flow 15:19:05
like everybody else ;)
jonas’ 15:19:17
flow, except that we claim XHTML in the doctype
flow 15:19:30
pfff, doctype…
flow 15:19:46
do browsers even look at this?
flow 15:20:02
but yes, should definelty be fixed either way
jonas’ 15:20:45
going to fix both
pep. 15:20:49
I used to do <!DOCTYPE html><html xml:lang="en">..</html> declared as application/xhtml+xml
pep. 15:20:54
on a previous static website
Link Mauve 15:21:24
pep., with this doctype you can use either MIME types.
pep. 15:21:35
sure
pep. 15:21:43
I like when my browser tells me what I'm doing wrong :x
pep. 15:21:53
Instead of trying to interpret stuff randomly
Link Mauve 15:22:11
Same, I use the same Content-Type as yours wherever I can.
jonas’ 15:22:36
so the fix is to set indent='no' on xsl:output
pep. 15:23:31
it's been ages I haven't done xslt.. I should have a look again someday
jonas’ 15:27:00
flow, https://github.com/xsf/xeps/pull/885 (cc @ pep.)
pep. 15:28:05
do you have a rendered version of a xep? /me pulls the change
jonas’ 15:28:21
yeah sure, one sec
pep. 15:28:30
nvm
pep. 15:28:34
I'll do that locally so I can diff
jonas’ 15:28:40
https://sotecware.net/files/xeps/xep-0424.html
jonas’ 15:28:46
the diff won’t be pretty
jonas’ 15:28:58
like, seriously not pretty
jonas’ 15:29:02
due to indent='no'
pep. 15:29:03
I'm curious if the `indent='no'`is going to change much
pep. 15:29:05
right
jonas’ 15:29:12
yes
jonas’ 15:29:15
it removes all the linebreaks
pep. 15:31:56
yeah diff is a bit useless :x
pep. 15:32:08
I guess we'll see if that affects anything else?
jonas’ 15:33:05
yupp
pep. 15:33:19
LGTM otherwise
jonas’ 15:33:53
I’m going to put the html doctype on its own line tho
jonas’ 15:34:21
updated the PR
pep. 15:34:46
ok
pep. 15:34:50
sure
jonas’ 15:35:12
merged
pep. 15:35:13
Hmm, it does need to be on its own line no?
pep. 15:35:19
Just curious
jonas’ 15:35:21
I think it does
pep. 15:35:23
k
flow 15:40:50
jonas’, thanks for the attribution :)
jonas’ 15:41:47
thanks for finding it
stpeter has joined 15:53:35
flow 15:54:28
jonas’, and respect for finding *that* fix.
jonas’ 15:54:57
so, it was clear that this was the newline between the <li/> elements being interpreted as whitespace as per HTML
jonas’ 15:55:02
I’ve got some experience with messing with HTML ;)
jonas’ 15:55:10
then it was just a question of how to make XSLT not emit that newline
flow 15:57:15
yep, if you do not know or think about '\n' being transformed to ' ', then you are probably lost on this one
pep. 15:58:08
flow, I would think that's one of the first thing you learn when doing html :x
pep. 15:58:12
also xml:space !
flow 16:10:46
pep., the time where I did html is more than a decade ago, so I either never know or forgot about it
flow 16:11:11
pep., xml:space?
pep. 16:11:30
hmm, I might be mistaken but isn't that related?
pep. 16:11:37
Or it's just a rule of html?
flow 16:11:46
dunno
pep. 16:11:46
https://www.w3.org/TR/REC-xml/#sec-white-space
pep. 16:12:03
using 'default' and not preserve
pep. 16:12:12
I'd like that xmpp use that explicitely :x
flow 16:14:52
do we? I assume whitespace to be preserved e.g. in </body>, or is this other whitespace?
pep. 16:17:12
Maybe it's declared in the RFCs, but I don't think I've seen any mention of that in XEPs
lnj has left 16:28:01
lnj has joined 16:28:12
Link Mauve 16:28:18
jonas’, in HTML5, you don’t use <meta http-equiv="Content-Type" content="text/html; charset=utf-8"> anymore but <meta charset="utf-8"/>
jonas’ 16:28:24
Link Mauve, true!
jonas’ 16:28:29
make a PR, while you’re at it
jonas’ 16:28:31
? :)
Link Mauve 16:28:55
I have to send a letter to my UK bank, that’ll be a very good procrastination object!
pep. 16:29:05
nooo
pep. 16:29:12
so close yet so far
Link Mauve 16:29:41
jonas’, I… can’t find this element in the XSLT stylesheet.
jonas’ 16:29:55
I’m scared that xsl:output generates it
Link Mauve 16:30:02
Me too.
jonas’ 16:30:05
oh dear
jonas’ 16:30:11
Link Mauve, try without encoding=''
jonas’ 16:30:17
though I’d be hesitant to remove that again
jonas’ 16:30:20
otherwise we have to sed it
pep. 16:30:25
ugh
jonas’ 16:30:27
:D
Link Mauve 16:30:54
And add a normal <meta/> node?
jonas’ 16:32:49
maybe
Link Mauve 16:33:10
~~While you’re at it, maybe merge #669.~~ ✎
Link Mauve 16:33:15
While you’re at it, maybe merge https://github.com/xsf/xmpp.org/pull/669 ✏
jonas’ 16:34:02
I don’t have +w there anymore
Link Mauve 16:34:14
Ah.
jonas’ 16:34:31
otherwise I would’ve
pep. 16:34:37
I do, but I don't think I should (just because I'm board)
Link Mauve 16:34:38
That doesn’t sound like a thing that is wanted.
pep. 16:37:53
merged. xsf/xmpp.org stuff belongs to commteam now though
Link Mauve 17:04:45
Also, meh, #879 didn’t reach Council.
Link Mauve 17:04:52
Despite it being quite trivial.
jonas’ 17:05:03
Link Mauve, it did
jonas’ 17:05:06
everyone voted +1 on it
jonas’ 17:05:33
I’m not sure why you think it didn’t, because it was both on the agenda and discussed in the meeting
Link Mauve 17:05:52
Ah, it wasn’t in the minutes.
jonas’ 17:06:04
there haven’t been any minutes sent yet?
jonas’ 17:06:12
those are from last week
Link Mauve 17:06:14
Oh, it was for last week.
Link Mauve 17:06:17
Nvm then. :)
lnj has left 18:06:13
lnj has joined 18:06:51
lnj has left 18:11:20
lnj has joined 18:11:27
lnj has left 18:12:18
lnj has joined 18:12:30
winfried has left 18:31:32
winfried has joined 18:31:33
winfried has left 18:32:06
winfried has joined 18:32:10
winfried has left 19:49:43
winfried has joined 19:49:44
winfried has left 20:26:27
winfried has joined 20:26:28
debacle has left 20:45:11
lnj has left 21:10:23
Tobi has left 21:39:08
debacle has joined 22:47:17
winfried has left 23:36:15
winfried has joined 23:36:16
winfried has left 23:36:54
winfried has joined 23:36:55
winfried has left 23:48:37
winfried has joined 23:48:38
winfried has left 23:49:49
winfried has joined 23:49:50