dcostinett / and-bookworm

Automatically exported from code.google.com/p/and-bookworm
0 stars 0 forks source link

Sort should be case insensitive #84

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Have more than one title begin with the same letter.
2. Make at least one title have a uppercase first letter.
3. Make another title have a lowercase first letter.
4. Order by title.
5. Uppercase titles come before lowercase titles.  That is, "Azzz" is before 
"aaaa".

What is the expected output? What do you see instead?

Convention is that ordering is case insensitive.

What version of the product are you using? On what operating system?

Android 2.1; BookWorm 1.0.14

Please provide any additional information below.

Attaching a patch file for suggested solution; very minimal testing.  If you 
have an alternative patch and would like me to test, please let me know.

Original issue reported on code.google.com by RobGre...@gmail.com on 13 Jul 2010 at 1:53

Attachments:

GoogleCodeExporter commented 9 years ago
I apologize, that title should be "Sort should be case insensitive".  I don't 
see an edit, so adding as comment.

Original comment by RobGre...@gmail.com on 13 Jul 2010 at 9:07

GoogleCodeExporter commented 9 years ago
Thanks for the report and the patch (awesome)! I will check this out soon. 
Sometimes I like the case sensitive sort, but you are correct the default 
should be case insensitive. Maybe I'll add an option (or just another sort 
metric).

Original comment by charlie....@gmail.com on 14 Jul 2010 at 3:31

GoogleCodeExporter commented 9 years ago

Original comment by charlie....@gmail.com on 14 Jul 2010 at 3:31

GoogleCodeExporter commented 9 years ago
In SVN.

Original comment by charlie....@gmail.com on 25 Jul 2010 at 4:19

GoogleCodeExporter commented 9 years ago

Original comment by charlie....@gmail.com on 25 Jul 2010 at 6:37