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.
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add GUI #2375
base: dev/1.4
Are you sure you want to change the base?
Add GUI #2375
Changes from 250 commits
4be33d5
0d3f67b
61bfb35
9dd5bf1
371550b
ed074ff
08214c2
094b5e6
04eb47c
650b135
3f0cfa0
7e33812
c265204
d442be2
7a09c01
901ba13
b256155
10bb95b
b62df3a
0dbad8d
9e50d30
ebd27df
c98748f
7649de4
68657e8
c95c0c2
8f36498
10fc066
c9ac2a5
5dc393d
d667d99
caa0a38
8f77a72
f08d477
7f4a460
cfe01a5
83cbf4a
c353ee3
fbf6ed1
997f29f
68f8993
96e9621
f9e67ec
48bcccf
c65299a
0acf868
5871137
999e348
fa5a17f
8fae435
332ff64
3710d41
1714428
8bcde48
a0e2dc5
f232172
09e975c
bfb78bc
a35fc8f
169f3eb
436d4b1
ae911ae
2621b7e
0b2b12d
239feed
8388166
520da7b
54ff8f1
d8393e8
836e752
169aa09
c09d5e7
df3b5f8
3ada1f6
6edbf7b
fbaa8aa
26b3b94
bae7a9a
7f613d9
271a77c
9575b36
054ded6
cef8757
1772f93
a1969c3
78059b3
7d57894
59d5c86
531e2b3
67eea0f
41401e4
0e48dbb
4cb99c8
e2a74ca
ef592f6
80e88c8
edc5953
6a0962e
b4748af
3d8cbc2
c880f70
0455188
44447f3
98e7887
4b5dc36
7f6798d
7347e62
59ce725
c502f6d
e0e85c6
70cc80d
8ca4efb
ba35d56
10d4bf5
c100342
e51478c
bbbc3ff
4c9794b
30348f1
8583bf9
43687db
754a7f4
8332c31
6b07a12
1bf8334
3f46739
dc293bd
9860e71
2ab14d6
beb3593
aa0d174
af0f2d7
0b45a7f
7cb710c
e4f0d93
854410d
5cb8b48
ce13c8e
e9f2bee
38edcbf
9d407d2
043d9fa
0aa5514
6475f15
6a7bf93
0b99f79
75bc0c1
beb506b
fdca466
e3f0aae
37359f0
31529a9
1203df7
42dcd13
5fef2a1
78add14
fd2dc9d
c9177bf
e71a6da
1457cdd
c9b3157
477af1f
7ccffd3
11691f6
4934498
f0ebdd0
fa07ecd
7ae6558
52c0d28
5931066
a197c1e
189a1e7
aa6aff8
077e163
40d926f
334bd4a
8d0ee7a
fc9302b
2dce250
4e83314
012b61e
f7dacdc
41075e6
2f4ce0f
04bb5b3
f237f6a
20a1d30
5279a74
6a87332
a5e0642
d88532f
6904c05
d6c03fa
f702b2b
bc8a8d4
53d136e
15b8e6c
4b2487d
ae63b66
8094d8a
3a29609
8a7073a
b8bb449
5d4af60
c4ace76
d73f8f5
237230d
2511b08
14008ad
c3a7089
a10bb1b
e300764
47a9a97
ecbf411
d984227
c05a6fc
923bc96
81389df
d1592ac
bb3b3e8
3929d73
f91d08b
0bbe7eb
b404109
9fda417
847a1c9
3ef755a
79413fc
e467333
edf432f
ca5306c
66aa974
185c659
152368f
ab7cf58
be1d389
a63f4c4
3563bc0
c19f8cc
7a9d1ec
3bca5af
b338f12
46734a0
fcc6c1c
bd0e4e1
fd267ba
374b4d0
dcdfc1a
8ffb614
5a43240
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🛠️ Refactor suggestion
Remove @ts-ignore by properly typing the bounds property.
Instead of using @ts-ignore, consider adding proper type definitions for the bounds property in the ISpriteRenderer interface.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Above three lines and following line can be optimized using destructuring to reduce redundancy and improve readability. Here’s an optimized version:
const { _subChunk: { chunk: { vertices } }, color: { r, g, b, a } } = renderer;
const finalAlpha = a * alpha;
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🛠️ Refactor suggestion
Improve readability by simplifying assignment statements
Consider breaking down the combined assignment statements into separate lines for better clarity and maintainability.
Apply this diff to refactor the code:
📝 Committable suggestion
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can not agree more
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@singlecoder, glad to hear that you agree! If there's anything else I can assist with, please let me know.
(^ᆺ^)ノ
If you found this review helpful, would you consider giving us a shout-out on X?
Thank you for using CodeRabbit!