Skip to content
Commits on May 24, 2011
  1. @dhirenp

    Add FB xmlns to html element.

    dhirenp committed May 23, 2011
  2. @dhirenp

    Add FB xmlns to html element.

    dhirenp committed May 23, 2011
  3. @dhirenp

    Add FB xmlns to html element.

    dhirenp committed May 23, 2011
  4. @dhirenp

    Add FB xmlns to html element.

    dhirenp committed May 23, 2011
  5. @dhirenp

    Add FB xmlns to html element.

    dhirenp committed May 23, 2011
  6. @dhirenp

    Add FB xmlns to html element.

    dhirenp committed May 23, 2011
  7. @dhirenp

    Add FB xmlns to html element.

    dhirenp committed May 23, 2011
Commits on Mar 10, 2011
  1. @jeffbowen
Commits on Nov 25, 2010
  1. No commit message

    dpurdy committed Nov 25, 2010
Commits on Nov 20, 2010
  1. No commit message

    dpurdy committed Nov 19, 2010
Commits on Jun 15, 2010
  1. create fb:friendpile, a legal-friendly version of fb:facepile

    yariv committed Jun 15, 2010
    Summary:
    as the title says
    
    @Naitik, this isn't working and I'm not sure why. Any ideas? I see a 'Tag
    FB.XFBML.Friendpile was not found'
    error in the console.
    
    Test Plan:
    in http://fbrell.com/?server=www.yariv.dev238, entered
    <fb:friendpile></fb:friendpile>
    
    DiffCamp Revision: 123657
    Reviewed By: naitik
    CC: brent, naitik, davidrecordon, achao
    Tasks:
    #215544: Change "Facepile" name and tag to something else without "Face(s)"
    
    Revert Plan:
    OK
Commits on Jun 9, 2010
  1. JS component of the fb:login-button with iframe + perms fix.

    yariv committed Jun 9, 2010
    Summary:
    see http://tools.intern.facebook.com/D121897.
    
    Test Plan:
    see http://tools.intern.facebook.com/D121897
    
    DiffCamp Revision: 121898
    Reviewed By: jcain
    CC: ahaugen, jcain, brent
    Tasks:
    #213799: login-button w/ faces (perms)
    
    Revert Plan:
    OK
  2. connect js component of render-in-iframe for fb:login-button

    yariv committed Jun 4, 2010
    Summary:
    This allows the developer to pass render-in-iframe="true" to fb:login-button,
    to force the widget to render in an iframe.
    
    Test Plan:
    in http://fbrell.com/?server=www.yariv.dev238, entered <fb:login-button
    render-in-iframe="true"></fb:login>, and verified the button was rendered in an
    iframe with no faces. Also checked some of the existing configs of fb:login and
    fb:facepile to make sure they didn't break. (Btw, we really need to unit test
    this stuff.)
    
    DiffCamp Revision: 120838
    Reviewed By: brent
    CC: ahaugen, brent, yariv
    Tasks:
    #204567: create an iframe param for <fb:login-button>
    
    Revert Plan:
    OK
Commits on Jun 4, 2010
  1. @daaku

    auto refresh session every 20 minutes

    daaku committed Jun 4, 2010
    Summary:
    adds logic to setup a timer to ping login_status every 20 minutes. we
    only do this if getLoginStatus() was invoked at least once. this ensures
    that applications that do not want to fetch sessions in JS will not
    get this timer. if the session will not expire, we do not setup a timer
    either.
    
    Test Plan:
    For testing, I reduced the timeout from 20 minutes to 30 seconds. Then
    tested that the timer is setup on page load, if a session is found, and
    on login, if a session is returned.
    
    DiffCamp Revision: 120550
    Reviewed By: wzhu
    CC: platform-diffs@lists, lshepard, brent, mattwkelly, wzhu, selekman
    Revert Plan:
    OK
  2. @daaku

    fix m.js reference

    daaku committed May 25, 2010
    Test Plan:
    None
    
    DiffCamp Revision: 117799
    Reviewed By: alpjor
    CC: platform-diffs@lists, alpjor
    Revert Plan:
    OK
Commits on Jun 2, 2010
  1. @daaku

    revert 251069

    daaku committed Jun 1, 2010
    Test Plan:
    None.
    
    DiffCamp Revision: 119682
    Reviewed By: mwebber
    CC: platform-diffs@lists, mwebber, naitik
    Revert Plan:
    OK
  2. @ptarjan

    YUIdoc thinks the first param is the name

    ptarjan committed Jun 1, 2010
    Summary:
    YUIdoc does slightly different things than our coding styles.
    This fix makes
    http://developers.facebook.com/docs/reference/javascript/FB.Canvas.setAutoResize
    have the right names in the parameter table
    
    Test Plan:
    no code changes
    
    DiffCamp Revision: 119773
    Reviewed By: naitik
    CC: lshepard, brent, naitik, mattwkelly, selekman
    Revert Plan:
    OK
