Point of order:

I am planning on using this module, but I am not sure how my target audience is going to take to the word axis. I get it, you get it, I'm sure most other Drupal folks get it, but it is overly technical for general use, IMHO. I think the word "criteria" is more widely understood/acceptable. I understand why we wouldn't use the word "category" even though that would be even more applicable.

Comments

jo1ene’s picture

jo1ene commented

I see how deeply embedded in the code it is. Could this be configurable for display purposes? Like Drupal 5.0 content types admin - you can change the title field label and such.

vm’s picture

vm commented

If the text is wrapped in a t() string in the .module file, then it can be changed using the locale.module which is part of core.

jo1ene’s picture

jo1ene commented

True. But I hate hacking things that way. If nobody is interested in such an option, Crell can mark this wont fix.

vm’s picture

vm commented

Was just offering it up as an option. using the locale.module to change text strings, wouldn't fall under "hacking" as far as I know. Again was just an option for you while awaiting Crell's final decision.

Crell’s picture

Crell commented

Does the UI currently say "axis" on user-side pages, or only on admin-targeted pages? I can see the argument for changing or making the name configurable for user-side pages, but for admin-targeted pages I don't think the concept of a "review axis" is too much mental overhead (especially when, as mentioned, Drupal already uses a lot of good words).

BradM’s picture

BradM commented

Well I don't even know what 'axis' refers to on the drupal side of things. ;)

However, it can be changed from a user standpoint, so the teaser doesn't show this term.

Edit nodereview_node_nodereview.inc, line 332, which is:
$header = array(t('axis'), t('score'), t('review'));

Change 'axis' to a different word.

jo1ene’s picture

jo1ene commented
Status: Active » Closed (fixed)