Hey, lets make those things on the node/add page editable. And fix up some code style along the way.

CommentFileSizeAuthor
#5 node.module_6.diff2.51 KBdrumm
#1 node.module_3.diff3.5 KBdrumm
node.module_2.diff4.44 KBdrumm

Comments

drumm’s picture

drumm
he/him
Location NY, US
commented
StatusFileSize
new node.module_3.diff3.5 KB

Smaller patch which doesn't break things.

Bèr Kessels’s picture

Bèr Kessels commented

I like this. But it needs some small changes IMO.

on the node/add (function node_add) we should use DL and DT instead of a LI with a nested DIV. We also should avoid having DIVS without classes and such, for they only add clutter.

Bèr

dries’s picture

dries commented

This will be fixed by the CCK.

killes@www.drop.org’s picture

killes@www.drop.org commented

Let's "won't fix" it then.

drumm’s picture

drumm
he/him
Location NY, US
commented
StatusFileSize
new node.module_6.diff2.51 KB

An updated patch for testing and review by the CCK team.