Commits on Jun 1, 2010
  1. @ptarjan

    Fix various parts of the docs

    ptarjan committed May 18, 2010
    Summary:
    Someone synced over my doc updates, so I guess I better put
    them in the code. I also removed all references to joey, and hopeuflly
    made all the links work.
    
    Test Plan:
    No code changes
    
    DiffCamp Revision: 119644
    Reviewed By: naitik
    CC: lshepard, brent, naitik, mattwkelly, selekman
    Revert Plan:
    OK
Commits on May 27, 2010
  1. @daaku

    Move facebook.com/facebook-widgets to facebook.com/badges

    daaku committed May 25, 2010
    Summary:
    Goes along with http://tools.intern.facebook.com/D117046
    
    Test Plan:
    None.
    
    DiffCamp Revision: 118196
    Reviewed By: ahimel
    Commenters: mwebber
    CC: platform-diffs@lists, ahimel, mwebber
    Revert Plan:
    OK
Commits on May 26, 2010
  1. @daaku

    Add some lost docs back

    daaku committed May 25, 2010
    Test Plan:
    None.
    
    DiffCamp Revision: 118231
    Reviewed By: brent
    CC: platform-diffs@lists, lshepard, brent, naitik, mattwkelly, selekman
    Revert Plan:
    OK
  2. @daaku

    doc updates

    daaku committed May 25, 2010
    Summary:
    from D117915
    
    Test Plan:
    None.
    
    DiffCamp Revision: 118134
    Reviewed By: btaylor
    CC: platform-diffs@lists, btaylor, lshepard, brent, mattwkelly, selekman
    Revert Plan:
    OK
Commits on May 25, 2010
  1. @daaku

    fan height is a px property. closes #87

    daaku committed May 25, 2010
    Summary:
    http://github.com/facebook/connect-js/issues#issue/87
    
    Test Plan:
    Before:
    http://fbrell.com/bugs/gh87-fan-height
    
    After:
    http://fbrell.com/bugs/gh87-fan-height?server=sb
    
    DiffCamp Revision: 117807
    Reviewed By: rhe
    CC: platform-diffs@lists, rhe
    Revert Plan:
    OK
  2. @daaku

    it's user_message_prompt not user_prompt_message. closes #63

    daaku committed May 25, 2010
    Test Plan:
    None.
    
    DiffCamp Revision: 117803
    Reviewed By: brent
    CC: platform-diffs@lists, lshepard, brent, mattwkelly, selekman
    Revert Plan:
    OK
  3. @daaku

    fix popup position in IE

    daaku committed May 25, 2010
    Test Plan:
    In any IE, trigger the login popup-
    
    Busted in prod:
    http://fbrell.com/
    
    Fixed after this:
    http://fbrell.com/?server=sb
    
    DiffCamp Revision: 117797
    Reviewed By: brent
    CC: platform-diffs@lists, lshepard, brent, naitik, mattwkelly, selekman
    Revert Plan:
    OK
Commits on May 24, 2010
  1. @ptarjan

    Don't use https for static resources

    ptarjan committed May 20, 2010
    Summary:
    If an internal url uses https the outer frame is still using
    http, so we shouldn't base the protocol assumption on the internal frame
    url.
    
    If we do cross domain communication to an outer https url, then rethink
    this.
    
    Test Plan:
    Tested with an app of https://fbrell.com/ as the canvas endpoint. Correctly
    resized for sanbox code, but was fataling with currently deployed code
    
    DiffCamp Revision: 116653
    Reviewed By: naitik
    Commenters: mathewsb
    CC: ptarjan, lshepard, brent, mathewsb, naitik, mattwkelly, selekman
    Tasks:
    #212617: make the fbconnect never use https://
    
    Revert Plan:
    OK
Commits on May 21, 2010
  1. Choose larger default and min width when action="recommend"

    jcain committed May 21, 2010
    Summary:
    Title says it all.  We're seeing some clipping of the
    small button when action is set to button_count and action is
    set to recomment.  90 pixels is just too short for Recommend.
    This doesn't address internationalization issues at all, but
    the client also has the ability to specify the width when typing
    out <iframe.. or <fb:like...
    
    Test Plan:
    Loaded www.jerrycain.com/Z100.html (directed at my
    sandbox) and confirmed the iframe widths were what I wanted them
    to be.
    
    DiffCamp Revision: 116910
    Reviewed By: naitik
    Commenters: cat
    CC: cat, naitik, zhen
    Revert Plan:
    OK
