Skip to content

Comments

refactor: candid UI#464

Merged
mergify[bot] merged 5 commits intomasterfrom
candid-refactor
Mar 26, 2020
Merged

refactor: candid UI#464
mergify[bot] merged 5 commits intomasterfrom
candid-refactor

Conversation

@chenyan-dfinity
Copy link
Contributor

@chenyan-dfinity chenyan-dfinity commented Mar 17, 2020

This PR looks large, but it simply cuts idl-ui.ts into two smaller pieces for extensibility:

  • candid-core.ts contains the InputBox and InputForm class definitions. Most of the HTMLElement are factored out, and becomes input to the class.
  • candid-ui.ts is the implementation of the Render and Parse class that specify the actual HTMLElement used for the UI.
  • Move css file out of index.html.

The goal is that developers only need to implement candid-ui.ts for customizing their UIs, and the core.ts always stays the same.

Guide for reviewers: The diff is mostly due to change in package-lock.json. Almost all the code in core and ui stays the same as before. The only thing I add is the interface type for specifying HTMLElement, which are at the top of the ts files.

@chenyan-dfinity chenyan-dfinity marked this pull request as ready for review March 17, 2020 20:52
@chenyan-dfinity chenyan-dfinity requested a review from a team as a code owner March 17, 2020 20:52
@mergify mergify bot merged commit 5957a4c into master Mar 26, 2020
@chenyan-dfinity chenyan-dfinity deleted the candid-refactor branch March 26, 2020 23:19
dfinity-bot added a commit that referenced this pull request Nov 2, 2020
## Changelog for advisory-db:
Branch: master
Commits: [rustsec/advisory-db@a0e59ff2...65b9aa70](rustsec/advisory-db@a0e59ff...65b9aa7)

* [`0da539a2`](rustsec/advisory-db@0da539a) Add unmaintained crate advisory for `safe-nd` ([RustSec/advisory-db⁠#467](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/467))
* [`51fd5e3c`](rustsec/advisory-db@51fd5e3) Assigned RUSTSEC-2020-0063 to safe-nd ([RustSec/advisory-db⁠#469](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/469))
* [`3adba0fc`](rustsec/advisory-db@3adba0f) Add unmaintained crate advisory for `ffi_utils` ([RustSec/advisory-db⁠#464](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/464))
* [`74c2e86f`](rustsec/advisory-db@74c2e86) Assigned RUSTSEC-2020-0064 to ffi_utils ([RustSec/advisory-db⁠#470](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/470))
* [`a949bd46`](rustsec/advisory-db@a949bd4) Add unmaintained crate advisory for `fake_clock` ([RustSec/advisory-db⁠#465](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/465))
* [`00a4c19a`](rustsec/advisory-db@00a4c19) Assigned RUSTSEC-2020-0065 to fake_clock ([RustSec/advisory-db⁠#471](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/471))
* [`3761ab58`](rustsec/advisory-db@3761ab5) Add unmaintained crate advisory for `safe_bindgen` ([RustSec/advisory-db⁠#466](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/466))
* [`d5cf9d76`](rustsec/advisory-db@d5cf9d7) Assigned RUSTSEC-2020-0066 to safe_bindgen ([RustSec/advisory-db⁠#472](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/472))
* [`9757ff20`](rustsec/advisory-db@9757ff2) Add unmaintained crate advisory for `quic-p2p` ([RustSec/advisory-db⁠#468](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/468))
* [`65b9aa70`](rustsec/advisory-db@65b9aa7) Assigned RUSTSEC-2020-0067 to quic-p2p ([RustSec/advisory-db⁠#473](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/473))
mergify bot pushed a commit that referenced this pull request Nov 2, 2020
## Changelog for advisory-db:
Branch: master
Commits: [rustsec/advisory-db@a0e59ff2...65b9aa70](rustsec/advisory-db@a0e59ff...65b9aa7)

* [`0da539a2`](rustsec/advisory-db@0da539a) Add unmaintained crate advisory for `safe-nd` ([RustSec/advisory-db⁠#467](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/467))
* [`51fd5e3c`](rustsec/advisory-db@51fd5e3) Assigned RUSTSEC-2020-0063 to safe-nd ([RustSec/advisory-db⁠#469](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/469))
* [`3adba0fc`](rustsec/advisory-db@3adba0f) Add unmaintained crate advisory for `ffi_utils` ([RustSec/advisory-db⁠#464](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/464))
* [`74c2e86f`](rustsec/advisory-db@74c2e86) Assigned RUSTSEC-2020-0064 to ffi_utils ([RustSec/advisory-db⁠#470](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/470))
* [`a949bd46`](rustsec/advisory-db@a949bd4) Add unmaintained crate advisory for `fake_clock` ([RustSec/advisory-db⁠#465](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/465))
* [`00a4c19a`](rustsec/advisory-db@00a4c19) Assigned RUSTSEC-2020-0065 to fake_clock ([RustSec/advisory-db⁠#471](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/471))
* [`3761ab58`](rustsec/advisory-db@3761ab5) Add unmaintained crate advisory for `safe_bindgen` ([RustSec/advisory-db⁠#466](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/466))
* [`d5cf9d76`](rustsec/advisory-db@d5cf9d7) Assigned RUSTSEC-2020-0066 to safe_bindgen ([RustSec/advisory-db⁠#472](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/472))
* [`9757ff20`](rustsec/advisory-db@9757ff2) Add unmaintained crate advisory for `quic-p2p` ([RustSec/advisory-db⁠#468](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/468))
* [`65b9aa70`](rustsec/advisory-db@65b9aa7) Assigned RUSTSEC-2020-0067 to quic-p2p ([RustSec/advisory-db⁠#473](http://r.duckduckgo.com/l/?uddg=https://github.com/RustSec/advisory-db/issues/473))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants