Skip to content

Commit b0050a3

Browse files
committed
fix(core): dot absolute positioning bug fix
1 parent 23ebd78 commit b0050a3

File tree

3 files changed

+6
-21
lines changed

3 files changed

+6
-21
lines changed

CHANGELOG.md

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,9 @@
11
## [1.3.1](https://github.com/shivanshkc/graviton-web-client/compare/v1.3.0...v1.3.1) (2022-04-07)
22

3-
43
### Bug Fixes
54

6-
* **core:** bigget dot creation fix on mobile devices ([2179611](https://github.com/shivanshkc/graviton-web-client/commit/2179611756190fb3140193ef512e8c865f375e72))
7-
* **core:** dots being created do not participate in simulation ([4aa1587](https://github.com/shivanshkc/graviton-web-client/commit/4aa1587f136aebd6b6c2480e69def08e531fa242))
5+
- **core:** bigget dot creation fix on mobile devices ([2179611](https://github.com/shivanshkc/graviton-web-client/commit/2179611756190fb3140193ef512e8c865f375e72))
6+
- **core:** dots being created do not participate in simulation ([4aa1587](https://github.com/shivanshkc/graviton-web-client/commit/4aa1587f136aebd6b6c2480e69def08e531fa242))
87

98
# [1.3.0](https://github.com/shivanshkc/graviton-web-client/compare/v1.2.1...v1.3.0) (2022-04-07)
109

src/pages/home/home.component.ts

Lines changed: 2 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -79,14 +79,11 @@ export class HomeComponent implements AfterViewInit, OnDestroy {
7979
const posX = event instanceof MouseEvent ? event.clientX : event.touches[0].clientX;
8080
const posY = event instanceof MouseEvent ? event.clientY : event.touches[0].clientY;
8181

82-
// This position moves the center of the dot to the click position.
83-
const centeredPos = new Vector2(posX - defaultDotDiameter / 2, posY - defaultDotDiameter / 2);
84-
8582
const dot = {
8683
color: getRandomColor(),
8784
mass: defaultDotMass,
8885
diameter: defaultDotDiameter,
89-
position: centeredPos,
86+
position: new Vector2(posX, posY),
9087
velocity: Vector2.zero,
9188
};
9289

@@ -138,17 +135,7 @@ export class HomeComponent implements AfterViewInit, OnDestroy {
138135
// Dot's mass and dimensions do not exceed the max limit.
139136
if (dotBeingCreated.diameter < maxDotDiameter) {
140137
dotBeingCreated.mass += defaultDotMass * defaultIncrementRatio;
141-
142-
// Storing the diameter change in a separate variable to keep the dot's center at the click position.
143-
const diaChange = defaultDotDiameter * defaultIncrementRatio;
144-
// Updating diameter.
145-
dotBeingCreated.diameter += diaChange;
146-
147-
// Moving dot's center to the click position.
148-
dotBeingCreated.position = new Vector2(
149-
dotBeingCreated.position.x - diaChange / 2,
150-
dotBeingCreated.position.y - diaChange / 2,
151-
);
138+
dotBeingCreated.diameter += defaultDotDiameter * defaultIncrementRatio;
152139
}
153140

154141
newDots[newDots.length - 1] = { ...dotBeingCreated };
@@ -200,7 +187,6 @@ export class HomeComponent implements AfterViewInit, OnDestroy {
200187
/** Calculates the force of gravity between given dots. */
201188
private _gravityBetweenDots(dot1: Dot, dot2: Dot): Vector2 {
202189
const displacement = dot1.position.displacement(dot2.position);
203-
204190
const displacementDir = displacement.direction();
205191
const displacementMag = displacement.magnitude();
206192

src/shared/components/dot/dot.component.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,8 +25,8 @@ export class DotComponent implements OnInit {
2525

2626
this.ngStyle = {
2727
position: 'absolute',
28-
top: `${this.dot.position.y}px`,
29-
left: `${this.dot.position.x}px`,
28+
top: `${this.dot.position.y - this.dot.diameter / 2}px`,
29+
left: `${this.dot.position.x - this.dot.diameter / 2}px`,
3030

3131
borderRadius: '50%',
3232
width: `${this.dot.diameter}px`,

0 commit comments

Comments
 (0)