-
Notifications
You must be signed in to change notification settings - Fork 23
Conversation
Needed to block right click menu when a file is locked!
js/files.js
Outdated
currentFile.find('.filename .nametext').click(); | ||
|
||
// W2G2 Integration | ||
if ($(currentFile).find(".statelock").length == 0) { |
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.
You made such a big refractor, I think it would be better to only check if the file is locked and if it is, just return null so the menu does not even process :)
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.
I think the spaces made the big refactor... VSCode auto beautifier...
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.
@dnviti could you update the pull request or make a new pull request without the auto beautifier? :)
But thank you for the PR, nice point in handling this haha :) |
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.
Is it ok now? Used Notepad++ this time :D
Same result, take a look at this |
cc @juliushaertl on this one regarding the files side since it might conflict with other stuff? |
Needed to block right click menu when a file is locked!