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

jquery.contextMenu.js is version 1.01 - 4 years old (js-hotkeys, jspdf, canvg/rgbcolor/stackblur-canvas, mathjax also have new versions and/or need updating from ours) #51

Closed
sebhaase opened this issue Dec 22, 2015 · 7 comments

Comments

@sebhaase
Copy link

jquery.contextMenu.js is version 1.01 - 4 years old
while https://github.com/swisnl/jQuery-contextMenu
shows now version 2.0.1
It appears to have been continuously maintained...

does the new version work as a drop-in replacement ?

can anyone estimate how hard it would be to get it to work (if not) ?

@GoTeamScotch
Copy link
Contributor

It's not a drop-in replacement. A few areas in SVG Edit would need to be modified. I just tried it and it seems to break the app because of javascript errors. Some research would need to be done to find the new function names and update the code to reflect those and other changes to contextMenu. That being said, it's probably more work than its worth. That old saying comes to mind: if it ain't broke, don't fix it.

@brettz9
Copy link
Contributor

brettz9 commented May 19, 2018

Upgrading to the latest version would require an update to jQuery too as requested in issue #52

@brettz9 brettz9 changed the title jquery.contextMenu.js is version 1.01 - 4 years old jquery.contextMenu.js is version 1.01 - 4 years old (jquerybbq, js-hotkeys, jspdf also have new versions) May 19, 2018
@brettz9 brettz9 changed the title jquery.contextMenu.js is version 1.01 - 4 years old (jquerybbq, js-hotkeys, jspdf also have new versions) jquery.contextMenu.js is version 1.01 - 4 years old (jquerybbq, js-hotkeys, jspdf, mathjax also have new versions) May 20, 2018
@brettz9
Copy link
Contributor

brettz9 commented Sep 13, 2018

Regarding jsPDF, per #269 (comment) , we might take a look at https://github.com/alafr/SVG-to-PDFKit or per #254 (comment) , see https://github.com/devongovett/svgkit or https://github.com/foliojs/pdfkit

Regarding hotkeys, though there is this version, we might look at this non-jQuery alternative: https://github.com/jaywcjlove/hotkeys

@brettz9 brettz9 changed the title jquery.contextMenu.js is version 1.01 - 4 years old (jquerybbq, js-hotkeys, jspdf, mathjax also have new versions) jquery.contextMenu.js is version 1.01 - 4 years old (jquerybbq, js-hotkeys, jspdf, canvg, mathjax also have new versions) Sep 25, 2018
@brettz9 brettz9 changed the title jquery.contextMenu.js is version 1.01 - 4 years old (jquerybbq, js-hotkeys, jspdf, canvg, mathjax also have new versions) jquery.contextMenu.js is version 1.01 - 4 years old (jquerybbq, js-hotkeys, jspdf, canvg/rgbcolor/stackblur-canvas, mathjax also have new versions and/or need updating from ours) Nov 3, 2018
This was referenced Apr 17, 2019
@brettz9 brettz9 changed the title jquery.contextMenu.js is version 1.01 - 4 years old (jquerybbq, js-hotkeys, jspdf, canvg/rgbcolor/stackblur-canvas, mathjax also have new versions and/or need updating from ours) jquery.contextMenu.js is version 1.01 - 4 years old (js-hotkeys, jspdf, canvg/rgbcolor/stackblur-canvas, mathjax also have new versions and/or need updating from ours) Jan 5, 2020
jfhenon added a commit that referenced this issue Mar 5, 2021
* #issue-fix The new menu will ask if you want to erase the current content but it will ignore the answer

* #issue-fix  dialog needs to be closer than the original

* #issue-fix main menu alignment changes

* #issue_fix double click and opensvg issue fixed

* #issue-fix process_cancel change to seConfirm

* #issue-fix review how the top toolbar display when many buttons are displayed

* #issue-fix unwanted css reoved

* #issue-fix BOTTOM TOOLS Make sure all features of the bottom toolbar are working

* #issue-fix IMPORT IMAGE menu open issue fixed

* #issue-fix alert dialog overwrite style

* #issue-fix lint issue fixed

* npm update + associated fixes

* #36 look of opacity button should like the zoom button (without the dropdown button)

* #37 Clicking anywhere on the bottom bar (for example below the opacity button is displaying the “color popup”.)

* #38 The opacity button does not update with the current element

* #42 When you import an image, it works well but the dialog should disappear automatically

* Fixes #53: 4th option for the background display fixed

* Fixes #53: 4th option for the background fixed

* #49 the x button does not work

* Fixes #41: Alignment fixes in bottom bar

* fix test scenario

