Thomas - 12/07/2007 15:20 (laatste wijziging 12/07/2007 15:20)
Moderator
Voer pas veranderingen door als ze ook echt af zijn?
Overigens, dat scriptje is van toepassing op de link die je ZIET in een reactie, niet de daadwerkelijke link in je href.
Dus als je een link X hebt, die je met een regexp uit een reactie hebt, vervang je deze door <a href="X">wordwrap2(X)</a>.
Ik dacht dat je daar zelf wel op zou komen.
Je moet zelf niet zo zeiken als mensen commentaar leveren op functionaliteit die niet werkt, niet getest is of gewoon niet af is.
In dat opzicht was mijn code beter, deze werkte tenminste.
Ik heb in IE trouwens ook een lelijke vertical scrollbar in codeblokken, dat ziet er ook echt niet uit.
Ibrahim - 12/07/2007 15:21 (laatste wijziging 12/07/2007 15:25)
PHP expert
@jelle, als je een quote vergeet in PHP, krijg je een parse error, dat hij door gaat kleuren betekent dus dat je de quote moet afsluiten... geen fout, behalve voor jou misschien
@Fangorn, die scrollbar hoort er te zijn... Bekijk de source maar eens
Als je dan toch verder wilt met deze nieuwe UBB, voer deze dan site-wide door. Het feit dat alle onderdelen apart worden "gepatched" geeft aan dat zaken die centraal geregeld zouden moeten zijn, dit niet zijn. Dit was in het verleden het "probleem" met de UBB / codeblokken (eerst bestond dit alleen op het forum, daarna is dit overal gecopy-paste op de rest van de site, in plaats van het in een include of class zetten). Nu is de eerste stap gezet met een class / parser / whatever. Regel het dan meteen centraal en pas alle onderdelen aan met gebruik van deze centrale class.
Straks heb je anders (wéér) tig varianten van hetzelfde spul rondzwerven. Het zou nogal dom zijn om opnieuw deze fout te maken. Dit geeft ook een beetje aan dat het ontwerp van deze site rammelt, en aan een grondige revisie toe is. Durf gewoon eens een nieuw, degelijk ontwerp te maken, in plaats van eindeloos door te bouwen op en aan oude rommel (V1).
@Fangorn, we hebben expres de ubb alleen toegepast op het forum, we waren van plan te wachten totdat bijna alle bugs eruit zijn (want alle bugs eruit in een script is in mijn ogen onmogelijk) dan word alles van de oude ubb of gedelete of beland in een archief. Daarna word alles geinclude (wat hier ook gebeurt )
Als je dan toch verder wilt met deze nieuwe UBB, voer deze dan site-wide door. Het feit dat alle onderdelen apart worden "gepatched" geeft aan dat zaken die centraal geregeld zouden moeten zijn, dit niet zijn. Dit was in het verleden het "probleem" met de UBB / codeblokken (eerst bestond dit alleen op het forum, daarna is dit overal gecopy-paste op de rest van de site, in plaats van het in een include of class zetten). Nu is de eerste stap gezet met een class / parser / whatever. Regel het dan meteen centraal en pas alle onderdelen aan met gebruik van deze centrale class.
Straks heb je anders (wéér) tig varianten van hetzelfde spul rondzwerven. Het zou nogal dom zijn om opnieuw deze fout te maken. Dit geeft ook een beetje aan dat het ontwerp van deze site rammelt, en aan een grondige revisie toe is. Durf gewoon eens een nieuw, degelijk ontwerp te maken, in plaats van eindeloos door te bouwen op en aan oude rommel (V1).
Fangron ik heb al meerdere malen gereageerd op een reply van je met de opmerking dat we als crew bezig zjin met V3. Zou je dan nu willen stoppen met de opmerkingen van 'op oude zooi verder borduren' en 'eens beginnen met een degelijk ontwerp' ? We zijn namelijk met een ontwerp bezig die niet voor borduurd op de oude 'rommel' en ook nog degelijk is.
en een quote in een quote geeft ook problemen zo blijkt uit bovenstaand voorbeeld.
woeps, per ongeluk verkeerde knopje gedrukt.
Maar om duidelijkheid te scheppen.
Citaat:
[quote]bla
blaat[/quote] zal de quote in de quote niet parsen. Normaal dient er gebruik gemaakt te woren van het quote knopje onder de repley/topic. Ik ga dat knopje eens bekijken vanavond.
@simon: De vorige quote werkt ook zo. Je moet maar eens een reactie quoten in een nieuwsbericht die een quote bevat . De quote zal vervangen worden door [..quote..] en dit hebben we nu ook gedaan. Trouwens zal dit op den duur lelijk worden en onoverzichtelijk. En trouwens je quote iemand zijn reactie en niet op de quote in de reactie. Dan moet je die reactie maar quoten (ingewikkeld hé ).
Er is nog een ding wat ik heel erg mis, namelijk opsommingen. Zou het niet een idee zijn om deze toe te voegen? dus bijvoorbeeld dat of [*] of * worden opgezet naar list items?
<?php preg_replace('#\[mysql\](.+?)\[/mysql\]#si','Opzoeken op MySQL.org: <a href="http://dev.mysql.com/doc/mysql/search.php?version=5.0&q=\\1" target="_blank">\\1</a>',$this->text);
?>
<?phppreg_replace('#\[mysql\](.+?)\[/mysql\]#si','Opzoeken op MySQL.org: <a href="http://dev.mysql.com/doc/mysql/search.php?version=5.0&q=\\1" target="_blank">\\1</a>',$this->text);
?>
@siliecom14: NOG EEN BUG: bekijk de preg_match is origineel en dan gepost moeten dubbele \\ zijn voor de 1 en niet een enkele.
(lol) hier doet hij het ook,
Nog lastig is ik geef meestal een enter achter de [ code ] tags en dan krijg je een blank lijn. zie hierboven.
De quote knop werkt. Meneer'ke siliecom14 heeft de knop zichtbaar gemaakt voor iedereen maar de achterliggende code had hij verkeerd gemaakt Hij maakte het zo zodat de crew de knop kon gebruiken. (ja we zijn al bezig met schelden op hem ).