The way the additional row of fields is created is kind of snazzy, but it's a bit contrary to normal drupal behavior. I would suggest adding an 'add another row' button of some sort, rather than auto-creating a new row when the field name field loses focus and has data present.

It's also not clear that you have to manually add the primary key field in the field list. This is probably an issue that can be fixed by adding a description to that field, or some help-text about how the form functions somewhere on the page.

It seems to work well though, user experience issues aside.

Comments

sukr_s’s picture

the rows are shown using the #states behaviour in D7, which allows to dynamically hide / show fields based on input in other fields. i guess it's a new UI pattern introduced in D7...

for primary key (and UI in general), i agree it can be better. currently i'm focusing on the functionality to save lot of time writing monotonous code. beautification will follow :-)

willvincent’s picture

the rows are shown using the #states behaviour in D7, which allows to dynamically hide / show fields based on input in other fields. i guess it's a new UI pattern introduced in D7...

Ahh nifty, I wasn't aware of this. :)

sukr_s’s picture

Status: Active » Closed (won't fix)