Suggestion: Add class and id attributes to form elements

RT Developers
1

A suggestion for future RT development (post-3.6.1), if this is not
already in the works: apply class and id attributes more systematically to
form elements. This will give developers more options with CSS and DOM
scripting that avoid RT code customization.

Thanks,
David

2

It’s something we’d be thrilled to do. We’d be even more thrilled if someone started sending patches… wink------- Original message -------
From: David Chandek-Stark david.chandek.stark@duke.edu
Sent: 10/27, 15:52

A suggestion for future RT development (post-3.6.1), if this is not
already in the works: apply class and id attributes more systematically to
form elements. This will give developers more options with CSS and DOM
scripting that avoid RT code customization.

Thanks,
David

List info: The rt-devel Archives

3

Fair enough. What’s the best way to do contribute?

Thanks,
David

It’s something we’d be thrilled to do. We’d be even more thrilled if
someone started sending patches… wink

------- Original message -------
From: David Chandek-Stark david.chandek.stark@duke.edu
Sent: 10/27, 15:52

A suggestion for future RT development (post-3.6.1), if this is not
already in the works: apply class and id attributes more
systematically to
form elements. This will give developers more options with CSS and DOM

scripting that avoid RT code customization.

Thanks,
David

List info:
The rt-devel Archives

4

Fair enough. What’s the best way to do contribute?

Start sending along unified diffs. (diff -ur). Probably worth sending
the first few to rt-devel for commentary. But this is really cool.
Thanks!

5

Hello!2006/10/27, Jesse Vincent jesse@bestpractical.com:

It’s something we’d be thrilled to do. We’d be even more thrilled if someone started sending patches…

Patch for html/Elements/MessageBox. Adds id=“messagebox”.
IMHO, class=“messagebox” should remain for compatibility reasons.

Alexandr Ciornii, http://chorny.net

MessageBox.patch (665 Bytes)

6

Hello!

It’s something we’d be thrilled to do. We’d be even more thrilled if
someone started sending patches…

Patch for html/Elements/MessageBox. Adds id=“messagebox”.
IMHO, class=“messagebox” should remain for compatibility reasons.

My one concern is that it’s quite possible to use “MessageBox” twice on
one page. Why not make the id == the name here?

-j