Hey it's ok:)

no problem i'll go ahead and change it again..

I havent formed a opinion on code style yet:)

regards Nino

Ryan Holmes wrote:
Wow, that sounded a little harsh. Please consider it as a recommendation/observation, not an order ;)

-Ryan

On Aug 19, 2007, at 1:26 PM, Ryan Holmes wrote:

That should be 'BbTextArea' and 'BbLabel'. Only the first letter of standard abbreviations should be capitalized. This convention is recommended by Scott Ambler (among others) and Wicket uses it very consistently. So should contributed Wicket code.

My personal preference would be 'BbCodeTextArea' and 'BbCodeLabel'. The names are still short, but much clearer (the extra four characters certainly won't bother anyone).

-Ryan

On Aug 16, 2007, at 4:29 AM, Gerolf Seitz wrote:

i took a quick look at it and i have to say that it works nice (also
detecting manually closed bb tags).
i personally would prefer not naming the bb components TextArea and Label,
because if you want to use both (wicket and bb) you would have to fully
qualify at least one of them.
why not call them BBTextArea and BBLabel? that way you know at first sight
what it really does/is.

btw: there are some errors in the example code on the wiki. i would only need to hit the save button to save the corrections i made, just wanted to
ask for your consent...

gerolf


On 8/16/07, Nino Saturnino Martinez Vazquez Wael <[EMAIL PROTECTED]>
wrote:

sure will do:)

Gerolf Seitz wrote:
if you want more exposure, maybe also post this announcement on the
users
list.
a wiki page (maybe with a screenshot) would be great too. also make sure that you add the label "component" so that it appears on the wiki start
page.

gerolf

On 8/16/07, Nino Saturnino Martinez Vazquez Wael <
[EMAIL PROTECTED]>
wrote:

So now its working.. Knock yourselfs out crasy writing bbcode:)

A little about the component:

This is a very basic version.

Has a bbcodecontrol where you can click on buttons to insert bbcode in
a
previously marked textfield or textarea. Textfield and textarea are
supplied by the component.

Currently only supports simple formatting like bold italic etc. Has no
validation.
requireds jdk 1.5


regards Nino










Reply via email to