Skip to content
This repository has been archived by the owner on Feb 11, 2021. It is now read-only.

Engine uses MouseMotion() instead of saving Current and Prev MousePositions() #405

Open
3 of 19 tasks
ch0m5 opened this issue Jun 7, 2020 · 3 comments
Open
3 of 19 tasks
Assignees
Labels
Enhancement New feature or request Low Priority Last in the priority queue.

Comments

@ch0m5
Copy link
Collaborator

ch0m5 commented Jun 7, 2020

Bug Description

Save the current mouse pos and previous mouse pos to calculate the direction vector and get an accurate mouse movement, not SDL_Mouse_Motion which is pretty bad. See this Issue get invalidated for "not being important" when dragging the mouse through the screen x3 times the necessary distance is tedious as f*ck. >:c

Type of Bug

Select the type of bug with and "x" ([x])

  • Visual
  • Physics
  • Audio
  • Particles
  • Resource Management & Save/Load
  • Materials
  • Components
  • Game Objects
  • Camera

UI/UX

  • Scripting
  • Other

Severity

Select the severity of bug affection and mark with "x" ([x])

  • Crash
  • Game stopper/slower
  • Cosmetic

Reproduction

Steps to reproduce the behavior:

  1. Rotate the camera or move it, any interaction that requires moving the camera

Frequency

Select the frequency with which the bug appears and mark it "x" ([x])

  • Always
  • Very Often
  • Usually
  • Few Times
  • Few Times under specific conditions

Conduct

Expected result:

Smooth and precise camera rotation, pans, and movements

Actual result:

Shitty unresponsive and inaccurate camera movements

Screenshots and Illustrations:

fucking_goddamn_mouse_motion

Build

Since 0.0.1

Observations and Additional Information

This is bad and you should feel bad Aitor, you can invalid the Issue but you can't invalid the truth: There's SDL_Mouse_Motion in your code.

@ch0m5 ch0m5 added Bug Something isn't working DO ASAP Resolve as soon as possible! labels Jun 7, 2020
@lucho1
Copy link
Collaborator

lucho1 commented Jun 7, 2020

gonna mark this as invalid due to the unrespect to the programming team >:(

@lucho1 lucho1 added Invalid This doesn't seem right invalidx2 labels Jun 7, 2020
@AitorSimona AitorSimona added Shut Useful to handle angry designers and removed Bug Something isn't working DO ASAP Resolve as soon as possible! Invalid This doesn't seem right invalidx2 labels Jun 7, 2020
@ch0m5
Copy link
Collaborator Author

ch0m5 commented Jun 7, 2020

THERES MOUSE MOTION IN YOUR CODE, AND NOTHING YOU DO BUT FIXING IT WILL CHANGE THAT.

@ch0m5
Copy link
Collaborator Author

ch0m5 commented Jun 7, 2020

YOU CAN CLOSE MY ISSUE BUT YOU CANT CLOSE THE TRUTH.

@ch0m5 ch0m5 changed the title Engine uses MouseMotion() instead of saving Current and Prev MousePositions() (THIS WILL GET INVALIDATED BECAUSE PROGRAMMERS ARE LAZY FUCKERS) Engine uses MouseMotion() instead of saving Current and Prev MousePositions() Jun 7, 2020
@ch0m5 ch0m5 added Enhancement New feature or request Low Priority Last in the priority queue. and removed Shut Useful to handle angry designers labels Jul 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Enhancement New feature or request Low Priority Last in the priority queue.
Projects
None yet
Development

No branches or pull requests

4 participants