[PR #444] [MERGED] Sort instances by status then name directly in template #659

New issue

Closed

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

kerem commented

2026-02-27 16:40:03 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/retspen/webvirtmgr/pull/444
Author: @daniviga
Created: 10/1/2014
Status: ✅ Merged
Merged: 10/2/2014
Merged by: @retspen

Base: master ← Head: upstream

📝 Commits (1)

35db6f4 Sort instances by status then name directly in template

📊 Changes

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

View changed files

📝 templates/instances.html (+1 -1)

📄 Description

This PR completes work on #439 adding the default sort (by status then name) in the instances.html template. In some cases (Firefox on slow machine with lot of instances), before this, table is sorted after hundreds of milliseconds.; with this fix instances are already sorted in the right order.

_{🔄 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/retspen/webvirtmgr/pull/444 **Author:** [@daniviga](https://github.com/daniviga) **Created:** 10/1/2014 **Status:** ✅ Merged **Merged:** 10/2/2014 **Merged by:** [@retspen](https://github.com/retspen) **Base:** `master` ← **Head:** `upstream` --- ### 📝 Commits (1) - [`35db6f4`](https://github.com/retspen/webvirtmgr/commit/35db6f4a2b9ee208b6d58af04035bb18bd0412d7) Sort instances by status then name directly in template ### 📊 Changes **1 file changed** (+1 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `templates/instances.html` (+1 -1) </details> ### 📄 Description This PR completes work on #439 adding the default sort (by status then name) in the `instances.html` template. In some cases (Firefox on slow machine with lot of instances), before this, table is sorted after hundreds of milliseconds.; with this fix instances are already sorted in the right order. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>