XSF Editor Team - 2020-01-22

  1. stpeter has left

  2. debacle has left

  3. stpeter has joined

  4. stpeter has left

  5. Tobi has joined

  6. stpeter has joined

  7. stpeter has left

  8. lnj has joined

  9. winfried has left

  10. winfried has joined

  11. winfried has left

  12. winfried has joined

  13. winfried has left

  14. winfried has joined

  15. winfried has left

  16. winfried has joined

  17. winfried has left

  18. winfried has joined

  19. winfried has left

  20. winfried has joined

  21. debacle has joined

  22. winfried has left

  23. debacle has left

  24. winfried has joined

  25. debacle has joined

  26. winfried has left

  27. winfried has joined

  28. winfried has left

  29. winfried has joined

  30. winfried has left

  31. winfried has joined

  32. winfried has left

  33. winfried has joined

  34. stpeter has joined

  35. Guus has left

  36. Guus has joined

  37. winfried has left

  38. winfried has joined

  39. winfried has left

  40. winfried has joined

  41. stpeter has left

  42. winfried has left

  43. winfried has joined

  44. winfried has left

  45. winfried has joined

  46. flow

    "Authors: Lance Stout , JC Brand" why is there an extra space before the comma?

  47. winfried has left

  48. winfried has joined

  49. jonas’


  50. jonas’

    probably XSL bug

  51. flow

    jonas’, i think it's a css bug

  52. jonas’


  53. jonas’


  54. flow

    affecting all the xeps

  55. jonas’

    I don’t want to go through all the XEPs to find one with >1 author, please

  56. jonas’

    you seem to know at least one affected

  57. flow

    jonas’, I think you can select a random xep with multiple authors

  58. jonas’


  59. flow


  60. jonas’

    I don’t know a single XEP off the top of my head with >1 author

  61. jonas’

    and it’s not visbile in the extension list

  62. flow


  63. jonas’


  64. jonas’

    XSL bug

  65. jonas’

    thanks, looking into it

  66. flow

    doesn't the xsl just build an html list?

  67. jonas’


  68. jonas’

    it adds spaces between the <li/> tho

  69. jonas’

    cause bug

  70. flow

    curious to see the fix, as I only see the space introduced between firstname and surname in the xsl

  71. jonas’


  72. jonas’

    not quite sure yet where it comes from

  73. jonas’

    (to be clear: the "space" is the newline between the <li/> element?)

  74. jonas’

    (to be clear: the "space" is the newline between the <li/> elements)

  75. jonas’

    eh, also, we’re claiming XHTML, but writing HTML :/

  76. pep.


  77. jonas’

    but that’s a different hill to work on

  78. pep.


  79. jonas’

    I blame the xsltproc html writer

  80. pep.

    We're probably not serving the correct mime type though right? Otherwise we'd be seeing xml errors?

  81. jonas’

    we’re not serving any Content-Type header

  82. jonas’

    ah, we are

  83. jonas’ slaps firefox

  84. jonas’

    it’s of course text/html

  85. pep.


  86. pep.

    Ok so we're "not" doing xhtml :)

  87. flow

    like everybody else ;)

  88. jonas’

    flow, except that we claim XHTML in the doctype

  89. flow

    pfff, doctype…

  90. flow

    do browsers even look at this?

  91. flow

    but yes, should definelty be fixed either way

  92. jonas’

    going to fix both

  93. pep.

    I used to do <!DOCTYPE html><html xml:lang="en">..</html> declared as application/xhtml+xml

  94. pep.

    on a previous static website

  95. Link Mauve

    pep., with this doctype you can use either MIME types.

  96. pep.


  97. pep.

    I like when my browser tells me what I'm doing wrong :x

  98. pep.

    Instead of trying to interpret stuff randomly

  99. Link Mauve

    Same, I use the same Content-Type as yours wherever I can.

  100. jonas’

    so the fix is to set indent='no' on xsl:output

  101. pep.

    it's been ages I haven't done xslt.. I should have a look again someday

  102. jonas’

    flow, https://github.com/xsf/xeps/pull/885 (cc @ pep.)

  103. pep.

    do you have a rendered version of a xep? /me pulls the change

  104. jonas’

    yeah sure, one sec

  105. pep.


  106. pep.

    I'll do that locally so I can diff

  107. jonas’


  108. jonas’

    the diff won’t be pretty

  109. jonas’

    like, seriously not pretty

  110. jonas’

    due to indent='no'

  111. pep.

    I'm curious if the `indent='no'`is going to change much

  112. pep.


  113. jonas’


  114. jonas’

    it removes all the linebreaks

  115. pep.

    yeah diff is a bit useless :x

  116. pep.

    I guess we'll see if that affects anything else?

  117. jonas’


  118. pep.

    LGTM otherwise

  119. jonas’

    I’m going to put the html doctype on its own line tho

  120. jonas’

    updated the PR

  121. pep.


  122. pep.


  123. jonas’


  124. pep.

    Hmm, it does need to be on its own line no?

  125. pep.

    Just curious

  126. jonas’

    I think it does

  127. pep.


  128. flow

    jonas’, thanks for the attribution :)

  129. jonas’

    thanks for finding it

  130. stpeter has joined

  131. flow

    jonas’, and respect for finding *that* fix.

  132. jonas’

    so, it was clear that this was the newline between the <li/> elements being interpreted as whitespace as per HTML

  133. jonas’

    I’ve got some experience with messing with HTML ;)

  134. jonas’

    then it was just a question of how to make XSLT not emit that newline

  135. flow

    yep, if you do not know or think about '\n' being transformed to ' ', then you are probably lost on this one

  136. pep.

    flow, I would think that's one of the first thing you learn when doing html :x

  137. pep.

    also xml:space !

  138. flow

    pep., the time where I did html is more than a decade ago, so I either never know or forgot about it

  139. flow

    pep., xml:space?

  140. pep.

    hmm, I might be mistaken but isn't that related?

  141. pep.

    Or it's just a rule of html?

  142. flow


  143. pep.


  144. pep.

    using 'default' and not preserve

  145. pep.

    I'd like that xmpp use that explicitely :x

  146. flow

    do we? I assume whitespace to be preserved e.g. in </body>, or is this other whitespace?

  147. pep.

    Maybe it's declared in the RFCs, but I don't think I've seen any mention of that in XEPs

  148. lnj has left

  149. lnj has joined

  150. Link Mauve

    jonas’, in HTML5, you don’t use <meta http-equiv="Content-Type" content="text/html; charset=utf-8"> anymore but <meta charset="utf-8"/>

  151. jonas’

    Link Mauve, true!

  152. jonas’

    make a PR, while you’re at it

  153. jonas’

    ? :)

  154. Link Mauve

    I have to send a letter to my UK bank, that’ll be a very good procrastination object!

  155. pep.


  156. pep.

    so close yet so far

  157. Link Mauve

    jonas’, I… can’t find this element in the XSLT stylesheet.

  158. jonas’

    I’m scared that xsl:output generates it

  159. Link Mauve

    Me too.

  160. jonas’

    oh dear

  161. jonas’

    Link Mauve, try without encoding=''

  162. jonas’

    though I’d be hesitant to remove that again

  163. jonas’

    otherwise we have to sed it

  164. pep.


  165. jonas’


  166. Link Mauve

    And add a normal <meta/> node?

  167. jonas’


  168. Link Mauve

    While you’re at it, maybe merge #669.

  169. Link Mauve

    While you’re at it, maybe merge https://github.com/xsf/xmpp.org/pull/669

  170. jonas’

    I don’t have +w there anymore

  171. Link Mauve


  172. jonas’

    otherwise I would’ve

  173. pep.

    I do, but I don't think I should (just because I'm board)

  174. Link Mauve

    That doesn’t sound like a thing that is wanted.

  175. pep.

    merged. xsf/xmpp.org stuff belongs to commteam now though

  176. Link Mauve

    Also, meh, #879 didn’t reach Council.

  177. Link Mauve

    Despite it being quite trivial.

  178. jonas’

    Link Mauve, it did

  179. jonas’

    everyone voted +1 on it

  180. jonas’

    I’m not sure why you think it didn’t, because it was both on the agenda and discussed in the meeting

  181. Link Mauve

    Ah, it wasn’t in the minutes.

  182. jonas’

    there haven’t been any minutes sent yet?

  183. jonas’

    those are from last week

  184. Link Mauve

    Oh, it was for last week.

  185. Link Mauve

    Nvm then. :)

  186. lnj has left

  187. lnj has joined

  188. lnj has left

  189. lnj has joined

  190. lnj has left

  191. lnj has joined

  192. winfried has left

  193. winfried has joined

  194. winfried has left

  195. winfried has joined

  196. winfried has left

  197. winfried has joined

  198. winfried has left

  199. winfried has joined

  200. debacle has left

  201. lnj has left

  202. Tobi has left

  203. debacle has joined

  204. winfried has left

  205. winfried has joined

  206. winfried has left

  207. winfried has joined

  208. winfried has left

  209. winfried has joined

  210. winfried has left

  211. winfried has joined