- Issue was unassigned.
- Status changed to Needs review
almost 3 years ago 7:11am 20 January 2023 - Status changed to Fixed
almost 3 years ago 10:22am 20 January 2023 Looking for the README.md template documentation: https://www.drupal.org/docs/develop/managing-a-drupalorg-theme-module-or... →
I'm seeing some errors of the standards
1) Table of contents:
This one is the first one in the README.md file, and after that there is an "Introduction" with the description. For example in the Search API README that is used as example in the documentation: https://git.drupalcode.org/project/search_api/-/raw/8.x-1.x/README.md2) Requirements
When there is no requirement the documentations says that the text should be## Requirements This module requires no modules outside of Drupal core.3) Configuration
As mentioned at #3335089-7: hook_help function missing in .module file → is optional but I think we can put an example about "how to use" to help the cms editors- Status changed to Needs work
almost 3 years ago 2:48am 23 January 2023 - @renatog opened merge request.
- Status changed to Needs review
almost 3 years ago 2:54am 23 January 2023 MR with the improvements reported in the #7: https://git.drupalcode.org/project/email_contact/-/merge_requests/11
- 🇭🇺Hungary nagy.balint
Actually I think, the previous is the correct order, see attached image.
Actually I think, the previous is the correct order, see attached image.
Yeah, you are right, good catch
I'll send a comment in the documentation explaining that, that the README linked as "example" is using an different order than have in the documentation and it can confuse the users reading the doc instead of helping
I just updated the MR, keeping this order as was before and I'm just including the "How to use" steps to help the users: https://git.drupalcode.org/project/email_contact/-/merge_requests/11
Thank you so much
- 🇭🇺Hungary nagy.balint
Hi!
There seem to be some issues in the patch. There are multiple 1s.
Attached image.
- 🇮🇳India jay jangid Jaipur
Thanks @nagy.balint for notify me.
Fixed issue , please review. - Status changed to RTBC
almost 3 years ago 7:12pm 24 January 2023 @nagy.balint just to let you know, this is the syntax for Markdown with "numbered lists" you know?
Look that on this online editor https://markdownlivepreview.com/
In the left space paste this code:## HOW TO USE Since there are no extra settings, so to see that working you can follow these steps: 1. Add the `"email"` field anywhere you want, for example: `"Basic Page"` 1. Go to Manage Display at: Configuration » Structure » Content types » < CONTENT-TYPE > Manage display 1. In the `"Format"` column you will see new options, for example: 1. Email contact inline 1. Email contact link 1. You can save that and see it working in the front-endThe resul will be with the correct ordinal numbers
Example on screenshot:
As I can see the MD syntax support both
1. item 1. item 1. itemand
1. item 2. item 3. itemBoth will render as ordinal in the FE
Honestly I think the first one is better because if we have a lot of items, we don't need to put all of them following the order, 1,2,3,4 etc
Just put 1,1,1,1 the MK will apply that automatically in the FE
But since in our case we have just few items I think this last update on MR is also working fine, so for this reason I put as RTBC
-
nagy.balint →
committed d9ba77e5 on 2.x authored by
RenatoG →
Issue #3335082 by RenatoG, Jay Jangid, nagy.balint: Added How To Use to...
-
nagy.balint →
committed d9ba77e5 on 2.x authored by
RenatoG →
- Status changed to Fixed
almost 3 years ago 11:26am 25 January 2023 Automatically closed - issue fixed for 2 weeks with no activity.