Download & Extend
IMCE » Issues

Out of alphabetical order

Posted by eyecon on March 18, 2009 at 1:43am
Project:IMCE
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

Works like a champ with both BUE and WYSIWYG but the files are out of order. Small sample in attached image.

AttachmentSize
imce.jpeg18.46 KB
Login or register to post comments

Comments

#1

Posted by ufku on March 20, 2009 at 10:23pm
Category:bug report» support request

this is a theme issue.
You can style the filename cells using
#imce-content table.files td.name {
text-align: left !important;
}

#2

Posted by eyecon on March 21, 2009 at 1:16am

The sort order is a theme issue?

#3

Posted by ufku on March 21, 2009 at 1:43am

So it was the sort order ha.

Is the behavior always observed? For any directory list?
Tried logging out, clearing cookies, logging in?

#4

Posted by eyecon on March 21, 2009 at 1:54am

It only occurs on the server. Local directories list perfectly. This never happened on the 5.x branch. I cleared the cache and the cookies - even trying a different browser (Opera v Firefox). The sort is all over the place.

#5

Posted by kakor on April 6, 2009 at 2:52pm

I can confirm this behaviour, I have the same problem. Attaching a screen capture.

AttachmentSize
imce_file_order.png 89.72 KB

#6

Posted by samtherobot on April 16, 2009 at 3:47pm

Same issue here.

#7

Posted by ufku on April 30, 2009 at 12:32pm
Status:active» needs review

Can you please review the patch?
It changes line 151 of js/imce_extras.js
from
imce.sortStrAsc = function(a, b) {return a.charAt(0).toLowerCase() < b.charAt(0).toLowerCase() ? -1 : b < a;};
to
imce.sortStrAsc = function(a, b) {return a.toLowerCase() < b.toLowerCase() ? -1 : 1;};

AttachmentSize
imce_extras.js_.patch 747 bytes

#8

Posted by nathanmcginty on June 11, 2009 at 7:09pm

exact same thing. i'll try the patch.

#9

Posted by nathanmcginty on June 11, 2009 at 7:18pm

nope. patch applied fine. didn't do anything i could tell.

#10

Posted by nathanmcginty on June 11, 2009 at 7:29pm

holy cr**.

it worked. sorry - had to flush the cache.

thanks!

#11

Posted by ufku on June 24, 2009 at 12:19pm
Version:6.x-1.2» 6.x-1.x-dev
Category:support request» bug report
Status:needs review» fixed

committed. thank you all.

#12

Posted by System Message on July 8, 2009 at 12:20pm
Status:fixed» closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

#13

Posted by Jorrit on September 16, 2009 at 1:42pm

Is a release containing this fix coming soon?

nobody click here