Skip to content
This repository has been archived by the owner on Apr 17, 2022. It is now read-only.

Mouse cursor shape ignores radar #3283

Closed
wzdev-ci opened this issue Mar 13, 2012 · 7 comments
Closed

Mouse cursor shape ignores radar #3283

wzdev-ci opened this issue Mar 13, 2012 · 7 comments

Comments

@wzdev-ci
Copy link
Contributor

resolution_fixed type_bug | by Cyp


Noticed this when playing a multiplayer 1v1 in β5:

Seems the mouse cursor is whatever it would have been, if the radar wasn't there. If the cursor is a big "X" over certain terrain, then moving the viewport so the radar covers it, the cursor still shows a big "X". (Which is confusing.)

Should either show an arrow, or show what the cursor would be over the corresponding terrain where the radar refers to.


Issue migrated from trac:3283 at 2022-04-16 09:35:48 -0700

@wzdev-ci
Copy link
Contributor Author

Cyp commented


Also, reported by Berg, clicking on the mini-map makes it start scrolling to the clicked location, but it doesn't finish scrolling.

Bisect blames both problems on 6e0c364c99f8c85b5be96087059db7f627c2d3bc.

@wzdev-ci
Copy link
Contributor Author

cybersphinx changed status from new to closed

@wzdev-ci
Copy link
Contributor Author

cybersphinx changed resolution from `` to fixed

@wzdev-ci
Copy link
Contributor Author

cybersphinx commented


Move radar handling from processInput() to processRadarInput().

Fixes #3283.

  • Changeset: [/changeset/91614562b0969c47ffed7020caa1e16c8ede5823 91614562b0969c47ffed7020caa1e16c8ede5823]
  • URL: [91614562]b0969c47ffed7020caa1e16c8ede5823

@wzdev-ci
Copy link
Contributor Author

cybersphinx commented


Move radar handling from processInput() to processRadarInput().

Fixes #3283.

  • Changeset: [/changeset/91614562b0969c47ffed7020caa1e16c8ede5823 91614562b0969c47ffed7020caa1e16c8ede5823]
  • URL: [91614562]b0969c47ffed7020caa1e16c8ede5823

2 similar comments
@wzdev-ci
Copy link
Contributor Author

cybersphinx commented


Move radar handling from processInput() to processRadarInput().

Fixes #3283.

  • Changeset: [/changeset/91614562b0969c47ffed7020caa1e16c8ede5823 91614562b0969c47ffed7020caa1e16c8ede5823]
  • URL: [91614562]b0969c47ffed7020caa1e16c8ede5823

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Aug 4, 2012

cybersphinx commented


Move radar handling from processInput() to processRadarInput().

Fixes #3283.

  • Changeset: [/changeset/91614562b0969c47ffed7020caa1e16c8ede5823 91614562b0969c47ffed7020caa1e16c8ede5823]
  • URL: [91614562]b0969c47ffed7020caa1e16c8ede5823

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant