MantisBT - Doomseeker
View Issue Details
0003669DoomseekerUIpublic2019-06-23 10:322019-07-30 10:13
Zalewa 
Zalewa 
nonetweakN/A
closedfixed 
1.2 
1.31.3 
0003669: Unable to reorder file search paths
It should be possible to reorder the file paths (WAD search paths) in the configuration box by drag'n'drop of the path rows.
No tags attached.
Issue History
2019-06-23 10:32ZalewaNew Issue
2019-06-23 12:06WubTheCaptainStatusnew => acknowledged
2019-06-23 12:19WubTheCaptainNote Added: 0020823
2019-06-23 12:23WubTheCaptainNote Edited: 0020823bug_revision_view_page.php?bugnote_id=20823#r12687
2019-06-23 12:24WubTheCaptainNote Edited: 0020823bug_revision_view_page.php?bugnote_id=20823#r12688
2019-06-23 21:43ZalewaAssigned To => Zalewa
2019-06-23 21:43ZalewaStatusacknowledged => assigned
2019-06-24 17:48ZalewaNote Added: 0020827
2019-06-24 17:48ZalewaStatusassigned => needs review
2019-06-24 17:48ZalewaSeverityfeature => tweak
2019-06-24 17:48ZalewaTarget Version => 1.3
2019-06-24 17:48ZalewaSummaryAllow to reorder file paths => Unable to reorder file paths
2019-06-24 17:48ZalewaSummaryUnable to reorder file paths => Unable to reorder file search paths
2019-07-02 21:27ZalewaStatusneeds review => needs testing
2019-07-12 12:08Pol MNote Added: 0020870
2019-07-12 12:08Pol MStatusneeds testing => resolved
2019-07-12 12:08Pol MFixed in Version => 1.3
2019-07-12 12:08Pol MResolutionopen => fixed
2019-07-30 10:13WubTheCaptainStatusresolved => closed

Notes
(0020823)
WubTheCaptain   
2019-06-23 12:19   
(edited on: 2019-06-23 12:24)
I'm assuming your intention is to say it's not currently possible to reorder (which it's not IIRC), but you'd like to have it possible. 😛

For clarification in future, the issue's summary could be summarized into an actionable issue such as: "unable to reorder WAD search paths". If you think of it that way, it might be a tweak to an existing functionality.

The description would then describe what the current implementation is, what is the issue with it. I personally sometimes keep proposals how to fix things to notes of the issue.

A little bit more effort to keep this tracker clean every day. 🙂

(0020827)
Zalewa   
2019-06-24 17:48   
I hoped that this would be trivial, but instead it was complicated. Let the commmit speak for itself:https://bitbucket.org/Doomseeker/doomseeker/commits/4b797b0b3e02d88237318ee8afe033aff7d5655c [^]

Here's also a "minimal" PoC I prepared when I was figuring out how to do this

As far as your remarks go, Wub, I suppose we just have a different way of thinking about this. I already knew what needed to be done hence the brevity. But your remarks have merit, of course, and I'll take them into consideration in the future.
(0020870)
Pol M   
2019-07-12 12:08   
Tested. Everything works as expected, and no issues emerged after testing most combinations. Resolving.