BB Codes - Githubissues

csrdelft / onontdekt

Hybride app voor C.S.R. Delft

https://csrdelft.nl/lustrum

1 stars 1 forks source link

BB Codes #55

Closed Manduro closed 7 years ago

Manduro commented 7 years ago

Nog te implementeren:

[x] [fotoalbum]
[x] [peiling]
[x] [maaltijd]

Verbeteringen:

[x] [foto] via native photo viewer?
[x] Ketzers, documenten, fotoalbums, etc. in groter full width item?
[x] [lid] en [citaat] halen naam op voor een uid

Bugs:

[x] [lid=1403] en [lid=1419]
[x] [offtopic][citaat=1604]citaat[/(citaat)]reactie[/]
[x] Citaat van ...: meerdere _ vervangen

qurben commented 7 years ago

Is het niet een idee om de server de bb te laten parsen? Op die manier krijg je consistentie tussen de app en de stek.

Er zitten nog een heel aantal gekke dingen in de bbparser van de stek, bijv: [rul], lege sluit tags [/] en misschien nog wel at andere dingen die er ooit ingeslopen zijn.

Er zal dan misschien nog wel wat moeten veranderen in de teruggegeven html van de stek, maar dat kan gefixt worden. De stek kan eventueel ook andere dingen teruggeven aan de app, als dat nodig is.

Manduro commented 7 years ago

Ja ik twijfel er al vanaf het begin over. BB codes zijn naar en vooral die lege sluit tags. Er bestaat ook geen goede parser in JS.

Er is nog wel eea te verbeteren, maar het wordt nooit ideaal zo, dus misschien toch maar server-side doen. Hij moet dan wel andere dingen teruggeven voor de app inderdaad, omdat er veel specifieke template code in mee komt. Zou jij daar een snel structuur opzetje voor willen klussen, dat ik de verdere invulling van elke tag doe?

qurben commented 7 years ago

Ik heb er een issue voor gemaakt op https://github.com/csrdelft/csrdelft.nl/issues/287

Ik zal even in de groep gooien of iemand m wil oppakken, anders kan ik binnenkort wel wat bouwen. Zit helaas wel redelijk krap op tijd.