[WIP] turn on collapseProperties#2951
Conversation
|
continuing discussion on this PR. ads.min and everything.min looks good. |
420934f to
70a1b60
Compare
|
Can you post a gist of v0.js and some random extension? |
|
@jridgewell so i didnt run v0.js through the collapseProps since something definitely broke, do you want the version that would have been ran through collapse properties? |
|
@jridgewell https://gist.github.com/erwinmombay/75c6e289554b7317728f54673b3bb902 contains amp-audio.js and a v0.js transpiled with collapse properties turned on. |
|
|
|
|
|
@jridgewell with labels collapsed but with readable names https://gist.github.com/erwinmombay/d2eb139399cc6c6a96fbb8545b8534e6 |
|
Will be interesting to take a deeper look when I have some time. A page On Wed, Apr 20, 2016 at 2:13 PM, erwin mombay notifications@github.com
|
|
@cramforce so looking at everything.amp.min.html (and ads.min) with a v0.js that has collaps-props on, i don't see any errors, but its failing on travis consistently with a disconnect (see latest build) and i haven't been able to replicate the same error locally yet but that might also just be because I haven't been able to get a green local test run in a long time now. |
|
Better than expected :) On Thu, Apr 21, 2016 at 9:42 AM, erwin mombay notifications@github.com
|
|
@cramforce closing this in favor of #2972 |
|
reopening just for travis testing/debugging |
3bae066 to
8c92c95
Compare
No description provided.