by

Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
    Jeroen van Warmerdam
    @jerone
    I added a PR OpenUserJs/OpenUserJS.org#449
    Chris Holland
    @Zren
    Ya saw
    https://oujs.herokuapp.com/scripts/Zren/Textarea_Keyboard_Shortcuts
    Jeroen van Warmerdam
    @jerone
    thnx for the help with adding a script to the html

    https://oujs.herokuapp.com/scripts/Zren/Textarea_Keyboard_Shortcuts

    ow cool

    Chris Holland
    @Zren
    Ya, setup a staging env for prs
    Simple as doing scripts/stage 449 :]
    http://i.imgur.com/ZjGGZQ6.png
    Hmmmm....
    It's not really a fair comparison to the earlier ones as the other is production
    Chris Holland
    @Zren
    but... it took longer to load completely
    Jeroen van Warmerdam
    @jerone

    but... it took longer to load completely

    you mean the new site?

    ...with the new code, compared to the old situation?
    Chris Holland
    @Zren
    Yeah, but it's an unfair test
    and it's only the images
    Both before and after load in ~2s
    So it's negible
    Jeroen van Warmerdam
    @jerone
    Just did a small adjustment to make it even less blocking.
    you have time to update the demo again :P
    Jeroen van Warmerdam
    @jerone
    did anyone succesfully added another strategy to the dev?
    besides github
    Chris Holland
    @Zren
    I forget if the dev uses the same client key/secret as production
    but i think localhost:8080 is allowed even if it's not "exact"
    Jeroen van Warmerdam
    @jerone
    the problem is that the dropdown is empty
    http://localhost:8080/user/preferences
    hmmm, this appears to be a bug
    I had to logout and register with same username and with for example Google to add it to my profile.
    Jeroen van Warmerdam
    @jerone
    hmm and now it's gone again
    weird, have to check this out later
    Jeroen van Warmerdam
    @jerone
    just added another PR :P
    OpenUserJs/OpenUserJS.org#460
    Time to go to bed
    Chris Holland
    @Zren
    Hmmm. Lets see if I can finish the PushDescription thing tonight.
    Got distracted last night when I wondered why we still had modelList.js referenced.
    Jeroen van Warmerdam
    @jerone
    that would be cool. i really love that feature (thats why I proposed it :smile: ) )
    I hope that you don't have to change a lot
    modelList.js ?
    Jeroen van Warmerdam
    @jerone
    question: how can I get the script that belongs to a comment?
    I can link a discussion to the comment.
    I can link a category to the discussion.
    But I can't link a script to the category/discussion/comment...
    Chris Holland
    @Zren
    Yeah... The discussion model doesnt link to the script id. You have to guess based on the category. Just ignore showing the most recent discussion for script issues.
    Jeroen van Warmerdam
    @jerone
    was for another thing, will post a PR soon
    Jeroen van Warmerdam
    @jerone
    it was for this OpenUserJs/OpenUserJS.org#475
    Jeroen van Warmerdam
    @jerone
    is there a reason for the big bottom margin on script description?
    body > .container-fluid > .row > .container-fluid.col-sm-8 {
 margin-bottom: 100px; 
}
    Chris Holland
    @Zren

    padding

    Not really.

    Go ahead and remove it.
    Meant to use a media query on it to hide it when viewing on mobile (cuz it's stacked on top of the sidebar) but it's probably better to just remove it.
    Jeroen van Warmerdam
    @jerone
    I noticed it on mobile
    but on desktop its also redundant I guess
    Jeroen van Warmerdam
    @jerone
    website offline?
    oke, online again!
    bordot
    @bordot
    Hello, is anyone around by chance?