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
Implement natural sort for thumbnails in MediaPage #2
Conversation
This addresses bug #717960, "Natural sorting of photo titles" I grafted NaturalSort.vala from my own https://github.com/tobiatesan/NaturalCmp Please find the ticket at: https://bugzilla.gnome.org/show_bug.cgi?id=717960
13b1605
to
5bb6c30
Compare
Just to be really sure. Especially to rule out weird things happening with non-arabic numerals.
The experienced behaviour is in fact correct, since "十" nor any other Chinese numeral is considered a numeral by isdigit(), which only cares about alternate /glyphs/ for the plain old decimal/arabic system. The Chinese numbering system has its own composition rules (e.g. "2" is written "二" - i.e. two horizontal strokes, "1" is written "一" - i.e. single horizontal stroke, but "21" is not written "二一").
The code seems merged. Can you please close this PR? I cannot. |
Thank you for contributing to shotwell! shotwell uses Bugzilla for code review. If you have never contributed to GNOME before make sure you have read the Otherwise please visit |
Addresses bug #717960.
(But beware: I'm still getting my feet wet with Vala - coming from C++ - and Shotwell, so it can be likely ugly and/or not
very Vala-idiomatic. Please do voice any concerns.)