Last modified: 2014-06-11 16:06:05 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T67738, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 65738 - Fix code styling and architecture issues for Special:PageMigration
Fix code styling and architecture issues for Special:PageMigration
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
Translate (Other open bugs)
unspecified
All All
: Normal enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
https://www.mediawiki.org/wiki/Manual...
:
Depends on:
Blocks: 65740
  Show dependency treegraph
 
Reported: 2014-05-25 06:13 UTC by Pratik Lahoti
Modified: 2014-06-11 16:06 UTC (History)
5 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Pratik Lahoti 2014-05-25 06:13:41 UTC
The CSS applied for Special:PageMigration as of now is quite trivial. Input fields, buttons, and other blocks of text need to be styled to meet the standards followed.

Refer: https://gerrit.wikimedia.org/r/#/c/133720/17/resources/css/ext.translate.special.pagemigration.css
Comment 1 Nemo 2014-05-25 07:50:06 UTC
Niklas to define what needs doing, as he called this a blocker for deployment.
Comment 2 Nemo 2014-05-26 06:43:19 UTC
The CSS work is part of bug 65736.
Comment 3 Nemo 2014-05-30 19:12:06 UTC
Niklas, can you please tell if this was fixed as byproduct of https://gerrit.wikimedia.org/r/135750 / bug 65736 or what else is left to do to comply with [[mw:CC]]?
Comment 4 Niklas Laxström 2014-05-30 20:03:39 UTC
It's getting better. With some further work and adding some QUnit tests this can be closed. Note that I do not just mean mw:CC but code architecture as well.
Comment 5 Nemo 2014-06-04 17:26:00 UTC
Next task defined:

Nikerabbit> BPositive: you can start by creating a function like listen() which adds all the click handlers... try to have the callbacks for those handlers as separate functions so that the listen function itself is very short.Nikerabbit> BPositive: $( document ).ready( listen ); function listen() { var $form = ...; $listing = ...; $listing.on( 'click', ..., someCallbackFunctionName  ) }
Comment 6 Gerrit Notification Bot 2014-06-04 19:48:03 UTC
Change 137406 had a related patch set uploaded by BPositive:
Code architecture fixed for Special:PageMigration

https://gerrit.wikimedia.org/r/137406
Comment 7 Gerrit Notification Bot 2014-06-06 10:18:37 UTC
Change 137406 merged by jenkins-bot:
Code architecture fixed for Special:PageMigration

https://gerrit.wikimedia.org/r/137406
Comment 8 Nemo 2014-06-06 10:31:51 UTC
Anything else to do?
Comment 9 Niklas Laxström 2014-06-11 16:06:05 UTC
Let's close this one because it has been to vague for the whole time.

Ideally the code would be made a class or jquery plugin following examples in other files.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links