Out of alphabetical order

eyecon - March 18, 2009 - 01:43
Project:IMCE
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

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

#1

ufku - March 20, 2009 - 22:23
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

eyecon - March 21, 2009 - 01:16

The sort order is a theme issue?

#3

ufku - March 21, 2009 - 01:43

So it was the sort order ha.

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

#4

eyecon - March 21, 2009 - 01:54

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

kakor - April 6, 2009 - 14:52

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

AttachmentSize
imce_file_order.png 89.72 KB

#6

samtherobot - April 16, 2009 - 15:47

Same issue here.

#7

ufku - April 30, 2009 - 12:32
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

nathanmcginty - June 11, 2009 - 19:09

exact same thing. i'll try the patch.

#9

nathanmcginty - June 11, 2009 - 19:18

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

#10

nathanmcginty - June 11, 2009 - 19:29

holy cr**.

it worked. sorry - had to flush the cache.

thanks!

#11

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

committed. thank you all.

#12

System Message - July 8, 2009 - 12:20
Status:fixed» closed

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

#13

Jorrit - September 16, 2009 - 13:42

Is a release containing this fix coming soon?

 
 

Drupal is a registered trademark of Dries Buytaert.