Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge FurMe with Twinkle #15

Closed
Amalthea opened this issue Jun 5, 2011 · 7 comments
Closed

Merge FurMe with Twinkle #15

Amalthea opened this issue Jun 5, 2011 · 7 comments

Comments

@Amalthea
Copy link
Collaborator

Amalthea commented Jun 5, 2011

From TW-B-414:
;Rtrim issue on Google Chrom
Alright, I'm using both Firefox and Chrome for Ubuntu, and I'm running into an interesting issue on a script which relies on Twinkle. I don't know if it's your fault or AWeenieMan's fault, but Chrome is getting stuck on something that FF isn't.

The code is the following (line 377), [[User:AWeenieMan/furme.js]]:
:editSummary += 'adding ' + imageTags.replace('\n', ', ').rtrim(', ');

Firefox will jump into line 1120, [[User:AzaToth/morebits.js]], into the following function:
:String.prototype.rtrim = function stringPrototypeRtrim( chars )

Google Chrome, however, errors out with this message:

:Uncaught TypeError: Object {{tl|Copy to Wikimedia Commons}}, has no method 'rtrim' {{right|1=/w/index.php?title=User:AWeenieMan/furme.js&action=raw&ctype=text/javascript:377}}
:furmeEditText {{right|1=/w/index.php?title=User:AWeenieMan/furme.js&action=raw&ctype=text/javascript:377}}
:furme {{right|1=/w/index.php?title=User:AWeenieMan/furme.js&action=raw&ctype=text/javascript:132}}
:addOnloadHook {{right|1=bits.wikimedia.org/skins-1.5/common/wikibits.js?283-5:52}}
:(anonymous function) {{right|1=/w/index.php?title=User:AWeenieMan/furme.js&action=raw&ctype=text/javascript:52}}

[[User:Magog the Ogre|Magog the Ogre]]([[User talk:Magog the Ogre|talk]]) 22:49, 29 October 2010 (UTC)

PS. I've tried it on Windows and it fails there too. It can be reproduced by installing FURME, going to [[:File:Harry Bemis.jpg]] and clicking tag -> and choosing any of the options below (i.e., the ones not in the dropdown), and clicking submit. [[User:Magog the Ogre|Magog the Ogre]]([[User talk:Magog the Ogre|talk]]) 23:01, 29 October 2010 (UTC)

@atlight
Copy link
Collaborator

atlight commented Jun 8, 2011

If FurMe is upgraded to a ResourceLoader gadget that uses the new morebits, and this bug still occurs, we can look into fixing it. Otherwise, it's not worth worrying about - we've dropped support for the old morebits.

@Ebe123
Copy link
Contributor

Ebe123 commented Jun 11, 2011

It would of been good if we keeped a copy of the old twinkle for FurMe.

@ghost ghost assigned atlight Jun 17, 2011
@Amalthea
Copy link
Collaborator Author

Moving it to you, TTO -- I assume you'll test the new FurMe in Chrome anyway, and hopefully will just be able to close it. :)

@atlight
Copy link
Collaborator

atlight commented Aug 28, 2013

De-assigning myself - I have not worked on this for years.

@Amorymeltzer
Copy link
Collaborator

Closing as this has stalled. Image tagging has been happy and healthy in Twinkle for a while now (see ab9f789) and we support Di options, so I'm not certain there's much else we'd really want anyway.

@siddharthvp
Copy link
Member

@Amorymeltzer Has all of FurMe been merged into TW? The note at WP:FurMe says "image maintenance tagging has been partially merged". If so, it'd be better to leave this open to allow an interested editor to merge the rest.

@Amorymeltzer
Copy link
Collaborator

That's precisely my point. The work has been entirely abandoned for the better part of a decade — the branch was last touched in 2011 — and there's no indication anyone still wants the rest of it. The main thing missing are the extensive fair use stuff — Non-free currency-EU coin national and all that — so if there's a clamoring for it, sure, but the rest of it isn't relevant and it's not something that, as far as I can see, anyone wants to actively pursue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants