jenkins-bot has submitted this change and it was merged.

Change subject: Highlight the source section on hover of placeholders in 
translation column
......................................................................


Highlight the source section on hover of placeholders in translation column

Change-Id: I014bcb7db9cac3c5494826e0d4df0d5bb6f62ef5
---
M modules/source/styles/ext.cx.source.less
M modules/translation/ext.cx.translation.js
2 files changed, 6 insertions(+), 0 deletions(-)

Approvals:
  KartikMistry: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/source/styles/ext.cx.source.less 
b/modules/source/styles/ext.cx.source.less
index d175493..cfa7238 100644
--- a/modules/source/styles/ext.cx.source.less
+++ b/modules/source/styles/ext.cx.source.less
@@ -16,4 +16,7 @@
                .mw-ui-item;
                .mw-ui-one-third;
        }
+       .highlight {
+               background-color: #FFFEEC;
+       }
 }
diff --git a/modules/translation/ext.cx.translation.js 
b/modules/translation/ext.cx.translation.js
index eed5bc4..1b82c9b 100644
--- a/modules/translation/ext.cx.translation.js
+++ b/modules/translation/ext.cx.translation.js
@@ -157,13 +157,16 @@
                        } );
                        $section.on( 'mouseenter', function () {
                                $( this ).addClass( 'placeholder' ).html( '+ 
Add Translation' );
+                               $( '#' + $( this ).data( 'source' ) ).addClass( 
'highlight' );
                        } );
                        $section.on( 'mouseleave', function () {
                                $( this ).removeClass( 'placeholder' ).empty();
+                               $( '#' + $( this ).data( 'source' ) 
).removeClass( 'highlight' );
                        } );
                        $section.on( 'click', function () {
                                $( this ).removeClass( 'placeholder' )
                                        .unbind( 'mouseenter mouseleave click' 
);
+                               $( '#' + $( this ).data( 'source' ) 
).removeClass( 'highlight' );
                                mw.hook( 'mw.cx.translation.add' ).fire( $( 
this ).data( 'source' ) );
                        } );
 

-- 
To view, visit https://gerrit.wikimedia.org/r/121579
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I014bcb7db9cac3c5494826e0d4df0d5bb6f62ef5
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/ContentTranslation
Gerrit-Branch: master
Gerrit-Owner: Santhosh <santhosh.thottin...@gmail.com>
Gerrit-Reviewer: KartikMistry <kartik.mis...@gmail.com>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to