-
Notifications
You must be signed in to change notification settings - Fork 7.7k
[Core][Untitled files][Win only]: Incorrect sort order for untitled files. #4409
Comments
Reviewed - related to #4332. Move both to backlog. |
FBNC to @julieyuan |
@RaymondLim I checked this issue in build 0.28.0.8416. Pull request #4463 indeed fixed issue #4332. But another issue occurs. Please refer to snapshot: This issue also reproduces on MAC. Untitled-25.js is between Untitled-2.js and Untitled-3.js. I don't think it is right behavior. Please help to fix it .Thanks. |
Looks like we need another sorting when filenames end with numbers. Do you want to try it by breaking up the trailing numbers and do a secondary sorting if the filenames w/o trailing numbers are the same? Otherwise, we may have to close this one as "move to backlog." |
Yes. But I can work on it next week probably. |
Thanks. If you can get it done by next Tuesday, then we can keep the milestone. Otherwise, I have to move it out to next sprint. |
Issue #4409: Incorrect sort order for untitled files
FBNC to @julieyuan again. You can verify the remaining issue with b8496. |
Steps:
Result:
The sort order for these untitled files is different with OS sort order. On Brackets, its files order is: Untitled-1.js,Untitled-2.js,Untitled-3.js, Untitled.js, but its untitled folders order is : Untitled,Untitled-1,Untitled-2,Untitled-3. And OS sort order is: Untitled.js, Untitled-1.js,Untitled-2.js,Untitled-3.js.
Expected:
The sort order for untitled files should follow OS sort oder and untitled folders order.
ENV: Win7x64 ENU
Build: 0.27.0.8255
Notes:
On Mac, its sort order for untitled files is same with Win: Untitled-1.js,Untitled-2.js,Untitled-3.js, Untitled.js. But its OS sort order has same behavior. I think maybe MAC and Win follows different sort order rules.
Snapshots:
![incorrect sort by name](https://camo.githubusercontent.com/9e6cb0ff1ffa33a81cdad3b5964dbc391140a60c6efc79cc6167229da4cb80fe/68747470733a2f2f662e636c6f75642e6769746875622e636f6d2f6173736574732f343235313536332f3737333430372f36643466353830382d653933382d313165322d386539312d3837643630373839316562382e706e67)
Please refer to snapshot for details:
The text was updated successfully, but these errors were encountered: