Thanks for a great module, which saved me some work when a client asked for this exact function. Simple, efficient and easy to understand and use, just as I like it!

May I suggest that the Move down and Move up links in the node view actually get links ASAP? It seems a little odd that they exist, are displayed by default, but don't have any menu items in hook menu and thus no function at all... I realize this is work in progress, but just disabling them by default until they work would save some confusion.

May I also suggest that they get a class, so they can be styled? Right now they are just plain a href's, which cannot be hit by specific CSS. Either giving them a wrapper, rendering them as a list with a class or simply giving the href class will do the trick. That will enable styling them as arrows, moving them, changing font, colors etc., which will be very useful.... once they work.

Martin/Vertikal

Comments

vertikal.dk created an issue.

davps’s picture

davps commented

@vertikal.dk

May I suggest that the Move down and Move up links in the node view actually get links ASAP? It seems a little odd that they exist, are displayed by default, but don't have any menu items in hook menu and thus no function at all

Try to use 7.x-1.x-dev, this functionality was missed on porting from 6.x to 7.x. Functionality was ported and manually tested about year ago.

Or try to apply patch #2878287-2: Orderable links doesn't work.

May I also suggest that they get a class, so they can be styled? Right now they are just plain a href's, which cannot be hit by specific CSS. Either giving them a wrapper, rendering them as a list with a class or simply giving the href class will do the trick. That will enable styling them as arrows, moving them, changing font, colors etc., which will be very useful.... once they work

As quick solution, try to use theme_link preprocess to modify links.

vertikal.dk’s picture

vertikal.dk commented

Thanks for the feedback.
I'll try the dev-version.
Regarding the styling: I realize that theming is almost alawys an option, but I still think the links should have a class out of the box. That would also enable non-programmers to style the links.

Martin/Vertikal