Closed (fixed)
Project:
Metatag
Version:
7.x-1.x-dev
Component:
Other tags
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
17 Dec 2012 at 13:01 UTC
Updated:
14 Sep 2017 at 12:41 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
remkovdz commented+1. Right now, I use the Clean Pagination module to achieve this, but I'd rather have this handled by Meta Tags too. Seems logical?
Comment #2
damienmckennaComment #3
rattusrattus commented+1
Makes a lot of sense to me. I'd like to get involved in the development but after an hour poking round the module source I'm quite confident I'd need some guidance.
Comment #4
ForTheWin commented@remkovdz, Thank you for posting the link to the Clean Pagination module!! This is exactly what I'm looking for.
Comment #5
ForTheWin commentedComment #6
ben finklea commentedGreat idea. Any word on this?
Comment #7
damienmckennaWaiting for someone to put together a patch; I'll get to it eventually (#1572474: Add Revisions to Metatags for Entities comes first) if nobody beats me to it.
Comment #8
andrew_ar commented@DamienMcKenna, any luck you can add this? It'd be a great edition to MetaTags.
Comment #9
Anonymous (not verified) commentedAny hope of adding this?
In the Advanced section of Meta Tags, on a node level, I can declare anything I want for a Canonical tag.
Why would a similar type of input box for rel="prev" and rel="next" be a difficult feature to add to Meta Tags too?
Comment #10
damienmckenna@broadway: It isn't difficult and that's the plan, it's time to write it. Until now I've been focused on a busy primary job, several side contracts, keeping my family going (with a currently 17mo infant, two older children, wife, dog, two cats), trying to keep healthy & sane, and several bugs in several modules I maintain. I just haven't gotten to it, nor has anyone else submitted a patch.
I've tagged this as being a "novice" tag as it's actually quite simple to add a single (or two) extra meta tags to the module at this point: just decide whether they should go in the main module or a submodule, edit that module's "*.metatag.inc" file, scroll down to the hook_metatag_info() implementation, duplicate some of the $info['tags'] variables and tweak it accordingly, then upload the patch and collaborate with others in the community to make sure it works as intended. (I just posted a documentation with this info)
And yes, I could have written the patch in the time it has taken to write this, but I wanted to explain to you that everyone can help improve the module, and indeed Drupal as a whole, you don't have to sit waiting for someone else to do it. And you should definitely be patient with and respectful of those who pour tens and hundreds hours of their personal time into open source projects, rather than asking why something hasn't been done yet.
If you would like to try helping with this or any project, the first step is to learn how to write patches, then work with the issue queue to find issues you think you could help with, and collaborate to improve the patches sufficiently so that they may be marked as "Reviewed and Tested By Community" (i.e. "RTBC"); if you're interested you might try asking to help as a co-maintainer.
There are also people on IRC and local user groups that can help you learn to write patches, if you're unfamiliar with the process.
Thank you for collaborating with us :)
Comment #11
preshetin commentedWill try.
Comment #12
damienmckenna@preshetin: Excellent, thank you! If you need help doing a patch you can catch me in #drupal-seo on IRC.
Comment #13
preshetin commentedIn this patch I tried to implement #10 and documentation instructions.
It seems to me, that for submodule this feature is too small, so I added links to advanced section of main module.
Also the patch should not contain this:
So I will try to remove it.
It would be great to get feedback. I am a novice, so any feedback is welcome.
Comment #14
damienmckenna@preshetin: That's excellent, thank you!
The only thing that needs to be changed is the 'devel_generate' value, they should both have the 'type' of 'url' as they are both used to contain URLs. But otherwise it looks perfect!
Comment #15
preshetin commented@DamienMcKenna: Thank's for feedback!
Here is updated patch with #14 corrections.
Comment #16
damienmckennaThank you preshetin! Committed!
Comment #18
preshetin commentedDamienMcKenna, thank you for opportunity to participate in. I really enjoyed it.
Comment #21
Kristina Katalinic commented@preshetin can you please tell me which tokens do I have to use to get the rel=prev / rel=next meta tags for my dynamic paginated views? I tried adding token [metatag:prev] and [metatag:next] but that doesn't really result in any output :(
Thanks in advance
Comment #22
damienmckennaLets continue the discussion of using these with meta tags with Views in a new issue: #2502453: How to best use rel=next/prev in Views pages?
Comment #23
osopolarSee also follow-up issue: #2509246: Meta tags from other modules removed even if no values provided by Metatag
Comment #24
Dieter Geiling commented#21
Same problem here! I still get no output.
Can anyone confirm that the rel=next/prev metatags works with views? Wich token are the right one for that?
Anyone know how to resolve the problem?
Thanks for idears
Comment #25
damienmckenna@Dieter: Please continue the discussion in #2502453: How to best use rel=next/prev in Views pages?.
Comment #26
kappaluppa commentedSome FYI info about the prev/next links aside from reference to the modules. This does not relate directly to the modules, but may help others decide if they still need to classify their needs as a module issue:
@Kristina-Katalinic - You might want to take a look at these answers about using next/prev with dynamically loaded content... https://productforums.google.com/forum/?hl=en#!category-topic/webmasters...
"It's fine if your paginated content works like Google Image Search results where pagination doesn't create a new URL (so there's no way you can add rel="next"/"prev"), but instead dynamically loads content. However, you may want to make sure that Googlebot and users without JavaScript can still easily discover your content."
Although that answer is from 2011 I have not found anything else to the contrary. The biggest issue being that the URL does not change. If your URL changes with paginated dynamic content, then that's a different thing.
---------
Sandbox project for link prev/next with views: https://www.drupal.org/sandbox/nlisgo/2361421 plus this patch to make it work with the metatag module: https://www.drupal.org/node/2613216#comment-10554518
I've tested it on a regularly paginated view and it worked for me.
Comment #27
elukacs commentedThe pager_tokens module works well with Metatag. It provides the following tokens: