Comments by Jesse Andrews on scripts
51 comments
|
Comment on: |
lazyttrick, it was, I was a little over-aggressive and disabled all attributes for 30 minutes before realizing what I had done. I updated your script - http://userscripts.org/scripts/show/32579 - to show the png. Basically I clicked "edit metadata" then clicked save. |
|
Comment on: |
jerone, That is an oversight, I'll add it in to the next deploy :) MissGreen, The idea is to work towards moderation, unfortunately the class attributes they were using was causing the entire page to be overtaken. They were using the "editbox" css (which is displayed above all the other content on the page). It is what happens when you click "add comment" |
|
Comment on: |
I'm using whitelist - currently I'm allowing: tags:strong em b i p code pre tt output samp kbd var sub sup dfn cite big small address hr br div span h1 h2 h3 h4 h5 h6 ul ol li dt dd abbr acronym a img blockquote del ins fieldset legend attributes: href src width height alt cite datetime title dir Although there is strict requiremenst for src/href (must be http/https/other valid protocols) |
|
Comment on: |
italics is really annoying! ;p |
|
Comment on: |
I added an id to the caution box "install_warning" so you don't have to remove all notices & warnings. Sometimes the warnings are due to missing fields in forms. And notices for new features. |
|
Comment on: |
the gravatar was failing due to the email not being lowercased before the API call to gravatar. This has been fixed - no need to use this script - unless there is a reason that "display_name" must be lowercase? |
|
Comment on: |
@lazyttrick, email me (anotherjesse at gmail) if you need a json/xml feed to make the diffing/checking easier. |
|
Comment on: |
@znerp, I'll use that script as a test case and work on improving that. |
|
Comment on: |
Hi, I'm a userscripts.org admin - if you want to remove the ad, I put "sponsor" and "ad" as classes on the div - so it is much easier to remove than what you did. Either use the dom to remove the element or add a style to all userscripts.org pages:
Also ads are only shown when you aren't logged in, so it is rather easier to just login ;) |
|
Comment on: |
@Aquilax, I'm doing things in steps instead of doing huge rewrites - installing and viewing older versions is going to be possible (although there will be strong verbage suggesting that users should use the newest versions unless instructed) |
|
Comment on: |
In the last day there have been:
|
|
Comment on: |
@Photodeus, I'm going to revise the update step to ask the user to give a (optional) description of the change. I'll expose diff functionality to be able to look at code changes. |
|
Comment on: |
@jerone, there hasn't been visible progress, but progress on backend stuff has been happening. I'll try to get some user-facing features released this weekend. |
|
Comment on: |
I fixed that bug a few days ago - thanks for the heads up! |
|
Comment on: |
@Raffles, I've updated that (and it has a list of fans) |
|
Comment on: |
The stats are how many times someone has requested the install page. Some people game the numbers or have update scripts that don't check for updates properly. |
|
Comment on: |
Deployed my first fix based on feedback - http://userscripts.uservoice.com/suggestions/4768 It was an easy one since the feeds had already been implemented. |
|
Comment on: |
Pretty sure the installs were gamed |
|
Comment on: |
jk- I understand your concerns and I have the same! That is why we've not implemented it yet - I want to make sure it isn't breakable. |
|
Comment on: |
jerone, you can see what scripts of your are unlisted on your scripts listing. None of your scripts are unlisted. On the other hand, you can see this user's scripts are all unlisted. |
|
Comment on: |
"dirty word" filters is live - if your script gets unlisted contact the admins and we will re-enable it .... |
|
Comment on: |
Hotlinking wouldn't be an issue for scripts that haven't been accepted since: 12345.user.js isn't a file on the filesystem, it is a route to a rails call. In the rails call I can see if the user is logged in or not, if they aren't they are sent a 403 FORBIDDEN. Scripts that are accepted could be "hotlinked". The reason I don't want to disable that is that there are valid usecases of grabbing the .user.js without visiting us.o first, and you cannot trust referrer (some firewalls remove it, and there are techniques for removing it via nested iframes) |
|
Comment on: |
Great feedback. I'm not deploying fundamental changes like this yet because I want your feedback :) A few responses: trish: "And email notifications when one of my favorite scripts is updated, but I'm not holding my breath" there is an rss feed for updates, but I'm hesitant to add email since it is hard work to stay out of "spam" category if I start sending lots of emails. Raffles: "Better categorisation is essential and if anything, the ability to sort search results by number of installs should be a priority" You can click on the word "installs" in any script listing and it will sort via install count (also name, comments, updated date) ----- I think voting for comments and scripts is important and I will post an article about my questions about my concerns with voting on scripts. |
|
Comment on: |
Summarizing, by connecting a tag with a community a lot of things become easy, but the edge cases - when a script belongs in a community but hasn't been tagged, when a script has been tagged but shouldn't be in the community, and rankings need to be worked out. With respect to rankings, scripts that one community finds useful might not be useful for general people or another community. But does that mean voting is isolated and only affects it's placement that community? Is there an "overall" and community scores. Perhaps scripts can be marked controversial if they have high scores in some/a community and low other places? |
|
Comment on: |
I'm checking on new scripts several times a day... ASAP I'll want to move to not having to do this manually. What I don't want to do is ban all orkut scripts... |
