-
-
Notifications
You must be signed in to change notification settings - Fork 227
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Orders page sorting removed but infrastructure remains #6619
Comments
Sort by what? |
If you look at the code I referenced, you'll see the options. I'm not sure any of them are terribly useful though. It looks like it was modeled after the customer page sorting, but even back as far as 1.3.8, it wasn't fully implemented. |
Yes, I see. Personally I've never missed not having a sort. |
When? |
Sometimes expanding the filtering options is more advantageous than a "sort". Granted, we do have a search. |
I thought it had been but I looked waaaaay back and couldn't find it. So I guess this code has been there, unused, forever? Not strong on this, it's just a bit confusing if you're reading the code. |
The orders page sorting was removed but orders.php lines 1252-1279 still set the unused variable
$disp_order
.It might be nice to have a sort dropdown next to the order status filter. Or we could just remove this code.
The text was updated successfully, but these errors were encountered: