r111714 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r111713‎ | r111714 | r111715 >
Date:01:56, 17 February 2012
Author:kaldari
Status:deferred
Tags:
Comment:
switching to large button graphics for the large buttons, depends on core change r111710
Modified paths:
  • /trunk/extensions/ArticleCreationWorkflow/modules/ext.articleCreation.user/ext.articleCreation.user.js (modified) (history)

Diff [purge]

Index: trunk/extensions/ArticleCreationWorkflow/modules/ext.articleCreation.user/ext.articleCreation.user.js
@@ -35,7 +35,7 @@
3636 //setup button hover states
3737 ac.panel
3838 .find( '.ac-article-button' )
39 - .addClass('ui-button-blue')
 39+ .addClass('ui-button-blue-large')
4040 //attach other events here, just making first tooltip for now
4141 //testing hover effects
4242 .hover (function (){
@@ -95,8 +95,8 @@
9696
9797 $( this )
9898 //make it green
99 - .removeClass('ui-button-blue')
100 - .addClass('ui-button-green')
 99+ .removeClass('ui-button-blue-large')
 100+ .addClass('ui-button-green-large')
101101 .addClass('ac-button-selected')
102102 .parent()
103103 .find('.mw-ac-tooltip' )
@@ -237,11 +237,11 @@
238238 hideInterstitial : function($elements) {
239239 //remove green states and hide their tooltips
240240 $elements
241 - .removeClass('ui-button-green')
 241+ .removeClass('ui-button-green-large')
242242 .removeClass('ac-button-selected')
243243 .each ( function (i, e) {
244244 var color = $(this).data('ac-color');
245 - $(this) .addClass( 'ui-button-'+color )
 245+ $(this) .addClass( 'ui-button-'+color+'-large' )
246246 .parent()
247247 .find('.mw-ac-tooltip,.mw-ac-interstitial')
248248 .hide();

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r111710changing jquery.ui buttons to match assets from brandon, also adding separate...kaldari01:34, 17 February 2012

Status & tagging log