Hi there,

I'm starting to use your module in a new project and I like to share my experiences with you regarding some processes and UI stuff.

1. Permission
An user might be able to add/edit a nodeblock, but you don't want to give him the possibility to change settings within the node edit form.
If I don't give the user the permission "Maintain Nodeblock on a node" the block is not created even I have enabled it in the node type settings.

I think the block should be created regardless of this permission.

2. Block subject
You introduced the optional block title field in the node form. For the permission scheme we use and the processes of our client this is not really handy.
First of all (see #1) the user does not have the permission to edit those settings.
Second, what I really need is the possibility to omit the block subject and serving only the content to the user.
Because the node title is important to identify the node in the content admin and is mandatory anyway, we can't add an empty text.
What I did now, I added a new field "Block title", which the user can leave empty and otherwise is used as the block subject.

Summarised I would suggest to:
- remove your block title field
- add a field api field for the block subject

3. Block subject naming
You add hardcoded "(nodeblock)" to the title. Please just remove it. I think for an editor it is not important.

Thanks for the good module!

Comments

Johnny vd Laar’s picture

Thanks for your suggestions.

#1 is a bug and I will fix that.
#3 you are right and I will remove this.

About #2, the block title field currently is the title used on the block admin screen not as block title when you display the node. But perhaps you are right that we should use the node title as title inside the admin interface and the new "block title field" as the "display title" and then add a possibility to keep it empty.

Johnny vd Laar’s picture

Status: Active » Fixed

I committed some fixes based on your comments:
http://drupalcode.org/project/nodeblock.git/commitdiff/6aeec4f?hp=d6f622...

this also includes a new override field where you can enable the fields on the node form.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.