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 to jquery 1.3.x, jqueryUI 1.7.x #510

Closed
robocoder opened this issue Jan 18, 2009 · 13 comments
Closed

Update to jquery 1.3.x, jqueryUI 1.7.x #510

robocoder opened this issue Jan 18, 2009 · 13 comments
Assignees
Labels
Bug For errors / faults / flaws / inconsistencies etc. Major Indicates the severity or impact or benefit of an issue is much higher than normal but not critical.
Milestone

Comments

@robocoder
Copy link
Contributor

As a follow-up to #243, we did not update some ui plugins. jquery 1.3 was released this past week, and jqueryUI 1.6 is reportedly around the corner (it slipped its Dec 2008 release target).

@robocoder
Copy link
Contributor Author

This is dependent on various jQuery plugins being updated to support jQuery 1.3.x (which has introduced some compat busters).

http://docs.jquery.com/Release:jQuery_1.3#Upgrading

@robocoder
Copy link
Contributor Author

fdd2div 1.0 plugin didn’t support jQuery 1.2.6; we’ve sent our patch upstream but may have to forward port our local mods

@mattab
Copy link
Member

mattab commented Feb 4, 2009

vipsoft, when you say “we’ve sent our patch upstream but may have to forward port our local mods” do you mean that you’ve emailed the authors the patch? or did you patch the fdddiv locally to make it work (as well as patching our local modifications from previous versions) ?

@robocoder
Copy link
Contributor Author

Yes, there are upstream tickets for common bugs fixed in Piwik and local mods specific to Piwik (should upstream developers be interested in adopting). Yes, local mods were forward ported in the update to jQuery 1.2.6.

@mattab
Copy link
Member

mattab commented Mar 2, 2009

increasing priority as for the stable release we shouldn’t rely on libraries that depend on old Jquery versions; hopefully fdddiv can be updated to support jquery 1.3.x either by us or they will release a new release. vipsoft if you have any update on this please let us know!

@robocoder
Copy link
Contributor Author

In #243, we didn't touch ui.sortable.js or ui.mouse.js. (Code from circa Oct 2007. These have since been incorporated into jqueryUI.)

This is the regression found in #709 with IE7:

* Flash widgets (e.g., "Last visits graph" and "Visits by server time") can no longer be dragged & dropped to change the layout; error console reports: Line 13, Char 2817, "Object doesn't support this property or method"
* "Add a widget..." - the screen jumps when you mouseover "Visitor Settings" or "Visitors", making it difficult to select a widget
* "Add a widget..." - mouseover "Example Widgets" and then "Piwik Changelog" - the preview window is too tall and it isn't scrollable; you can't access any controls to either add the widget to the dashboard or close the window; the Escape key doesn't work 

@robocoder
Copy link
Contributor Author

For Superfish, consider adding supersubs.js. This allows for variable width submenus. May help with some long translations.

@robocoder
Copy link
Contributor Author

Alternatively: replace superfish and separate the "navigation" and "breadcrumbs" functionality.

@robocoder
Copy link
Contributor Author

These are the backward compatibility problems identified so far with the update to jquery 1.3.2:

  • IE7: dashboard: drag & drop Flash widgets works except
    • during the drag, the widget displays an OFC error (unable to load data)
    • on drop, the chart is offset within the div until clicked (causing a reload)
  • IE7: 'Add a widget' - screen still jumps during mouseover (introduced with 1.2.6 update)
  • FF3: dashboard: drag & drop broken
Error: [Exception... "Node cannot be inserted at the specified point in the hierarchy"  code: "3" nsresult: "0x80530003 (NS_ERROR_DOM_HIERARCHY_REQUEST_ERR)"  location: "http://localhost/libs/jquery/jquery.js?piwik=0.5 Line: 268"]
Source File: http://localhost/libs/jquery/jquery.js?piwik=0.5
Line: 268
  • FF3/Safari: feedback form; scrollbar(s) appear; worse in FF3 because submit button is obscured
  • FF3/IE7/Opera: left mouse down on submenu item, hover off, and release, leaves item in selected state

@robocoder
Copy link
Contributor Author

Ok. Moving the non-jquery-specific issues to separate tickets.

@robocoder
Copy link
Contributor Author

In [1553], fixes #510 - Update jquery.js

    * update jQuery to 1.3.2
    * add jquery-ui.js 1.7.2
    * move loading of external .css before scripts
    * remove ui.mouse.js
    * remove ui.sortable_modif.js 

@robocoder
Copy link
Contributor Author

In [1558], refs #510, refs #946 - remove sortable helper class (no longer used)

@robocoder
Copy link
Contributor Author

In [refs #510 - deprecated '@id=IDVALUE' changed to '#IDVALUE'

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug For errors / faults / flaws / inconsistencies etc. Major Indicates the severity or impact or benefit of an issue is much higher than normal but not critical.
Projects
None yet
Development

No branches or pull requests

2 participants