Modifier keys to vary the level of highlighting #136
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As you probably know some level editors show the relationship between linedefs and the tagged sector with a line connecting them. Since you may have already made a design decision to not do that I wanted to share an idea about how it could be opt-in, and how it could be patterned after 3D camera movement.
Since shift results in less camera movement, and control more, I decided to apply that to highlighting as well.
The shift key turns off all highlighting. This can be helpful when the user wants to the the original color of a highlighted or selected item. It can also be helpful to make the highlighted or selected items flash, to make them easier to see.
The control key draws additional highlighting. In particular it draws a line between the tagged items.
I consider this to be a rough draft with a few "TODO"s in it. There are things that I need to understand better, like the way event handling should work. It's more proof of concept.
Here's an example of what it looks like with the control key pressed. The vertical linedef on the right side is a switch that activates the two sectors.
Also, the whitespace change to
m_bitvec_test.cpp
is because the C++ compiler on one of my Linux systems didn't like the transition from tabs to spaces.