Skip to content
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

Private/gokay/cell cursors #9104

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Conversation

gokaysatir
Copy link
Contributor

  • Resolves: #
  • Target version: master

Summary

TODO

  • ...

Checklist

  • Code is properly formatted
  • All commits have Change-Id
  • I have run tests with make check
  • I have issued make run and manually verified that everything looks okay
  • Documentation (manuals or wiki) has been updated or is not required

@gokaysatir gokaysatir marked this pull request as draft May 19, 2024 17:16
@gokaysatir gokaysatir force-pushed the private/gokay/cell-cursors branch 3 times, most recently from 84ade6a to 28122c0 Compare May 23, 2024 11:38
}

adjustSVGProperties() {
const scale = app.getScale();

Check notice

Code scanning / CodeQL

Unused variable, import, function or class Note

Unused variable scale.
@gokaysatir gokaysatir force-pushed the private/gokay/cell-cursors branch 3 times, most recently from 119e9e7 to 2e58d7b Compare May 30, 2024 14:02
Signed-off-by: Gökay Şatır <gokaysatir@collabora.com>
Change-Id: Icc14501bbf78556ad4e5bfb7fb3d6e8ec37191d8
Signed-off-by: Gökay Şatır <gokaysatir@collabora.com>
Change-Id: Idea4181c5a405c461e4f3a60b0f21725ff06d1f9
Signed-off-by: Gökay Şatır <gokaysatir@collabora.com>
Change-Id: I3d991ed1275a53d6591b8280fb956f16fb11812b

return shapeRecProps;

dragDistance[0] *= app.pixelsToTwips;

Check warning

Code scanning / CodeQL

Unreachable statement Warning

This statement is unreachable.
@@ -51,6 +51,10 @@
return app.file.readOnly;
};

app.getScale = function() {
return (app.tile.size.pixels[0] / app.tile.size.twips[0]) * 15;
}

Check notice

Code scanning / CodeQL

Semicolon insertion Note

Avoid automated semicolon insertion (93% of all statements in
the enclosing script
have an explicit semicolon).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: To Review
Development

Successfully merging this pull request may close these issues.

None yet

1 participant