Port PageView to spina.
Review Request #12937 — Created April 5, 2023 and submitted
This change ports the base PageView class to spina and TypeScript. As
part of this, the code has been moved into thecommon
subdirectory to
match the bundle that it's a part of.The one change that was required outside of this class was to update the
DiffViewerPageModel
to usepreinitialize
instead ofconstructor
.
The use of the reserved nameconstructor
has bitten us in the past,
and wasn't working anymore now that the base class was an ES6 class.
- Ran js-tests.
- Verified that page views still rendered correctly.
Summary | ID |
---|---|
bf63aeed7cb7febfec71b026b87c46c6acc793c9 |
Description | From | Last Updated |
---|---|---|
Since this was accessible on the prototype, but isn't really an instance variable, I wonder if we should make this … |
chipx86 | |
Can we work to document all instance variables going forward? |
chipx86 | |
We can return super.remove() instead. |
chipx86 | |
Since this has multiple types involved, let's move actionId to its own line, so the :'s don't all run together. |
chipx86 | |
Are you able to just import 'jasmine-core' and get the same results? I've resolved linting locally with just an import … |
chipx86 |
-
-
Since this was accessible on the prototype, but isn't really an instance variable, I wonder if we should make this
static
and include it in@spina({prototypeAttrs: [...]})
? -
-
-
Since this has multiple types involved, let's move
actionId
to its own line, so the:
's don't all run together. -
Are you able to just
import 'jasmine-core'
and get the same results? I've resolved linting locally with just animport 'jasmine'
, and wasn't sure if it'd work the same way.
- Commits:
-
Summary ID 4b61d130832830223195d265e9b0e02c161cbd6d bf63aeed7cb7febfec71b026b87c46c6acc793c9 - Diff:
-
Revision 2 (+354 -190)