Commits on May 16, 2010
  1. @daaku

    indicate fbconnect and extern correctly

    daaku committed May 5, 2010
    Summary:
    Required for http://tools.intern.facebook.com/D111875
    
    Test Plan:
    Look at the other diff.
    
    DiffCamp Revision: 111878
    Reviewed By: brent
    CC: platform-diffs@lists, lshepard, brent, mattwkelly, selekman
    Revert Plan:
    OK
Commits on May 15, 2010
  1. @ptarjan @daaku

    Add iframe resizing to connect-js library

    ptarjan committed with daaku May 11, 2010
    Summary:
    Uses the newly created /connect/canvas_proxy.php to send
    messages to the canvas that we are inside. The first use of these
    messages is setSize so the internal iframe can tell the external page
    what size it wants to be
    
    Test Plan:
    Created a test iframe app. Called both FB.Canvas.setSize() and
    FB.Canvas.startTimerToSizeToContent() and saw it grow to the right size.
    FBML apps aren't affected. Smart-resizing apps work as well (since I
    disable smart resizing when the other end calls setSize()).
    
    Tested on Chrome 5 + Firefox 3.5 on OSX.
    
    Test app here: http://apps.ptarjan.dev229.facebook.com/paul_app_test/resize
    
    DiffCamp Revision: 113392
    Reviewed By: naitik
    Commenters: mathewsb
    CC: ptarjan, lshepard, brent, mathewsb, naitik, mattwkelly, selekman
    Tasks:
    
    Revert Plan:
    OK
Commits on May 14, 2010
  1. @daaku

    handle existing query strings

    daaku committed May 13, 2010
    Test Plan:
    http://www.naitik.dev575.snc1.facebook.com/fbconnect/tests/index.html?initialize&api&query%20string
    
    DiffCamp Revision: 113930
    Reviewed By: abanker
    Commenters: ptarjan
    CC: platform-diffs@lists, ptarjan, lshepard, brent, naitik, abanker,
    mattwkelly, selekman
    Revert Plan:
    OK
  2. @daaku

    luke changed his name

    daaku committed May 13, 2010
    Test Plan:
    http://www.naitik.dev575.snc1.facebook.com/fbconnect/tests/index.html?xfbmltags%20module%3A%20fb%3Aprofile-pic
    
    DiffCamp Revision: 113957
    Reviewed By: lshepard
    CC: platform-diffs@lists, lshepard
    Revert Plan:
    OK
  3. @daaku

    remove broken/dead loader/component code

    daaku committed May 13, 2010
    Test Plan:
    Ran the test suite.
    
    DiffCamp Revision: 113954
    Reviewed By: alpjor
    CC: platform-diffs@lists, lshepard, brent, mattwkelly, alpjor, selekman
    Revert Plan:
    OK
  4. @daaku

    Default options to empty {}

    daaku committed May 13, 2010
    Summary:
    API Key is not longer required.
    
    Test Plan:
    http://www.naitik.dev575.snc1.facebook.com/fbconnect/tests/index.html?initialize&api
    
    DiffCamp Revision: 113927
    Reviewed By: alpjor
    CC: platform-diffs@lists, lshepard, brent, mattwkelly, alpjor, selekman
    Revert Plan:
    OK
Commits on May 13, 2010
  1. [joey] make joey use total_count, as we do in sharepro

    rhe committed May 13, 2010
    Summary:
    it's currently using share_count
    
    Test Plan:
    http://fbrell.com/?server=www.rhe.devrs006
    <fb:like
    href="http://www.rottentomatoes.com/m/sherlock_holmes_2009/"></fb:like>
    <fb:share-button href="http://www.rottentomatoes.com/m/sherlock_holmes_2009/"
    type="button_count"></fb:share-button>
    
    Confirmed joey matches sharepro numbers.
    
    DiffCamp Revision: 113992
    Reviewed By: naitik
    CC: platform-diffs@lists, ahaugen, naitik
    Revert Plan:
    ok
  2. @daaku

    format=json-strings to handle 64 bit ints

    daaku committed May 12, 2010
    Test Plan:
    http://fbrell.com/fb.api/friends has ints.
    http://fbrell.com/fb.api/friends?server=sb has strings.
    
    DiffCamp Revision: 113511
    Reviewed By: rhe
    CC: platform-diffs@lists, lshepard, brent, mattwkelly, selekman
    Revert Plan:
    OK
Commits on May 10, 2010
  1. @daaku

    fix typoo

    daaku committed May 10, 2010
    Test Plan:
    No error on http://fbrell.com/?server=sb anymore.
    
    DiffCamp Revision: 112844
    Reviewed By: wzhu
    CC: platform-diffs@lists, lshepard, brent, mattwkelly, wzhu, selekman
    Revert Plan:
    OK
Something went wrong with that request. Please try again.