Skip to content
This repository has been archived by the owner on May 21, 2024. It is now read-only.

W2G2 Integration #62

Closed
wants to merge 2 commits into from
Closed

W2G2 Integration #62

wants to merge 2 commits into from

Conversation

dnviti
Copy link

@dnviti dnviti commented Apr 9, 2019

Needed to block right click menu when a file is locked!

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) {
Copy link
Member

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 :)

Copy link
Author

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...

Copy link
Member

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? :)

@NastuzziSamy
Copy link
Member

But thank you for the PR, nice point in handling this haha :)

Copy link
Author

@dnviti dnviti left a 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

@Ouack23
Copy link

Ouack23 commented Nov 19, 2019

Is it ok now? Used Notepad++ this time :D

Same result, take a look at this

@jancborchardt
Copy link
Member

cc @juliushaertl on this one regarding the files side since it might conflict with other stuff?

@skjnldsv skjnldsv closed this Oct 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants