Port FloatingBannerView and ReviewReplyDraftBannerView to spina.
Review Request #12810 — Created Jan. 26, 2023 and submitted
In order to be able to subclass the FloatingBannerView for the new
unified banner, we first need to update it to use TypeScript and spina.
This also required updating the ReviewReplyDraftBannerView to match.
- Verified that review reply banners worked correctly.
- Ran js-tests.
Summary | ID |
---|---|
4d080dde93dea99d9f29c6216540fb3dda846ae2 |
Commits: |
|
|||||||
---|---|---|---|---|---|---|---|---|
Diff: |
Revision 2 (+410 -272) |
Checks run (2 succeeded)
flake8
passed.
JSHint
passed.
-
-
-
This can use
modelEvents
:modelEvents = { 'saving destroying': ..., 'saved': ..., 'publishError': ..., };
Works just like
events
, so it should be mappable to a function or to a() => ...
if you want. -
reviewboard/static/rb/js/ui/index.ts (Diff revision 2) Do we want to maybe move general UI stuff into a
RB.UI
namespace?
Change Summary:
Made requested changes.
Commits: |
|
|||||||
---|---|---|---|---|---|---|---|---|
Diff: |
Revision 3 (+406 -272) |
Checks run (2 succeeded)
flake8
passed.
JSHint
passed.