Bug 1535725 Comment 8 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

I was just about to say that the lot in folderDisplay.js is for the fake box object.

Magnus and Alta88, can you suggest a replacement for:
https://hg.mozilla.org/comm-central/rev/d167921e42eb#l6.720
https://hg.mozilla.org/comm-central/rev/d167921e42eb#l6.923

So that leaves the Calendar stuff
```
calendar:
.\base\content\calendar-multiday-view.xml: let sbo = scrollbox.boxObject;
.\base\content\calendar-multiday-view.xml: let propertyValue = scrollbox.boxObject.firstChild.boxObject[propertyName];
```

https://searchfox.org/comm-central/rev/b2c998eb41d696d31a87384487514e9b386f201c/calendar/base/content/calendar-multiday-view.xml#1373
https://searchfox.org/comm-central/rev/b2c998eb41d696d31a87384487514e9b386f201c/calendar/base/content/calendar-multiday-view.xml#3687

For the first call site, we replace `let sbo = scrollbox.boxObject; ... ; sbo.scrollBy( ... );` with `scrollbox.scrollBy( ... );` ?

I'm happy for someone else to provide those tweaks after I've done the mass replacement.
I was just about to say that the lot in folderDisplay.js is for the fake box object.

Magnus and Alta88, can you suggest a replacement for:
https://hg.mozilla.org/comm-central/rev/d167921e42eb#l6.720
https://hg.mozilla.org/comm-central/rev/d167921e42eb#l6.923

So that leaves the Calendar stuff
```
calendar:
.\base\content\calendar-multiday-view.xml: let sbo = scrollbox.boxObject;
.\base\content\calendar-multiday-view.xml: let propertyValue = scrollbox.boxObject.firstChild.boxObject[propertyName];
```

https://searchfox.org/comm-central/rev/b2c998eb41d696d31a87384487514e9b386f201c/calendar/base/content/calendar-multiday-view.xml#1373
https://searchfox.org/comm-central/rev/b2c998eb41d696d31a87384487514e9b386f201c/calendar/base/content/calendar-multiday-view.xml#3687

For the first call site, we replace `let sbo = scrollbox.boxObject; ... ; sbo.scrollBy( ... );` with `scrollbox.scrollBy( ... );` ?

I'm happy for someone else to provide those tweaks after I've done the mass replacement.

EDIT:
BTW, https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=0fca98077f6a18a6489c25e1a32bd8b29a99debb is green, so the mass replacement was successful as far as we can tell.

Back to Bug 1535725 Comment 8