UFO: Alien Invasion Issue Tracker
UFO: Alien Invasion
Go to the previous open issue
Go to the previous issue (open or closed)
star_faded.png
Please log in to bookmark issues
icon_project.png UFO: Alien Invasion / Closed Feature request #3652 Scrolling in battlescape (with patch)
action_vote_minus_faded.png
0
Votes
action_vote_plus_faded.png
Go to the next issue (open or closed)
Go to the next open issue
This issue has been closed with status "Closed" and resolution "Not determined".
Issue basics
  • Type of issue
    Feature request
  • Category
    User interface
  • Targetted for
    Not determined
  • Status
    Closed
  • Priority
    3. Normal
User pain
  • Type of bug
    Not triaged
  • Likelihood
    Not triaged
  • Effect
    Not triaged
Affected by this issue (0)
There are no items
People involved
Times and dates
  • Posted at
  • Last updated
Issue details
  • Complexity
    Not determined
Attachments (0)
There is nothing attached to this issue
Duplicate issues (0)
This issue does not have any duplicates
Description
[http://sourceforge.net/p/ufoai/feature-requests/167 Item 167] imported from sourceforge.net tracker on 2013-01-28 20:14:20

Good evening.

I didn't like the scrolling in the battlescape, so I changed it. I thought:

1. The camera movement should slow down quicker than it speeds up. This means it actually stops where you meant it to rather than a screen later. I was having trouble with noticing that the scrolling was slowing down. This meant that I would move my mouse away from the edge of the screen, not notice that it is slowing down, and hit the other edge of the screen, causing it to scroll the other way again. Having it slow down quicker prevents this.
2. The scroll speed should be adjusted according to the zoom level, so that it isn't just a blur at high zoom, or annoyingly slow when zoomed out.

Very simple changes. Here is a patch:

Index: src/client/cl_input.c
===================================================================
--- src/client/cl_input.c (revision 6358)
+++ src/client/cl_input.c (working copy)
@@ -917,6 +917,9 @@
(cl_cammoveaccel->value >
MIN_CAMMOVE_ACCEL) ? ((cl_cammoveaccel->value < MAX_CAMMOVE_ACCEL) ? cl_cammoveaccel->value : MAX_CAMMOVE_ACCEL) : MIN_CAMMOVE_ACCEL;

+ movespeed /= cl.cam.zoom;
+ moveaccel /= cl.cam.zoom;
+
/* calculate camera omega */
/* stop acceleration */
frac = cls.frametime * rotaccel;
@@ -979,6 +982,9 @@
/* calculate camera speed */
/* stop acceleration */
frac = cls.frametime * moveaccel;
+ if (!(CL_GetKeyMouseState(STATE_FORWARD) || CL_GetKeyMouseState(STATE_RIGHT))) {
+ frac *= 3;
+ }
if (VectorLength(cl.cam.speed) > frac) {
VectorNormalize2(cl.cam.speed, delta);
VectorMA(cl.cam.speed, -frac, delta, cl.cam.speed);

===== Comments Ported from Sourceforge =====

====== mnw21cam (2007-02-18 23:43:39) ======

Logged In: YES
user_id=1661734
Originator: YES

Ooo. That patch really got splatted by the forum. Here, I&#039;ll upload a file instead.
File Added: patch1
====== mnw21cam (2007-02-19 12:40:40) ======

Logged In: YES
user_id=1661734
Originator: YES

Here&#039;s a better patch.
File Added: patch2
====== voovoos (2007-03-05 13:13:49) ======

Logged In: YES
user_id=194458
Originator: NO

Patch applied to trunk in r6620, thanks.
Todos (0 / 0)
Issue created
footer_logo.png The Bug Genie 4.3.1 | Support | Feedback spinning_16.gif