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