[PR #387] [MERGED] Fix sorting by date and size #969

New issue

Closed

opened 2026-03-02 16:02:33 +03:00 by kerem · 0 comments

kerem commented

2026-03-02 16:02:33 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/prasathmani/tinyfilemanager/pull/387
Author: @safraja
Created: 6/27/2020
Status: ✅ Merged
Merged: 7/20/2020
Merged by: @prasathmani

Base: master ← Head: patch-3

📝 Commits (1)

89f017f Fix sorting by date and size

📊 Changes

1 file changed (+20 additions, -8 deletions)

View changed files

📝 tinyfilemanager.php (+20 -8)

📄 Description

Based on suggestions in Sorting by size and time is wrong
but uses data-sort attribute (from DataTable wiki) instead of invisible characters, and also distinguishes files and folders, so they stay grouped even after sort.
(I am sorry for my bad English.)

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/prasathmani/tinyfilemanager/pull/387 **Author:** [@safraja](https://github.com/safraja) **Created:** 6/27/2020 **Status:** ✅ Merged **Merged:** 7/20/2020 **Merged by:** [@prasathmani](https://github.com/prasathmani) **Base:** `master` ← **Head:** `patch-3` --- ### 📝 Commits (1) - [`89f017f`](https://github.com/prasathmani/tinyfilemanager/commit/89f017fe83394e6c8cc635884d24ff1f857d5a5d) Fix sorting by date and size ### 📊 Changes **1 file changed** (+20 additions, -8 deletions) <details> <summary>View changed files</summary> 📝 `tinyfilemanager.php` (+20 -8) </details> ### 📄 Description Based on suggestions in [Sorting by size and time is wrong](https://github.com/prasathmani/tinyfilemanager/issues/197) but uses data-sort attribute (from [DataTable wiki](https://datatables.net/examples/advanced_init/html5-data-attributes.html)) instead of invisible characters, and also distinguishes files and folders, so they stay grouped even after sort. (I am sorry for my bad English.) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>