At admin/content/nodequeue/23/view/20 there is an Edit tab.
This actually edits the parent queue -- the tab label should reflect that, or the tab should be removed, as a tab jumping you up several levels is weird navigation.

CommentFileSizeAuthor
#2 625452-D6.patch479 bytesamateescu
#2 625452.patch481 bytesamateescu

Comments

joachim’s picture

joachim commented

I don't have the UI in front of me, but perhaps this could be put as a link in the text of that page instead?

amateescu’s picture

amateescu commented
Status: Active » Fixed
StatusFileSize
new 625452.patch481 bytes
new 625452-D6.patch479 bytes

Taking into consideration Views' 7.x "Edit view" contextual link and the fact that tabs were designed for this kind of navigation links, I decided to simply rename the tab title to "Edit queue".

This also fits with #670282: Change all references to "nodequeue" to queue and subqueue where appropriate.

Commited attached patches to 6.x and 7.x.

http://drupalcode.org/project/nodequeue.git/commit/42636d3
http://drupalcode.org/project/nodequeue.git/commit/c1b9265

joachim’s picture

joachim commented

But if it edits the parent queue, it's still jumping you around in the navigation tree, isn't it?

amateescu’s picture

amateescu commented

It is, but at least it tells you a bit more about where you're going.

joachim’s picture

joachim commented

That's still a really weird thing for a tab to do -- tabs go one level down, they don't jump around.

amateescu’s picture

amateescu commented

Well, I guess we can make an exception because there's no one level for editing subqueues? Or you *really* want that tab gone? :)

Status: Fixed » Closed (fixed)

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