-
-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added the ability to comment on a map #42
Conversation
Added comments support
Hi @YosoraLife, Thank you for submitting this, I actually think allowing comments for Maps makes complete sense and would make a nice addition to the plugin :) My only concern with this change is that by enabling this capability by default, some users with existing Maps might not realise and may unknowingly start publishing Maps with comments enabled by default (depending on the Adding a Waymark Setting to enable/disable comments would be one way around this. Do you have any thoughts? Cheers, Joe |
Hi Joe, I understand where you're coming from. And I agree that code changes should not affect already published maps. But this would only affect new maps. And then again, for newly published maps, this would only affect those who have enabled commenting on their site. I have a bit of mixed feelings about an extra setting. On the one hand, you have already built in many settings (which I have also gratefully made use of) and there will undoubtedly be people who want comments on their posts but not comments on the maps (although they can also disable comments per map). On the other hand, it is an extra setting that ultimately needs to be maintained. And it's just a small adjustment, but every little bit adds up. It would be a shame if it all became so much that the plugin could no longer be maintained. Cheers, |
Hi Remco, Thank you for your thoughts :) I have given this some more thought and I am definitely leaning towards enabling this by default. While I don't love the idea of creating a scenario where the user might unknowingly create a comments-enabled Map, I think the benefits of tapping into the WordPress post commenting system makes a lot of sense and might outweigh those drawbacks. Cheers, Joe |
Hi Joe, of course, your welcome :) Would it also be an idea to add 'excerpt' to the supports array? Because now in a search query I get this as a result: While its text from the actual page it's not really in context. And it would be nice to have some text that actually makes some sense when you read it (and to have some control on what's displayed as excerpt). And unlike with the comment this doesn't affect anyone if there not using it, it just another possibility than can be used if you want to. |
Hi @YosoraLife,
I am not apposed to this, but I am a little weary of overwhelming the Map Create/Edit interface by enabling too many of the Post capabilities. I think instead of a PR this might be better broken up into a separate Issue for further discussion. I am traveling at the moment and will give these some more time when I return. Thanks again! Joe |
No problem, i'll open a separate issue. Enjoy your travels ;) |
Enable or disable various WordPress features of the Map Custom Post Type, including the new ability to enable comments and excerpt integration. This Setting is Available in Settings > Advanced > Post Type. Thanks to [YosoraLife](https://github.com/YosoraLife) for requesting this ([here](#42) and [here](#44)).
Hi @YosoraLife, I have released version 1.2.0 of the plugin which includes the ability to enable comment support. Please do let me know if you encounter any issues! Thanks again for the suggestion. Joe |
It only a small change, but i think more people could benefit from this.
I am using Waymark to display motorcycle routes on my websites. And i thought it would be nice if people could comment/discuss those routes. So since Waymaps is a custom post type, I added the default wordpress ability to comment on a map.