Page MenuHomeWildfire Games

add num pad hotkey equivalents
ClosedPublic

Authored by Nescio on Oct 9 2020, 3:31 PM.

Details

Summary

As pointed out on the forums, pressing / to select idle units only works with the / from the alphanumerical section, but not with the / from the numeric section (numpad).
Likewise, for selection groups, pressing 0 ... 9 from the numeric section is ignored.
This patch solves that by adding equivalents (for comparison, both plusses already work for zooming in.)

Test Plan

Verify it works, agree this is an improvement.

Event Timeline

Nescio created this revision.Oct 9 2020, 3:31 PM
Nescio requested review of this revision.Oct 9 2020, 3:36 PM
Nescio added inline comments.Oct 9 2020, 3:41 PM
binaries/data/config/default.cfg
241–251

Perhaps this should be alphabetized?

Freagarach accepted this revision.Oct 27 2020, 8:23 PM
Freagarach added a subscriber: Freagarach.

Nice addition.

binaries/data/config/default.cfg
241–251

I assume you're talking about the whole file? Yeah, but not sure it's worth the hassle ;)

This revision is now accepted and ready to land.Oct 27 2020, 8:23 PM
Freagarach removed a reviewer: Restricted Owners Package.Oct 27 2020, 8:24 PM
vladislavbelov added inline comments.
binaries/data/config/default.cfg
190

Why Underscore? It's not minus.

241–251

Maybe we have to avoid that kind of alignment. It breaks SVN history for useless (imo) alignment.

Freagarach added inline comments.Oct 27 2020, 8:35 PM
binaries/data/config/default.cfg
190

Same reason as Equals above, because people won't want to use the shift to zoom. As such it is consistent.

241–251

Fully agree.

Nescio added inline comments.Oct 28 2020, 9:25 AM
binaries/data/config/default.cfg
241–251

Lines 248 and 249 needed extra space. I did the other lines of this subsection too, for consistency; see other parts of this file for comparison.

Freagarach added inline comments.Oct 28 2020, 5:21 PM
binaries/data/config/default.cfg
241–251

Yes and @vladislavbelov and I argue that we should not use that kind of alignment (in the future).

Nescio added inline comments.Oct 28 2020, 6:07 PM
binaries/data/config/default.cfg
241–251

Yes, I understand, but that would be introducing inconsistency. Just have a look at other sections, e.g. lines 185 to 210 or 286 to 321.

wraitii accepted this revision.Oct 31 2020, 3:30 PM
This revision was automatically updated to reflect the committed changes.
Freagarach added inline comments.Nov 3 2020, 7:18 PM
binaries/data/config/default.cfg
241–251

Using the proposed changes due to file consistency. If someone decides to clean this file in the future they might take this discussion in account.

vladislavbelov added inline comments.Nov 7 2020, 10:36 AM
binaries/data/config/default.cfg
241–251

So, you see that the file already contains inconsistent lines. So my point is to move from the one consistency (with useless alignment) to another (without between-line alignment).