Desktop Container API Review
See OpenFin API documentation & document donated by @Former user (Deleted)
Please comments in the sections below, please label your comments with your name for each of identification
General Feedback on the overall structure of the API
Name | Subject | Comment |
|---|---|---|
@Matthew Gardner | naming convention |
|
@Former user (Deleted) | API equivalence |
Should be noted that we abstract postMessage and use it to make it easier to run our apps on browser/container mode ... |
Modules
Name | API | Comment |
@Former user (Deleted) | setTrayIcon | Not clear how to support more practical tray icon functionality like menus, animations and more complex events. |
@Former user (Deleted) | General | No native dialogs ? |
@Matthew Gardner |
| Also need to preserve snippet tool for pulling in screenshots etc @Former user (Deleted) - There are a couple of dedicated Electron modules for this. |
Name | API | Comment |
|---|---|---|
|
|
|
Name | API | Comment |
|---|---|---|
@Former user (Deleted) | General | Seems like a positional wrapper around a standard sub-window, which is fine (we have one too) ... |
@Matthew Gardner |
| Should we match the work already done in this area https://www.w3.org/TR/notifications/ @Former user (Deleted) - This can (is) used by us on Electron for simple notifications ... |
Name | API | Comment |
|---|---|---|
|
|
|
Name | API | Comment |
|---|---|---|
|
|
|
Classes
Name | API | Comment |
|---|---|---|
|
|
|
Name | API | Comment |
|---|---|---|
|
|
|
Name | API | Comment |
|---|---|---|
|
|
|
Feedback Inter Op Strawman
Name | Subject | Comment |
|---|---|---|
Matthew Gardner |
| Looks good (certainly more listed than I could think of ) - be good to use them - that's the real test of an API - when they get used |