Skip to content

Conversation

andineck
Copy link
Contributor

@andineck andineck commented Jun 2, 2016

No description provided.

@coveralls
Copy link

coveralls commented Jun 2, 2016

Coverage Status

Coverage increased (+0.3%) to 93.478% when pulling ef400a9 on intesso:master into 9df21ba on imfly:master.

@coveralls
Copy link

coveralls commented Jun 2, 2016

Coverage Status

Coverage increased (+0.3%) to 93.478% when pulling be8358a on intesso:master into 9df21ba on imfly:master.

@coveralls
Copy link

coveralls commented Jun 3, 2016

Coverage Status

Coverage decreased (-1.04%) to 92.143% when pulling 2568fdb on intesso:master into 9df21ba on imfly:master.

@imfly
Copy link
Owner

imfly commented Jun 3, 2016

andineck, Thank you very much.

The sort directory entries is better to character languages, like english; but is not to the grapheme languages, like Chinese or Japanese. So, I give an option for users by adding "num-" prefix. Do you have a better idea?

@imfly imfly merged commit dfa7e9c into imfly:master Jun 4, 2016
@andineck
Copy link
Contributor Author

andineck commented Jun 8, 2016

@imfly Thank you for merging the PR. Sorry for my late answer. I think in general it is good to have the directories listed before the files. Is there already an option to switch off the "num-" prefix auto removing from the Links? I have several files that have "-" in the name and the first part get's removed automatically, even though it's not a number. If it is possible to switch off this behaviour, that would also be helpful. What do you think?

@imfly
Copy link
Owner

imfly commented Jun 9, 2016

@andineck Yes,if the prefix is "num-" or "azAZ-", the prefix will be uesed to order defaultly, #6

I'm going to switch off this behaviour defaultly for character languages, like english. And users can add the "num-" prefix for the grapheme languages, like Chinese or Japanese.

imfly added a commit that referenced this pull request Jun 10, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants