Closed Bug 1090710 Opened 10 years ago Closed 6 years ago

[tracking] migrate from yui to jquery

Categories

(Bugzilla :: Bugzilla-General, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: glob, Unassigned)

References

Details

(Keywords: meta)

tracking bug for migrating from yui2 to jquery + jquery-ui + datatables.net.
Depends on: 1090727
Severity: normal → enhancement
OS: Mac OS X → All
Hardware: x86 → All
I want to start working on YUI to jquery migration . May I get the instructions please ? Thanks.
(In reply to Kapil Bakshi[geekspark] from comment #1)
> I want to start working on YUI to jquery migration . May I get the
> instructions please ? Thanks.

That is great, thanks for your interest! 
What kind of "instructions" do you expect?
On which files should I start working on . I think I would have to start coding on the Javascript files inside the directory  bugzilla/js,right. Any other files I need to take care of..? Thanks.
Most of the javascript will be in the js/ directory -- inline javascript does exist but we can get to that last.

I think a good starting point would be https://github.com/bugzilla/bugzilla/blob/master/js/TUI.js
and https://github.com/bugzilla/bugzilla/blob/master/js/util.js,
which are collections of small utility functions. jquery's conciseness may even make them unnecessary.

Feel free to drop by #bugzilla on irc.mozilla.org if you have other questions and need a quick-ish response.
Thanks David. I am excited to start working on this. Can this project become a part of GSOC 2015 ?
Dylan Sir, the files you mentioned in your comment consist of functions only. I would like to know where are they being called . Also the code can be easily migrated to jquery using selectors and filters. Shall I only write the corresponding jquery code of these  two files now . Is there anything else I should consider .?

Thanks :)
and regarding cookie implementation should I use the carhart jquery plugin ?
(In reply to Kapil Bakshi[geekspark] from comment #6)
> Dylan Sir, the files you mentioned in your comment consist of functions
> only. I would like to know where are they being called . Also the code can
> be easily migrated to jquery using selectors and filters.

it's best to perform a straight migration of the existing instead of refactoring at the same time to use jquery selectors and filters.  this makes the code much easier to review and test.

refactoring to make better use of jquery is an excellent idea, but something that should be done later.

> and regarding cookie implementation should I use the carhart jquery plugin ?

yes; that's the one we ship with bugzilla.


i'll file a new bug for the TUI.js and util.js work and assign it to you.
Depends on: 1128408
Depends on: 1159589
Depends on: 1159596
Depends on: 1201266
Depends on: 1204238
Blocks: 1155522
Depends on: 1211118
Keywords: meta
Depends on: 1270991
I have closed some dependencies as WONTFIX as we are going to remove jQuery as well, just like GitHub. We are more fortunate because we have already dropped the IE support. Let’s say goodbye to YUI *and* jQuery!

https://githubengineering.com/removing-jquery-from-github-frontend/
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
See Also: → 1493992
You need to log in before you can comment on or make changes to this bug.