* #39 opacity button/stroke size/radius button don’t allow the 0 value

* #41 The look and alignment of stroke size and stroke style should be consistent with the rest of the bottom bar.

* #44 Text font dropdown is broken

* #52 Fix the export

* Fixes #44: Text font dropdown fixed

* Fixes #44: Text font dropdown fixed

* Fixes #44: Text font dropdown fixed

* Fixes #44: Text font dropdown fixed

* Update index.html

* update husky

* fixes

* moves from index.html to js

* fix #66

* fix #48 first set of new icons

* Reorganize tools in left panel to be more intuitive

* Update button styles to adapt to new icons

* Fix #48 new set of icons for alignments

* Variabilisation of icon bkgd colors
Update of color choice to fit new icons design

* Update canvas and rulers colors

* Improve layer handle design

* Modernize SVG Logo but keeping original spirit

* Fix #48 continue improve icons

* Continue to reorganise left panel

* Update right panel handle for layers

* Fix #48 new set of icons for main menu

* Improve main menu design

* Rework menu organisation

* Rework menu organisation

* Update input element design

* New icons

* Improve Zoom Module

* Improve Color Picker Module

* New icons

* New icons

* #65 restore the feature of start/end marker lines partially did

* #64 The export dialog needs to include the quality options (except for PDF)

* #39 opacity button/stroke size/radius button don’t allow the 0 value

* #39 eslint changes

* #69 menu button style overwrite

* #65 restore the feature of start/end marker lines

* #43 With a small window, the look of the top toolbar is broken

* #43 tool top alignment revert

* #46 Move this jQuery component to a web component  for graduate and picker components

* #46 Move this jQuery component to a web component

* remove some non standard lint rules

* #46 picker convert as pure javascript changes

* #46 jquery plugin convert to pure javascript

* #46 jquery plugin change to pure javascript

* #46 jquery remove and convert to pure javascript

* #46 slider issue fixed

* #46 ColorValuePicker and js convertion

* #46 globals $ remov from slider class

* #46 jquery convert pure javascript changes

* #46 jquery to js convertion changes

* #46 paintbox and current element color set issue fixed

* #46 unwanted files removed

* #46 $.extend modify changes

* #46 extend modifiey changes

* #46 $.extend change to pure javascript

* #46 extend and data changes

* #46 jquery removed

* #45 These buttons don't work and dropdown is broken

* #45 These buttons don't work and dropdown is broken

* #45 These buttons don't work and dropdown is broken commit reverted

* #43 With a small window, the look of the top toolbar is broken

* Update jQuery.jPicker.js

* #76 draggable modification changes

* #76 jquery-ui-1.8.17.custom.min.js removed from editor

* #76 jquery ui file removed

* npm update

* #77 console issue fixed

* #77 <SVG> button issue fixed

* #77 shortkey issue fixed

* #77 jquery hotkeys plugin changes

* #77 hotkey plugin related changes

* #78 hotkey related code comment.

* #78 js-hotkeys/jquery.hotkeys.min.js file removed from svgedit.js

* #51 Rewrite the color palette without elix

* #81 unwanted files removed

* #81 folder name renamed

* #81 folder rename changes

* #81 jquery-ui  folder renamed

* #81 jquery modify changes

* #81 globals $ removed

Co-authored-by: Agriya Dev5 <[email protected]>
Co-authored-by: mathieucura <[email protected]>
jfhenon added a commit that referenced this issue Mar 14, 2021
* #issue-fix The new menu will ask if you want to erase the current content but it will ignore the answer

* #issue-fix  dialog needs to be closer than the original

* #issue-fix main menu alignment changes

* #issue_fix double click and opensvg issue fixed

* #issue-fix process_cancel change to seConfirm

* #issue-fix review how the top toolbar display when many buttons are displayed

* #issue-fix unwanted css reoved

* #issue-fix BOTTOM TOOLS Make sure all features of the bottom toolbar are working

* #issue-fix IMPORT IMAGE menu open issue fixed

* #issue-fix alert dialog overwrite style

* #issue-fix lint issue fixed

* npm update + associated fixes

* #36 look of opacity button should like the zoom button (without the dropdown button)

* #37 Clicking anywhere on the bottom bar (for example below the opacity button is displaying the “color popup”.)

* #38 The opacity button does not update with the current element

* #42 When you import an image, it works well but the dialog should disappear automatically

* Fixes #53: 4th option for the background display fixed

* Fixes #53: 4th option for the background fixed

* #49 the x button does not work

* Fixes #41: Alignment fixes in bottom bar

* fix test scenario

* #39 opacity button/stroke size/radius button don’t allow the 0 value

* #41 The look and alignment of stroke size and stroke style should be consistent with the rest of the bottom bar.

* #44 Text font dropdown is broken

* #52 Fix the export

* Fixes #44: Text font dropdown fixed

* Fixes #44: Text font dropdown fixed

* Fixes #44: Text font dropdown fixed

* Fixes #44: Text font dropdown fixed

* Update index.html

* update husky

* fixes

* moves from index.html to js

* fix #66

* fix #48 first set of new icons

* Reorganize tools in left panel to be more intuitive

* Update button styles to adapt to new icons

* Fix #48 new set of icons for alignments

* Variabilisation of icon bkgd colors
Update of color choice to fit new icons design

* Update canvas and rulers colors

* Improve layer handle design

* Modernize SVG Logo but keeping original spirit

* Fix #48 continue improve icons

* Continue to reorganise left panel

* Update right panel handle for layers

* Fix #48 new set of icons for main menu

* Improve main menu design

* Rework menu organisation

* Rework menu organisation

* Update input element design

* New icons

* Improve Zoom Module

* Improve Color Picker Module

* New icons

* New icons

* #65 restore the feature of start/end marker lines partially did

* #64 The export dialog needs to include the quality options (except for PDF)

* #39 opacity button/stroke size/radius button don’t allow the 0 value

* #39 eslint changes

* #69 menu button style overwrite

* #65 restore the feature of start/end marker lines

* #43 With a small window, the look of the top toolbar is broken

* #43 tool top alignment revert

* #46 Move this jQuery component to a web component  for graduate and picker components

* #46 Move this jQuery component to a web component

* remove some non standard lint rules

* #46 picker convert as pure javascript changes

* #46 jquery plugin convert to pure javascript

* #46 jquery plugin change to pure javascript

* #46 jquery remove and convert to pure javascript

* #46 slider issue fixed

* #46 ColorValuePicker and js convertion

* #46 globals $ remov from slider class

* #46 jquery convert pure javascript changes

* #46 jquery to js convertion changes

* #46 paintbox and current element color set issue fixed

* #46 unwanted files removed

* #46 $.extend modify changes

* #46 extend modifiey changes

* #46 $.extend change to pure javascript

* #46 extend and data changes

* #46 jquery removed

* #45 These buttons don't work and dropdown is broken

* #45 These buttons don't work and dropdown is broken

* #45 These buttons don't work and dropdown is broken commit reverted

* #43 With a small window, the look of the top toolbar is broken

* Update jQuery.jPicker.js

* #76 draggable modification changes

* #76 jquery-ui-1.8.17.custom.min.js removed from editor

* #76 jquery ui file removed

* npm update

* #77 console issue fixed

* #77 <SVG> button issue fixed

* #77 shortkey issue fixed

* #77 jquery hotkeys plugin changes

* #77 hotkey plugin related changes

* #78 hotkey related code comment.

* #78 js-hotkeys/jquery.hotkeys.min.js file removed from svgedit.js

* #51 Rewrite the color palette without elix

* #81 unwanted files removed

* #81 folder name renamed

* #81 folder rename changes

* #81 jquery-ui  folder renamed

* #81 jquery modify changes

* #81 globals $ removed

* remove licence-badger (breaks ci)

* build

* fix tests

* fix folder spelling

* skip accessibility for now

* Update onpush.yml

* use npm dragmove

* #82 jquery convertion changes

* #82 javascript convertion changes

* #82 workarea related javascript convertion changes

* #82 javascript convertion cahnges

* #82 ungroup button added changes

* #82 menu unwanted code removed and button changes

* Update .gitignore

* refactor with Panel classes

* add MainMenu class

* move handlers

* fix lint

* fix test

Co-authored-by: Agriya Dev5 <[email protected]>
Co-authored-by: mathieucura <[email protected]>
@jfhenon
Copy link
Collaborator

jfhenon commented May 15, 2021

This issue has no activity for more than 2 years.

@jfhenon jfhenon closed this as completed May 15, 2021
@sebhaase
Copy link
Author

Please don’t close tickets just because they are old.

@jfhenon
Copy link
Collaborator

jfhenon commented May 15, 2021

Sorry for this but I think a lot of tickets don't apply anymore to the current svgEdit status.
I will close ticket and reopen immediately them if any reaction like this one.

By the way, we are indeed updating all libraries to their most recent version. regarding jQuery, we are moving to native DOM functions with V7.

@jfhenon jfhenon reopened this May 15, 2021
@jfhenon
Copy link
Collaborator

jfhenon commented Jun 6, 2021

We don't use any more jQuery and Jquery UI.

@jfhenon jfhenon closed this as completed Jun 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants