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.

Sign up for GitHub

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

Jump to bottom

TWEAKS refactoring #505

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

TWEAKS refactoring #505

wants to merge 14 commits into from

Conversation

MysticJay
Copy link
Contributor

TWEAKS plugins refactoring
namespace usage
code style fixes according to Eslint rules
Minor code changes

@MysticJay MysticJay added development general development issue plugin Ready to merge labels May 16, 2021
modos189
modos189 approved these changes Jul 4, 2021
View reviewed changes
plugins/link-show-direction.js Outdated Show resolved Hide resolved
johnd0e
johnd0e requested changes Dec 9, 2021
View reviewed changes
plugins/fix-china-map-offset.js Show resolved Hide resolved
plugins/link-show-direction.js Show resolved Hide resolved
plugins/link-show-direction.js Outdated Show resolved Hide resolved
plugins/scroll-wheel-zoom-disable.js Show resolved Hide resolved
@MysticJay MysticJay requested a review from johnd0e July 26, 2022 11:56
@github-actions
Copy link

github-actions bot commented Jul 26, 2022 •
edited
Loading

🤖 Pull request artifacts

file commit
IITC_Mobile-test.apk affc01d
test-0.36.1.20230811.143630.zip affc01d

See build on website

@MysticJay MysticJay changed the title Tweaks TWEAKS refactoring Jul 26, 2022
@MysticJay
namespace
cb5efc2 
nomalize namespace definition
remove all namespce occurences
add esxposed secition to top
@MysticJay
ESLINT
8580309 
changing code to match eslint rules.
function definition syntax
Version bump
@MysticJay
GIT Comments
cd995c5 
resolving GIT proposals
@MysticJay
Toolbox entry fixed
14ba222 
fixed onclick.
@MysticJay
Update periodic-refresh.js
035a8a3 
exposing refreshMinutes
@MysticJay
Comments resolved
8a722e4 
exposed values need pluginnames, refernces do not work
@MysticJay
eslint fixes
fd2844c
@MysticJay
eslint/prittier
1b3cc1c 
files adjusted to the proposals.
added pritter-ignore where needed
@MysticJay
line break
6c01715
@MysticJay
Update .eslintrc.json
2167da8
@MysticJay
Update periodic-refresh.js
db0cc65
@MysticJay
Update scroll-wheel-zoom-disable.js
bb44cca
@MysticJay
Update .prettierrc.json
67d0279 
remove changes
@MysticJay
Update link-show-direction.js
affc01d 
adressing change Request
for showDialog
@MysticJay MysticJay requested review from le-jeu and removed request for johnd0e August 11, 2023 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@johnd0e johnd0e johnd0e requested changes

@modos189 modos189 modos189 approved these changes

@le-jeu le-jeu Awaiting requested review from le-jeu

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
development general development issue plugin Ready to merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@MysticJay @modos189 @johnd0e @le-jeu