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

update jquery (and jqueryui) -- 1.7.1 is 4 years old #52

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

update jquery (and jqueryui) -- 1.7.1 is 4 years old #52

sebhaase opened this issue Dec 22, 2015 · 7 comments

Comments

@sebhaase
Copy link

SVG-Edit currently uses jQuery 1.7

version 1.8 or better 1.9 would be start towards jQuery 2.x, which is current.

E.g. jQuery.andSelf could be replaced with jQuery.addBack

@GoTeamScotch
Copy link
Contributor

I agree with this. On that note, I've been using jQuery v1.11.3 in my local devleopment copy with no apparent problems resulting from using the newer version. I haven't gone through the unit tests though so there could be some complications present.

Note: I'm also using jQuery UI v1.11.4 with no apparent problems.

@sivasaravanand
Copy link

please let me know what are the files needs to change in order to reference latest jquery file. i tried but throwing many script errors

@Harvie
Copy link

Harvie commented Feb 26, 2017

also please everywhere load the jquery using HTTPS!!!! #172

@brettz9 brettz9 changed the title update jquery -- 1.7.1 is 4 years old update jquery (and jqueryui) -- 1.7.1 is 4 years old May 19, 2018
@brettz9 brettz9 changed the title update jquery (and jqueryui) -- 1.7.1 is 4 years old update jquery (and jqueryui) -- 1.7.1 is 4 years old; the copy of jQuery used by the imagelib extension is even older May 20, 2018
@brettz9
Copy link
Contributor

brettz9 commented May 20, 2018

Though I fixed the HTTPS issue already, I'm working on a PR to avoid CDN use entirely (applies to imagelib and mathjax) (the latter URL wasn't working). This should help with local testing/offline apps as well as ensure future compatibility. Optional use of CDN could be added (PRs welcome).

Ideally any update to these older versions that I've copied will be added through npm instead, ensuring a smoother upgrade path.

There is a "jquery_release" variable in an HTML comment which was apparently intended to conditionally load the CDN version, but I'm not sure what build process uses that, as I couldn't find that elsewhere in the repo.

@brettz9 brettz9 changed the title update jquery (and jqueryui) -- 1.7.1 is 4 years old; the copy of jQuery used by the imagelib extension is even older update jquery (and jqueryui) -- 1.7.1 is 4 years old; the copy of jQuery used by the imagelib extension is even older; also canvg and plugins Jul 10, 2018
@brettz9 brettz9 changed the title update jquery (and jqueryui) -- 1.7.1 is 4 years old; the copy of jQuery used by the imagelib extension is even older; also canvg and plugins update jquery (and jqueryui) -- 1.7.1 is 4 years old; also canvg and plugins Sep 25, 2018
@brettz9
Copy link
Contributor

brettz9 commented Sep 25, 2018

I've at least gotten rid of the even older version of jQuery used by the Imagelib extension (it had been using a 1.4.4 copy)...

One approach I've been considering (and I'd like your feedback) is to actually move away from jQuery in places if not entirely, using ES6, DOM enhancements (like ChildNode and ParentNode interfaces), and focused libraries, such as my project Jamilih--which aims at a straightforward, light-weight approach to building the DOM--there is less need for jQuery.

And no doubt there are some light-weight, single-purpose substitute micro-libraries for traversal, etc. as well (https://github.com/WebReflection/query-result looks interesting to me despite its age).

There are advantages to jQuery and frameworks, but with the magic can come less focused and breaking version changes, and some bloat, and, as we're experiencing now, difficulty to migrate.

What do people think?

@brettz9 brettz9 changed the title update jquery (and jqueryui) -- 1.7.1 is 4 years old; also canvg and plugins update jquery (and jqueryui) -- 1.7.1 is 4 years old Sep 25, 2018
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 pushed a commit that referenced this issue Mar 14, 2021
jfhenon added a commit that referenced this issue Mar 14, 2021
@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.

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

6 participants