Skip to content

TouchBlick fix after reformat code#20

Merged
SCratORS merged 1 commit intomainfrom
touch-blick-fix
Feb 10, 2024
Merged

TouchBlick fix after reformat code#20
SCratORS merged 1 commit intomainfrom
touch-blick-fix

Conversation

@SCratORS
Copy link
Copy Markdown
Owner

No description provided.

@SCratORS SCratORS merged commit 6682a9b into main Feb 10, 2024
for (int i = 0; i < scenesSize; i++) {
scenes.get(i)
.setupPosition(surfaceWidth, surfaceHeight, scaleImageHeight, displayRotation);
.setupPosition(surfaceWidth, surfaceHeight, (i<scenesSize-1)?scaleImageHeight:scaleImageWidth, displayRotation);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry if I broke this. This code is not formatted now BTW (no spaces between operators).
Usually when I make any changes I press IDE shortcut to run a formatter.
Please consider doing the same for consistent formatting.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not against it, but I haven't learned how to use it yet.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you are using Android Studio on Linux, the default shortcut might be Ctrl + Alt + L.
You can find it here
Format code

And you can change the shortcut here

Formatter shortcut

I also recommend running "Optimize imports". It removes unused imports and rearranges them alphabetically.
Optimize imports

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I use Android Studio on Windows. Yes, it's in the same place. I will use it. Thanks for the hint

@SCratORS SCratORS deleted the touch-blick-fix branch February 13, 2024 06:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants