Closed (fixed)
Project:
Pathauto
Version:
7.x-1.x-dev
Component:
User interface
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
28 Dec 2010 at 19:54 UTC
Updated:
22 Feb 2011 at 18:20 UTC
Jump to comment: Most recent file
Comments
Comment #1
gregglesBoth of these seem worthwhile to me.
Comment #2
dave reidHrm, we're using 'Content' now in D7. It might be confusing that we're all of a sudden using 'Node' in the Pathauto UI text compared to the rest of the Drupal UI.
Comment #3
gregglesDidn't we switch to "Content" and then switch back once already in the 4.7 -> 5 -> 6 time frame? Bah.
Let's do it again :)
Comment #4
tsvenson commented+1 for using Content as well. In this case I would like to suggest that "Content pages" is used. That way it will be more clear that it has nothing to do with comments and other non node related pieces of content.
Comment #5
Freso commentedHere's a quick patch by me editing only through grep and vi. I haven't had a site to look at while changing the text, so I'm guessing some of them might be a bit silly. And I probably didn't around to some either.
Comment #6
dave reidI'm not keen on using 'content pages'. admin/content = 'Content'. node/add = 'Add content'.
Should be 'content' instead of 'content pages'
Again should just be 'content'
Same
Same
This should be 'Content paths' for the same as the other pathauto types.
Powered by Dreditor.
Comment #7
crashtest_ commentedRerolled based on the comments above.
Comment #8
Freso commentedHm. Should be rephrased? "... core content types: content, ..." sounds rather clumsy, plus it might create confusion if taxonomy terms and users are described as "content" here while "content" exclusively refers to nodes in all other places.
I've moved "bypass" back up on the first line again now that there's room again, as it will not break the 80 characters-per-line guideline, but I kept the "the" on the second line for prettyness. :)
I also caught a last "node type" that should be "content type" in API.txt.
Oh, and the .po's shouldn't be altered. For one, they're being decommissioned, and even if they weren't, PotX should've been the one to provide a new .pot for translators to work from.
Apart from the first comment, I think it should be pretty solid now.
Comment #9
dave reidRe-uploading without the 'd7' suffix so it can get tested by qa.drupal.org.
Comment #10
dave reidPasses locally 100%, so committed to 7.x-1.x!
http://drupal.org/cvs?commit=480398
Comment #11
Freso commentedHow about
Comment #12
dave reidWe should use core 'entity' types: content, terms, etc.
Comment #13
Freso commentedLike so?
Comment #14
Freso commentedDave said go, so I go'ed. :)
Also, I think the final agreement in #drupal-seo was that we shouldn't backport these wording changes to 6.x-2.x, so marking the issue "fixed".
Comment #15
jenlamptonI'm still wishing for this in D6 :)
.02