GrimRC wrote:

I've created 3 new patches. Below, they are shown diagrammatically like sets.

drag_bug-fix.patch ⊂ highlight.patch ⊂ train_drag.patch

'drag_bug-fix.patch' fixes a bug in the dragging mechanism, whereby changing the mouse-mode with the keyboard (say bulldozer mode) would not cancel the vehicle selection in a depot window properly. A screenshot of the bug, 'drag_bug.png', is attached.

'highlight.patch' contains 'drag_bug-fix.patch'. It highlights selected vehicles (plural!) properly in a train depot window (when dragging with the ctrl key held down); this can almost be considered a second bug-fix. A screenshot demonstrating the improved dragging behaviour is attached as 'highlight.png'.

'train_drag.patch' contains 'highlight.patch', and improves train vehicle dragging even further by changing the mouse cursor sprite to several vehicles when dragging with the ctrl key, rather than just one particular vehicle; this can almost be considered a bug-fix, too. It works by adding an array of sprites and another array of their offsets to struct 'CursorVars'. The current implementation does not remove the old cursor sprite method, but just supplements it. This may not be desirable, as it is basically redundant. I can improve the way this patch works if needed; for example, I could remove the old sprite cursor method and make all cursor-dependent code use the new, proposed method (my sprite array). A screenshot is attached as 'train_drag.png', demonstrating the new, more correct and prettier dragging behaviour.

Summarising, all 3 patches seem bug-free, and the first 2 patches seem clean. The third, biggest and more experimental patch 'train_drag.patch' might need criticism, as it might not quite be how 'pretty train dragging' should be implemented. The patches apply to trunk R6472.

This comment was imported from FlySpray: https://bugs.openttd.org/task/337#comment530

GrimRC wrote:

The patches & screenshots (I accidentally hit 'add comment') attached.

Attachments

• drag_bug-fix.patch (7.63 KiB)
• drag_bug.png (33.48 KiB)
• highlight.patch (9.90 KiB)
• highlight.png (42.48 KiB)
• train_drag.patch (16.83 KiB)
• train_drag.png (34.97 KiB)

This comment was imported from FlySpray: https://bugs.openttd.org/task/337#comment531

SmatZ closed the ticket.

Reason for closing: Implemented

Bug fixed in r12166
Highlight implemented in r12576
Cursor change won't be implemented

This comment was imported from FlySpray: https://bugs.openttd.org/task/337