Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
javascript
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
23 May 2015 at 20:54 UTC
Updated:
16 Jul 2015 at 07:14 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
tohesi commentedComment #2
droplet commentedBig changes. Applying patch, top right edit button disappeared
Comment #3
nod_Yeah their changelog is massive it's been more than a year since their last realease: http://backbonejs.org/#changelog
Too bad jessebeach is not around anymore.
It's a novice issue in the sense that a backbone dev should find it relatively easy to fix.
Comment #4
droplet commentedthis issue: https://github.com/jashkenas/backbone/issues/3611
Only 2 places used this pattern, we can do code refactoring before upstream fixing. It's even more readability.
Comment #5
nod_Need to add the unminified backbone source too (for the map file).
Comment #6
nod_Comment #7
chananapeeyush commentedUpdated issue summary.
Comment #8
tohesi commentedVersion 1.2.1 already out. Updated files, issue title & summary accordingly.
Comment #10
tohesi commentedForgot to update the test.
Comment #11
tohesi commentedComment #14
tohesi commentedComment #15
nod_Contextual links are working, toolbar, quickedit, ckeditor and the tour module as well.
Comment #18
naveenvalechaSetting RTBC :) As already by @nod_, Also Tested patch locally,Ckeditor, toolbar is working fine :)
Comment #20
dcmul commented@nod_ should we not look into the reasons why the tests are failing before RTBC? or is this an exception.
Comment #21
dcmul commentedpatch failing
Comment #22
dcmul commentedI have done a re-roll.
Comment #23
daffie commented@dcmul: Can you tell me what you changed in the reroll. If you are able to please add an interdiff.txt.
We do not need this file. We have backbone-min.js and backbone-min.map for that.
Comment #24
nod_No we need it, the map file is useless without the original source. Needs to be in the patch.
Comment #25
daffie commentedOK I nod_ says we need the file then we need the file. To RTBC.
Comment #28
daffie commentedBack to RTBC.
Comment #30
webchickCommitted and pushed to 8.0.x. Thanks!
Thanks also for the manual testing. Always spooky when updating these JS libs. :)