Un-used function. Clean-up is good. Etc.
Version: unspecified
Severity: enhancement
Un-used function. Clean-up is good. Etc.
Version: unspecified
Severity: enhancement
Status | Subtype | Assigned | Task | ||
---|---|---|---|---|---|
Resolved | Jdforrester-WMF | T74156 VisualEditor: Remove ve.bind() | |||
Resolved | Krenair | T64762 VisualEditor: Get rid of $.proxy, use native .bind() |
<James_F> Krinkle: Thoughts on actually removing ve.bind rather than just removing its use?
<Krinkle> James_F: Not just yet. Maybe in 1-2 weeks time?
<James_F> Krinkle: For whom do you think we should wait?
<Krinkle> James_F: documentation, community, etc.
<James_F> Krinkle: Eh.
<Krinkle> James_F: We may not have a public API yet, but that doesn't mean it's cool to break stuff without at least giving them the chance to fix is.
<Krinkle> random google search: https://www.mediawiki.org/wiki/VisualEditor/Adding_instrumentation_how-to
resolved later
gerritadmin wrote:
Change 169644 had a related patch set uploaded by Alex Monk:
Remove more references to ve.bind
gerritadmin wrote:
Change 169644 merged by jenkins-bot:
Remove more references to ve.bind
gerritadmin wrote:
Change 171002 had a related patch set uploaded by Jforrester:
[BREAKING CHANGE] Remove the ve.bind function
gerritadmin wrote:
Change 171002 merged by jenkins-bot:
[BREAKING CHANGE] Remove the ve.bind function