Better Facebook sign-in button.

hbbio · Mar 12, 2012 · 6368c16 · 6368c16
1 parent f44cba6
commit 6368c16
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 15 deletions.
diff --git a/resources/img/facebook_signin.png b/resources/img/facebook_signin.png
diff --git a/resources/style.css b/resources/style.css
@@ -22,6 +22,9 @@
 .search-pubDate {
   color: dimgrey;
 }
+.userbox {
+  float: right;
+}
 .username {
   color: lime;
 }

diff --git a/src/facebook.opa b/src/facebook.opa
@@ -65,14 +65,9 @@ Please re-run your application with: --fb-config option")
 
   xhtml =
     login_url = FBA.user_login_url([], redirect)
-    WBootstrap.Button.make(
-      { button:
-         <img style="width:18px; height:18px; vertical-align:top;" title="Facebook" src="https://opalang.org/sso/img/fb-icon.png" alt="Connect with Facebook" />
-         <span>Facebook</>
-      , callback: function(_) { Client.goto(login_url) }
-      },
-      []
-    )
+    <a onclick={function (_) { Client.goto(login_url) }}>
+      <img src="resources/img/facebook_signin.png" />
+    </>
 
   function login(token) {
     match (FBA.get_token_raw(token, redirect)) {

diff --git a/src/webshell.opa b/src/webshell.opa
@@ -60,9 +60,7 @@ function login_box() {
   function block(content) {
     <h3 style="float: right">{content}</>
   }
-  login =
-    prompt = <a>You can sign in with:</>
-    block(<>{prompt}{FacebookConnect.xhtml}</>)
+  login = <>{FacebookConnect.xhtml}</>
   logout =
     function do_logout(_) {
       Login.set_current_user({guest})
@@ -71,10 +69,12 @@ function login_box() {
     name = <a>{Login.get_current_user_name()}</>
     button = WBootstrap.Button.make({ button: <>Logout</>, callback: do_logout}, [])
     block(<>{button}{name}</>)
-  match (Login.get_current_user()) {
-    case {guest}: login
-    default: logout
-  }
+  content =
+    match (Login.get_current_user()) {
+      case {guest}: login
+      default: logout
+    }
+  <span class=userbox>{content}</>
 }
 
 function page() {