Reviewing merge request #12: Added an alternative meta list syntax.

Hi Waylan,

I would like to be able to use the "explicit" syntax. I see no problems with supporting it, given that its current behaviour is not useful.

Cheers,

David

Commits that would be merged:

Version 1
  • Version 1
  • 9fa60ed
  • cf76cc8
  • Added an alternative meta list syntax.

Showing 9fa60ed-cf76cc8

Comments

I guess I'm not following what you are trying to do here.

Do you want your meta-data to look like this:

key1: line 1 of key 1
key1: line 2 of key 1  
key2: line 1 of key 2

That must be what you mean by “explicit,” but it seems strange to me, so maybe I'm missing something obvious.

Hi Waylan,

I assumed that my commit message would be visible in the merge request, but apparently only diffs are displayed.

I'm working on category indexes for Mango. At the moment, category listings such as mango.io/docs/ are rather rudimentary: the category name (“Docs” in this case) is derived from the name of the directory containing the documents, and the category’s pages are simply listed alphabetically by title.

I seek to remove the coupling between directory name and category name (no such coupling exists between file name and document title), and to provide control over the order in which a category’s documents are listed.

page: setup
page: settings  
page: syntax  
page: data-types  
page: templates  


Documents  
=========  

Mango is undergoing rapid development; the documentation is struggling to keep up.

Sure, I could use the existing syntax but I find it aesthetically pleasing to repeat the key. The question is whether allowing this alternative syntax would cause anyone problems. The answer, I believe, is no, since listing a key multiple times does not currently produce useful output.

One of my goals with Mango is to introduce as little Mango-specific syntax as possible, which is why I'm keen to see this change make it into mainline. I hope I've done a better job this time of explaining the reasoning behind my request. :)

David

→ State changed from Open to Merged

Sorry for the delay. I finally got around to merging this.

Hi Waylan,

You may be interested to see a recent commit which mentions this change. Thanks again for merging it. :)

David

Add a new comment:

Login or create an account to post a comment

How to apply this merge request to your repository