Attachment 210981 Details for Bug 358571 – First proposition

Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

[patch] First proposition

clipboard.txt (text/plain), 2.04 KB, created by Malgorzata Janczarska

on 2012-02-14 11:05:18 EST

(hide)

Description:

Filename:

MIME Type:

Creator: Malgorzata Janczarska

Created: 2012-02-14 11:05:18 EST

Size: 2.04 KB

patch

obsolete

>diff --git a/bundles/org.eclipse.orion.client.core/web/orion/widgets/LoginDialog.js b/bundles/org.eclipse.orion.client.core/web/orion/widgets/LoginDialog.js
>index b413b0b..908dc0c 100644
>--- a/bundles/org.eclipse.orion.client.core/web/orion/widgets/LoginDialog.js
>+++ b/bundles/org.eclipse.orion.client.core/web/orion/widgets/LoginDialog.js
>@@ -26,6 +26,16 @@
> 		
> 		postCreate : function() {
> 			this.inherited(arguments);
>+			var _self = this;
>+
>+			dojo.connect(this.closeLink, "onmouseover", this.closeLink, function() {
>+				_self.closeLink.style.cursor = "pointer";
>+			});
>+			dojo.connect(this.closeLink, "onmouseout", this.closeLink, function() {
>+				_self.closeLink.style.cursor = "default";
>+			});
>+			
>+			dojo.connect(this.closeLink, "onclick", function() {dojo.hitch(_self, _self.closeDialog)();});
> 		},
> 	
> 		setPendingAuthentication: function(services){
>@@ -275,8 +285,7 @@
> 	    img.src = null; // no server request
> 		return url.match(re)[1].toString();
> 		},
>-	_onBlur: function(){
>-		this.inherited(arguments);
>+	closeDialog: function(){
> 		if(dijit.popup.hide)
> 			dijit.popup.hide(this); //close doesn't work on FF
> 		dijit.popup.close(this);
>diff --git a/bundles/org.eclipse.orion.client.core/web/orion/widgets/templates/LoginDialog.html b/bundles/org.eclipse.orion.client.core/web/orion/widgets/templates/LoginDialog.html
>index 16add52..87f30bc 100644
>--- a/bundles/org.eclipse.orion.client.core/web/orion/widgets/templates/LoginDialog.html
>+++ b/bundles/org.eclipse.orion.client.core/web/orion/widgets/templates/LoginDialog.html
>@@ -11,6 +11,12 @@
> 			<table dojoAttachPoint="otherUnauthenticated" style="display: none; width: 100%;">
> 			 <tbody dojoAttachPoint="otherUnauthenticatedList">
> 			</tbody>
>+			</table>
>+			<table class="LoginWindowRight" style="width: 100%;">
>+				<tr><td style="text-align: right;">
>+					<a class="LoginWindowRight" style="width: 100%;" dojoAttachPoint="closeLink">Close</a>
>+				</td></tr>
>+			</table>
> 		</div>
> 	</div>
> 	<div class="dijitTooltipConnector" role="presentation"></div>

Actions: View | Diff

Attachments on bug 358571: 210981 | 210987 | 210988 | 210989 | 211027