Hi guys,

Nothing major, but a bit confusing still.

function rules_action_taxonomy_load_term_form(...) {
  ...
  '#title' => t("!vocab's terms", array('!vocab' => $vocabulary->name)),
  ...
  '#description' => t('Optional: enter the term id (<em>not the term name</em>) that should be loaded . If this field is used "Select a term" field will be ignored.'),

The description of the text area following the select mentions "Select a term". But the title really is "My Vocab terms".

I vote for replacing the select box title by "Select a term" because "!vocab's terms" does not tell you anything about what you're supposed to do with that term...

Back soon with a patch to add functionality!
Thank you.
Alexis Wilke

CommentFileSizeAuthor
#3 rules-6.x-taxonomy_term_load.patch1.26 KBAlexisWilke

Comments

AlexisWilke’s picture

Component: User Interface » User interface

Btw, I know why it is done that way, but I think it is wrong.

When you get to that form, the vocabulary was defined on a previous screen and people could not see what it was. I suggest you do it like I've done it in the "Add new term" patch. That is, you add a concise message before hand that says which vocabulary was selected...

Thank you.
Alexis

fago’s picture

Sounds good, feel free to provide a patch.. :)

AlexisWilke’s picture

Assigned: Unassigned » AlexisWilke
StatusFileSize
new1.26 KB

He! He! 8-)

Okay, I changed the two messages. The #title is just "Select a term" and the #description mentions the vocabulary name.

Thank you.
Alexis

fago’s picture

Status: Active » Fixed

thanks, committed! =)

Status: Fixed » Closed (fixed)

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